Thread (28 messages) 28 messages, 2 authors, 2018-02-21

Re: [PATCH v2] blk-mq: Fix race between resetting the timer and completion handling

From: "tj@kernel.org" <tj@kernel.org>
Date: 2018-02-21 19:21:11

Hello, Bart.

On Wed, Feb 21, 2018 at 06:53:05PM +0000, Bart Van Assche wrote:
On Sun, 2018-02-18 at 05:11 -0800, tj@kernel.org wrote:
quoted
On Wed, Feb 14, 2018 at 04:58:56PM +0000, Bart Van Assche wrote:
quoted
With this patch applied the tests I ran so far pass.
Ah, great to hear.  Thanks a lot for testing.  Can you please verify
the following?  It's the same approach but with RCU sync batching.
Hello Tejun,

After having merged kernel v4.16-rc2 into my kernel tree and after having
applied patch "Avoid that ATA error handling hangs"
(https://www.mail-archive.com/linux-scsi@vger.kernel.org/msg71145.html) I
have not been able to reproduce the reported crash - neither with the patch
applied that was posted on February 13th nor without that patch. This makes
me wonder whether we should drop the discussed patches unless someone comes
up with a reproducible test case?
It is an actual bug in that we actually can override the timer setting
of the next request instance.  Given that the race window isn't that
large, it makes sense that the reproducibility is affected by
butterflies.  I think it makes sense to fix the bug.  Any chance you
can test the new patch on top of the reproducible setup?

Thanks.

-- 
tejun
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help