Re: [PATCH 30/57] multipathd: skip uninitialized devices during reconfiguration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 27, 2016 at 01:10:31PM +0200, Hannes Reinecke wrote:
> libudev has a separate filter setting to skip uninitialized
> devices during enumeration.

So, I assume if udev filters out a device here, it's because a uevent
should be coming when the device is fully initialized. correct?

The only thing I worry about with taking out all of the backup stuff is
what happens if udev simply times out processing an event, and doesn't
grab the UUID.  In this case we still won't get a UUID unless we force
another event, or grab the information ourselves.

But ACK on the patch itself.

-Ben

> 
> Signed-off-by: Hannes Reinecke <hare@xxxxxxx>
> ---
>  libmultipath/discovery.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/libmultipath/discovery.c b/libmultipath/discovery.c
> index 2fa0907..2cf2566 100644
> --- a/libmultipath/discovery.c
> +++ b/libmultipath/discovery.c
> @@ -148,6 +148,7 @@ path_discovery (vector pathvec, struct config * conf, int flag)
>  		return -ENOMEM;
>  
>  	udev_enumerate_add_match_subsystem(udev_iter, "block");
> +	udev_enumerate_add_match_is_initialized(udev_iter);
>  	udev_enumerate_scan_devices(udev_iter);
>  
>  	udev_list_entry_foreach(entry,
> -- 
> 2.6.6

--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel



[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux