[libvirt] [PATCH 1/3] nodedev: Move device enumumeration out of nodeStateInitialize

Erik Skultety eskultet at redhat.com
Thu Dec 14 10:42:13 UTC 2017


On Sat, Dec 09, 2017 at 12:29:12PM -0500, John Ferlan wrote:
> Let's move the udevEnumerateDevices into a thread to "speed
> up" the initialization process. If the enumeration fails we
> can set the Quit flag to ensure that udevEventHandleCallback
> will not run.
>
> Signed-off-by: John Ferlan <jferlan at redhat.com>
> ---
>  src/node_device/node_device_udev.c | 27 +++++++++++++++++++++++++--
>  1 file changed, 25 insertions(+), 2 deletions(-)
>

Speedup's okay (even though I don't expect anything significant), but if
something goes wrong with udev, we start into a sort of crippled state where
you're still able to start machines, but only if these don't make use of NPIV
in a sense that a vHBA would need to be created, or managed direct-assigned PCI
devices. Prior to this change if enumeration failed, the daemon terminated - I
think this is an improvement.

Reviewed-by: Erik Skultety <eskultet at redhat.com>




More information about the libvir-list mailing list