<div dir="ltr">Oh yes, sorry, this patch slipped past my cursor.<div><br></div><div>This patch highlights the lack of consistency in function prototypes in devmapper.c : some using "no_flush", others using "flush".</div><div><br></div><div>Anyway, your patch is now applied.</div><div>Thanks.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 27, 2017 at 4:18 PM, Bart Van Assche <span dir="ltr"><<a href="mailto:bart.vanassche@sandisk.com" target="_blank">bart.vanassche@sandisk.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello Christophe,<br>
<br>
The patch below was posted more than three months ago but has not yet received any feedback from you. Had I sent this patch to the right person? Do you perhaps want me to repost it?<span class="HOEnZb"><font color="#888888"><br>
<br>
Bart.</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On 01/04/17 11:56, Benjamin Marzinski wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Wed, Dec 07, 2016 at 03:37:44PM -0800, Bart Van Assche wrote:<br>
<br>
ACK<br>
<br>
-Ben<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Christophe and Hannes, do you want to comment on this patch?<br>
<br>
Thanks,<br>
<br>
Bart.<br>
<br>
On 11/18/2016 01:33 PM, Bart Van Assche wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Avoid that reloading a map while there are no paths triggers a flush<br>
and hence unwanted I/O errors if 'queue_if_no_path' is enabled.<br>
<br>
Fixes: commit d569988e7528 ("libmultipath: Fixup 'DM_DEVICE_RELOAD' handling")<br>
Signed-off-by: Bart Van Assche <<a href="mailto:bart.vanassche@sandisk.com" target="_blank">bart.vanassche@sandisk.com</a>><br>
Cc: Hannes Reinecke <<a href="mailto:hare@suse.de" target="_blank">hare@suse.de</a>><br>
---<br>
libmultipath/devmapper.c | 2 +-<br>
1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/libmultipath/devmapper.c b/libmultipath/devmapper.c<br>
index f92ebce..31f1962 100644<br>
--- a/libmultipath/devmapper.c<br>
+++ b/libmultipath/devmapper.c<br>
@@ -390,7 +390,7 @@ int dm_addmap_reload(struct multipath *mpp, char *params, int flush)<br>
                              params, ADDMAP_RO, SKIP_KPARTX_OFF);<br>
        }<br>
        if (r)<br>
-               r = dm_simplecmd(DM_DEVICE_RESUME, mpp->alias, flush,<br>
+               r = dm_simplecmd(DM_DEVICE_RESUME, mpp->alias, !flush,<br>
                                 1, udev_flags, 0);<br>
        return r;<br>
}<br>
</blockquote></blockquote></blockquote>
<br>
</div></div></blockquote></div><br></div>