[Devel] [PATCH VZ10 1/4] Revert "ve: Guard ve from ve_list with ve_list_lock"

Pavel Tikhomirov ptikhomirov at virtuozzo.com
Tue Dec 2 13:17:45 MSK 2025


This reverts commit f661b76b72b5b36d2e984a621dce0a8d0b86ba1f.

We have those uses of ve_list (vestat_seq_show() and veinfo_seq_show())
already under ve_list_lock via ve_seq_start() + ve_seq_stop(). Taking
the lock another time leads to deadlock.

https://virtuozzo.atlassian.net/browse/VSTOR-118289
Signed-off-by: Pavel Tikhomirov <ptikhomirov at virtuozzo.com>

Feature: ve: ve generic structures
---
 kernel/ve/vecalls.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/kernel/ve/vecalls.c b/kernel/ve/vecalls.c
index a9e82a133540..6caf269fe504 100644
--- a/kernel/ve/vecalls.c
+++ b/kernel/ve/vecalls.c
@@ -120,8 +120,6 @@ static int vestat_seq_show(struct seq_file *m, void *v)
 	struct kernel_cpustat kstat;
 
 	entry = (struct list_head *)v;
-
-	guard(mutex)(&ve_list_lock);
 	ve = list_entry(entry, struct ve_struct, ve_list);
 
 	curve = get_exec_env();
@@ -277,7 +275,6 @@ static int veinfo_seq_show(struct seq_file *m, void *v)
 {
 	struct ve_struct *ve;
 
-	guard(mutex)(&ve_list_lock);
 	ve = list_entry((struct list_head *)v, struct ve_struct, ve_list);
 
 	/* second 0 is deprecated ve->class_id */
-- 
2.51.1



More information about the Devel mailing list