<div dir="ltr"><div>Hi Bart-</div><div><br></div><div>it looks ok to me, these three patches combined with your previous patch that is already in the repo...</div><div><br></div><div><br></div>Acked-by: <a href="mailto:dragan.stancevic@canonical.com">dragan.stancevic@canonical.com</a><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 17, 2016 at 2:57 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">On 08/17/2016 12:36 PM, Dragan Stancevic wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Acked-by: <a href="mailto:dragan.stancevic@canonical.com" target="_blank">dragan.stancevic@canonical.com</a><br>
<mailto:<a href="mailto:dragan.stancevic@canonical.com" target="_blank">dragan.stancevic@canon<wbr>ical.com</a>><span class=""><br>
<br>
I agree with your patch, I have been tracking an issue where multipathd<br>
dumps core on the exit path just past the treads being canceled. Your<br>
patch is very similar to mine (minus nuking the depth) that I was going<br>
to send out to a user to test with. The checker thread accesses a valid<br>
pointer with garbage values....<br>
</span></blockquote>
<br>
Hello Dragan,<br>
<br>
When I prepared my patch I overlooked that multipathd creates most threads as detached threads. So I started testing the three attached patches on my setup. It would be appreciated if you could have a look at these patches.<br>
<br>
Thanks,<br>
<br>
Bart.<br>
</blockquote></div><br></div>