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 <[email protected]>

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

_______________________________________________
Devel mailing list
[email protected]
https://lists.openvz.org/mailman/listinfo/devel

Reply via email to