[Libguestfs] [PATCH nbdkit] locks: Cache the plugin thread model.

Eric Blake eblake at redhat.com
Fri Jan 19 15:15:05 UTC 2018


On 01/19/2018 09:06 AM, Richard W.M. Jones wrote:
> Commit 876d715e17a59bd25df53be34b942afd96e52da9
> ("Refactor plugin_* functions into a backend struct.") causes hidden
> crashes in the test suite (which unfortunately don't cause tests to
> fail).

Bummer about the non-failure in the testsuite making it harder to see
the problem.

> 
> These all come from handle_single_connection where we may lock the
> connection, run a single connection, free the backend, and then try to
> unlock the connection.  Unfortunately the unlock operation has to
> check the thread model again which fails because the backend has gone
> away:

Yep, makes sense.  Patch looks good to go.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libguestfs/attachments/20180119/6fe62bc6/attachment.sig>


More information about the Libguestfs mailing list