File 60c8a7ac-x86-vpt-fully-init-timers-before-enlis... of Package xen

Overview Repositories Revisions Requests Users Attributes Meta

File 60c8a7ac-x86-vpt-fully-init-timers-before-enlisting.patch of Package xen (Revision 1c8e7fe6c0fc776617671eeb7d750fe7)

Currently displaying revision 1c8e7fe6c0fc776617671eeb7d750fe7 , Show latest

# Commit 6d622f3a96bbd76ce8422c6e3805e6609417ec76
# Date 2021-06-15 15:14:20 +0200
# Author Jan Beulich <jbeulich@suse.com>
# Committer Jan Beulich <jbeulich@suse.com>
x86/vpt: fully init timers before putting onto list

With pt_vcpu_lock() no longer acquiring the pt_migrate lock, parties
iterating the list and acting on the timers of the list entries will no
longer be kept from entering their loops by create_periodic_time()'s
holding of that lock. Therefore at least init_timer() needs calling
ahead of list insertion, but keep this and set_timer() together.

Fixes: 8113b02f0bf8 ("x86/vpt: do not take pt_migrate rwlock in some cases")
Reported-by: Igor Druzhinin <igor.druzhinin@citrix.com>
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Roger Pau Monné <roger.pau@citrix.com>
Reviewed-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>

--- a/xen/arch/x86/hvm/vpt.c
+++ b/xen/arch/x86/hvm/vpt.c
@@ -553,14 +553,14 @@ void create_periodic_time(
     pt->cb = cb;
     pt->priv = data;
 
+    init_timer(&pt->timer, pt_timer_fn, pt, v->processor);
+    set_timer(&pt->timer, pt->scheduled);
+
     pt_vcpu_lock(v);
     pt->on_list = 1;
     list_add(&pt->list, &v->arch.hvm.tm_list);
     pt_vcpu_unlock(v);
 
-    init_timer(&pt->timer, pt_timer_fn, pt, v->processor);
-    set_timer(&pt->timer, pt->scheduled);
-
     write_unlock(&v->domain->arch.hvm.pl_time->pt_migrate);
 }

Places

File 60c8a7ac-x86-vpt-fully-init-timers-before-enlisting.patch of Package xen (Revision 1c8e7fe6c0fc776617671eeb7d750fe7)

Places