[Xen-devel] Re: [PATCH] make c/s 21089 work again with c/s 21092

["Jan Beulich" <jbeulich@xxxxxxxxxx>]

>>> Bastian Blank  04/14/10 9:16 AM >>>
>On Tue, Apr 13, 2010 at 02:36:55PM +0100, Jan Beulich wrote:
>> +    spin_lock_irqsave(&desc->lock, flags);
>
>Why do you lock here? The whole thing is racy anyway and this should only
>battle against concurent registrations from dom0.

It seems cleaner to do proper locking here, even though I too think it's not 
strictly needed.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel