<div dir="ltr">This set is merged.<div>Thanks.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 14, 2016 at 4:59 PM, Benjamin Marzinski <span dir="ltr"><<a href="mailto:bmarzins@redhat.com" target="_blank">bmarzins@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ACK for the set.<br>
<br>
-Ben<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Dec 14, 2016 at 11:05:28AM -0200, Mauricio Faria de Oliveira wrote:<br>
> This patch-set prevents this error message on 'change'<br>
> uevents of paths that are blacklisted:<br>
><br>
>     sdb: spurious uevent, path not found<br>
><br>
> And in that case, replaces it with this _debug_ message<br>
> (so it's not logged by default; only when requested):<br>
><br>
>     sdb: spurious uevent, path is blacklisted<br>
><br>
> Patch 1/3 fixes a potential leak observed while doing that.<br>
> Patch 2/3 minor refactors two blacklist/filter checks into<br>
>           pathinfo() to clean-up a bit and fix a couple of<br>
>           bugs (reported by Ben and Hannes)<br>
> Patch 3/3 finally does what the patch-set is supposed to.<br>
><br>
> Changelog:<br>
>  - v2: switch approach from using path_discover() to using<br>
>        alloc_path_with_pathinfo() (Benjamin Marzinski).<br>
>  - v3: drop calls to udev_device_ref/unref() (Benjamin Marzinski)<br>
>        move filter_property() into pathinfo() (Benjamin Marzinski)<br>
>  - v4: move filter_devnode() into pathinfo() (Hannes Reinecke)<br>
>        ensure equivalency between path_discover() and pathinfo()<br>
>        for multipathd behavior.<br>
><br>
> Mauricio Faria de Oliveira (3):<br>
>   libmultipath: prevent memory leak in alloc_path_with_pathinfo() if<br>
>     pp_ptr is NULL<br>
>   libmultipath: move filter_property|devnode() from path_discover() into<br>
>     pathinfo()<br>
>   multipathd: skip spurious event message for blacklisted paths<br>
><br>
>  libmultipath/blacklist.c |  4 ++++<br>
>  libmultipath/discovery.c | 23 +++++++++++++++--------<br>
>  multipathd/main.c        | 17 ++++++++++++++++-<br>
>  3 files changed, 35 insertions(+), 9 deletions(-)<br>
><br>
> --<br>
> 1.8.3.1<br>
</div></div></blockquote></div><br></div>