[dm-devel] [PATCH v4 0/3] multipathd: skip spurious event message for blacklisted paths

[Christophe Varoqui]

This set is merged.
Thanks.

On Wed, Dec 14, 2016 at 4:59 PM, Benjamin Marzinski <bmarzins at redhat.com>
wrote:

> ACK for the set.
>
> -Ben
>
> On Wed, Dec 14, 2016 at 11:05:28AM -0200, Mauricio Faria de Oliveira wrote:
> > This patch-set prevents this error message on 'change'
> > uevents of paths that are blacklisted:
> >
> >     sdb: spurious uevent, path not found
> >
> > And in that case, replaces it with this _debug_ message
> > (so it's not logged by default; only when requested):
> >
> >     sdb: spurious uevent, path is blacklisted
> >
> > Patch 1/3 fixes a potential leak observed while doing that.
> > Patch 2/3 minor refactors two blacklist/filter checks into
> >           pathinfo() to clean-up a bit and fix a couple of
> >           bugs (reported by Ben and Hannes)
> > Patch 3/3 finally does what the patch-set is supposed to.
> >
> > Changelog:
> >  - v2: switch approach from using path_discover() to using
> >        alloc_path_with_pathinfo() (Benjamin Marzinski).
> >  - v3: drop calls to udev_device_ref/unref() (Benjamin Marzinski)
> >        move filter_property() into pathinfo() (Benjamin Marzinski)
> >  - v4: move filter_devnode() into pathinfo() (Hannes Reinecke)
> >        ensure equivalency between path_discover() and pathinfo()
> >        for multipathd behavior.
> >
> > Mauricio Faria de Oliveira (3):
> >   libmultipath: prevent memory leak in alloc_path_with_pathinfo() if
> >     pp_ptr is NULL
> >   libmultipath: move filter_property|devnode() from path_discover() into
> >     pathinfo()
> >   multipathd: skip spurious event message for blacklisted paths
> >
> >  libmultipath/blacklist.c |  4 ++++
> >  libmultipath/discovery.c | 23 +++++++++++++++--------
> >  multipathd/main.c        | 17 ++++++++++++++++-
> >  3 files changed, 35 insertions(+), 9 deletions(-)
> >
> > --
> > 1.8.3.1
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/dm-devel/attachments/20161214/985ceded/attachment.htm>