<div dir="ltr"><div dir="ltr" class="gmail_attr">On Mon, Nov 25, 2019 at 12:48 PM Martin Wilck <<a href="mailto:mwilck@suse.de">mwilck@suse.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
I think you are seeing this FIXME:<br>
<br>
<a href="https://elixir.bootlin.com/linux/v4.19.79/source/drivers/md/dm-mpath.c#L612" rel="noreferrer" target="_blank">https://elixir.bootlin.com/linux/v4.19.79/source/drivers/md/dm-mpath.c#L612</a><br>
<br>
For your testing, perhaps you just remove that if(!pgpath) condition.<br>
<br>
Regards,<br>
Martin<br></blockquote><br>That's correct, thanks. It works as expected after removing that condition.<br><br>With some limited testing using fio, I don't see any obvious negative impacts of allowing choose_pgpath to run each bio mapping. I also don't see any significant increase in CPU usage or IO latency during benchmarking.<br><br>The FIXME comment acknowledges that this sort of defeats the balancing function of multipath, so I'm assuming this was removed for a good reason. Does anyone know why this was done? In my particular use case, I benefit from balancing the paths for throughput, so it's useful to me.<br><br>Thanks!<br>- Drew</div>