[libvirt] [PATCH v4 12/14] nodedev: Convert virNodeDeviceObj to use virObjectLockable

Erik Skultety eskultet at redhat.com
Fri Jul 14 09:28:18 UTC 2017


> >>   cleanup:
> >> -    if (obj)
> >> -        virNodeDeviceObjUnlock(obj);
> >> +    virNodeDeviceObjEndAPI(&obj);
> >>      testDriverUnlock(driver);
> >>      virNodeDeviceDefFree(def);
> >>      virObjectUnref(dev);
> >> @@ -5596,13 +5595,13 @@ testNodeDeviceDestroy(virNodeDevicePtr dev)
> >>       * taken, so we have to dup the parent's name and drop the lock
> >>       * before calling it.  We don't need the reference to the object
> >>       * any more once we have the parent's name.  */
> >
> > Not that this patch would be touching it directly, but ^this commentary is
> > pretty much useless now, since @parent_name is useless in this function,
> > nodeDeviceDestroy doesn't work that way anymore.
> >
> > Erik
> >
>
> Hmmm, I see said the blind man.  Looks like commit id '7ad479d0b' should
> have removed @parent_name as well.
>
> That should be a separate patch - I can create and add it to the end.

Sure, that was the expected outcome :), judging by a quick look at the hunk
only, I don't see a problem having such patch at the beginning of the series,
do you hit any merge conflicts?

Erik




More information about the libvir-list mailing list