Re: [PATCH v2] block: flush queued bios when the process blocks

2015-10-17 Thread Ming Lei
On Fri, Oct 16, 2015 at 11:29 PM, Mike Snitzer wrote: > On Thu, Oct 15 2015 at 11:08pm -0400, > Ming Lei wrote: > >> On Thu, Oct 15, 2015 at 4:06 PM, Mike Snitzer wrote: >> > On Wed, Oct 14 2015 at 11:27pm -0400, >> > Ming Lei wrote: >> > >> >> On Sat, Oct 10, 2015 at 3:52 AM, Mike Snitzer wro

Re: [PATCH v2] block: flush queued bios when the process blocks

2015-10-16 Thread Mike Snitzer
On Thu, Oct 15 2015 at 11:08pm -0400, Ming Lei wrote: > On Thu, Oct 15, 2015 at 4:06 PM, Mike Snitzer wrote: > > On Wed, Oct 14 2015 at 11:27pm -0400, > > Ming Lei wrote: > > > >> On Sat, Oct 10, 2015 at 3:52 AM, Mike Snitzer wrote: > >> > > >> > Turns out that this change: > >> > http://git.k

Re: [PATCH v2] block: flush queued bios when the process blocks

2015-10-15 Thread Ming Lei
On Thu, Oct 15, 2015 at 4:06 PM, Mike Snitzer wrote: > On Wed, Oct 14 2015 at 11:27pm -0400, > Ming Lei wrote: > >> On Sat, Oct 10, 2015 at 3:52 AM, Mike Snitzer wrote: >> > >> > Turns out that this change: >> > http://git.kernel.org/cgit/linux/kernel/git/snitzer/linux.git/commit/?h=wip&id=26396

Re: [PATCH v2] block: flush queued bios when the process blocks

2015-10-15 Thread Mike Snitzer
On Wed, Oct 14 2015 at 11:27pm -0400, Ming Lei wrote: > On Sat, Oct 10, 2015 at 3:52 AM, Mike Snitzer wrote: > > > > Turns out that this change: > > http://git.kernel.org/cgit/linux/kernel/git/snitzer/linux.git/commit/?h=wip&id=2639638c77768a86216be456c2764e32a2bcd841 > > > > needed to be revert

Re: [PATCH v2] block: flush queued bios when the process blocks

2015-10-14 Thread Ming Lei
On Sat, Oct 10, 2015 at 3:52 AM, Mike Snitzer wrote: > On Thu, Oct 08 2015 at 11:08am -0400, > Mike Snitzer wrote: > >> On Thu, Oct 08 2015 at 11:04am -0400, >> Mikulas Patocka wrote: >> >> > >> > >> > On Tue, 6 Oct 2015, Mike Snitzer wrote: >> > >> > > To give others context for why I'm caring

Re: [PATCH v2] block: flush queued bios when the process blocks

2015-10-09 Thread Mike Snitzer
On Fri, Oct 09 2015 at 3:52pm -0400, Mike Snitzer wrote: > Turns out that this change: > http://git.kernel.org/cgit/linux/kernel/git/snitzer/linux.git/commit/?h=wip&id=2639638c77768a86216be456c2764e32a2bcd841 > > needed to be reverted with: > http://git.kernel.org/cgit/linux/kernel/git/snitzer

Re: [PATCH v2] block: flush queued bios when the process blocks

2015-10-09 Thread Mike Snitzer
On Thu, Oct 08 2015 at 11:08am -0400, Mike Snitzer wrote: > On Thu, Oct 08 2015 at 11:04am -0400, > Mikulas Patocka wrote: > > > > > > > On Tue, 6 Oct 2015, Mike Snitzer wrote: > > > > > To give others context for why I'm caring about this issue again, this > > > recent BZ against 4.3-rc ser

Re: [PATCH v2] block: flush queued bios when the process blocks

2015-10-09 Thread kbuild test robot
Hi Mike, [auto build test ERROR on v4.3-rc4 -- if it's inappropriate base, please ignore] config: x86_64-lkp (attached as .config) reproduce: # save the attached .config to linux build tree make ARCH=x86_64 All errors (new ones prefixed by >>): In file included from include/

Re: [PATCH v2] block: flush queued bios when the process blocks

2015-10-08 Thread Mike Snitzer
On Thu, Oct 08 2015 at 11:04am -0400, Mikulas Patocka wrote: > > > On Tue, 6 Oct 2015, Mike Snitzer wrote: > > > To give others context for why I'm caring about this issue again, this > > recent BZ against 4.3-rc served as a reminder that we _need_ a fix: > > https://bugzilla.redhat.com/show_b

Re: [PATCH v2] block: flush queued bios when the process blocks

2015-10-08 Thread Mikulas Patocka
On Tue, 6 Oct 2015, Mike Snitzer wrote: > To give others context for why I'm caring about this issue again, this > recent BZ against 4.3-rc served as a reminder that we _need_ a fix: > https://bugzilla.redhat.com/show_bug.cgi?id=1267650 > > FYI, I cleaned up the plug-based approach a bit furthe

Re: [PATCH v2] block: flush queued bios when the process blocks

2015-10-06 Thread Mike Snitzer
On Tue, Oct 06 2015 at 4:16P -0400, Mike Snitzer wrote: > To give others context for why I'm caring about this issue again, this > recent BZ against 4.3-rc served as a reminder that we _need_ a fix: > https://bugzilla.redhat.com/show_bug.cgi?id=1267650 > > FYI, I cleaned up the plug-based appro

[PATCH v2] block: flush queued bios when the process blocks

2015-10-06 Thread Mike Snitzer
To give others context for why I'm caring about this issue again, this recent BZ against 4.3-rc served as a reminder that we _need_ a fix: https://bugzilla.redhat.com/show_bug.cgi?id=1267650 FYI, I cleaned up the plug-based approach a bit further, here is the incremental patch: http://git.kernel.o