<p dir="ltr"><br>
On Nov 3, 2014 6:27 PM, "Mikulas Patocka" <<a href="mailto:mpatocka@redhat.com">mpatocka@redhat.com</a>> wrote:<br>
><br>
><br>
><br>
> On Tue, 28 Oct 2014, Mike Snitzer wrote:<br>
><br>
> > I pushed rebased versions of these patches to linux-dm.git's<br>
> > 'dm-for-3.19' branch (and pulled into 'for-next' purely to get the<br>
> > <a href="http://kernel.org">kernel.org</a> autobuilders testing the code).  See top 3 commits here:<br>
> > <a href="https://git.kernel.org/cgit/linux/kernel/git/device-mapper/linux-dm.git/log/?h=dm-for-3.19">https://git.kernel.org/cgit/linux/kernel/git/device-mapper/linux-dm.git/log/?h=dm-for-3.19</a><br>
> ><br>
> > I'm open to any and all changes or dropping code entirely; so me staging<br>
> > like this is just to keep the review ball rolling as quickly as<br>
> > possible.<br>
> ><br>
> > But in particular we need to get Mikulas' and Bryn's feedback on how the<br>
> > dm_internal_{suspend,resume} changes impact dm-stats.<br>
> ><br>
> > Mike<br>
><br>
> Hi<br>
><br>
> As I said on irc - it is not correct to take a mutex from one syscall and<br>
> drop the mutex from another syscall.</p>
<p dir="ltr">I didn't modify that aspect of dm_internal_suspend+resume.  I only extended the interface to other targets via export.</p>
<p dir="ltr">> I hope Joe can use the bio prison to block bios while the pool is<br>
> suspended.</p>
<p dir="ltr">We'll see.<br>
</p>