[libvirt] [PATCH v2 02/14] nodedev: Fix locking in virNodeDeviceObjRemove
John Ferlan
jferlan at redhat.com
Fri May 26 11:52:48 UTC 2017
On 05/26/2017 02:57 AM, Peter Krempa wrote:
> On Thu, May 25, 2017 at 15:56:59 -0400, John Ferlan wrote:
>> The current mechanism doesn't lock each element in devs->objs as it's
>> looking at it, rather it keeps locking/unlocking the passed element for
>> which the removal is being attempted. Fix things to lock each element
>> as we're looking at them.
>
> Since you are comparing only the pointer of the item locking it is
> really not necessary.
>
> Any other justification for this change?
>
Beyond making this like other vir.*ObjRemove APIs, no and fixing what
would be an "obvious" coding error.
I suppose the argument is that the driver/test lock wouldn't allow
change to the list so true technically it doesn't matter. I can drop
this if that's what you'd prefer. Eventually it goes away completely,
but that's code I have in my branches.
John
More information about the libvir-list
mailing list