Looks fine. Reviewed-by: Guan Junxiong <guanjunxiong@xxxxxxxxxx> On 2017/7/14 19:32, Martin Wilck wrote: > We are only interested in disks anyway. This saves us checking > for devtype in the uevent processing queue, and is more powerful > than the previous test in the dropped function > uevent_can_discard_by_devpath(). > > Signed-off-by: Martin Wilck <mwilck@xxxxxxxx> > Reviewed-by: Hannes Reinecke <hare@xxxxxxx> > --- > libmultipath/uevent.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libmultipath/uevent.c b/libmultipath/uevent.c > index b688ca03..eb44da56 100644 > --- a/libmultipath/uevent.c > +++ b/libmultipath/uevent.c > @@ -770,7 +770,7 @@ int uevent_listen(struct udev *udev) > goto out; > } > err = udev_monitor_filter_add_match_subsystem_devtype(monitor, "block", > - NULL); > + "disk"); > if (err) > condlog(2, "failed to create filter : %s", strerror(-err)); > err = udev_monitor_enable_receiving(monitor); > -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
