<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>