Re: [PATCH blktests v5 2/3] bcache: add bcache/002

On Fri, Mar 06, 2026 at 12:27:51PM +0000, Shinichiro Kawasaki wrote:
> On Mar 05, 2026 / 16:01, Daniel Wagner wrote:
> > Add test case from Stephen Zhang [1][2], which is
> > 
> > 1. Wait for fio to complete (wait $fio_pid)
> > 2. Wait a few seconds for I/O to drain
> > 3. Then check the utilization
> > 
> > If utilization is still > 0% after I/O completes, then there's a real
> > accounting leak.
> > 
> > [1] https://lore.kernel.org/linux-bcache/CANubcdX7eNbH_bo4-f94DUbdiEbt04Vxy1MPyhm+CZyXB01FuQ@mail.gmail.com/
> > [2] https://lore.kernel.org/all/CANubcdU92Uv7wNdZH357LLTnfwPouekXNTPSjHnzYzK99-eZ9w@mail.gmail.com
> 
> Thanks for the updates. It's good that we do not need iostat :)
> 
> I ran this test case with v7.0-rc2 kernel, and it passes. Good.
> 
> I also ran this test case with older kernel without the kernel fix [1]. I
> expected failure, but it passed. I ran the Stephen's original test script and it
> did not fail either. So I guess my test environment does not fulfill the
> conditions to recreate the failure. Daniel, did you observe this test case
> failed with any older kernel?

Same here, it doesn't fail on older kernels either. Maybe we should just
not merge this test then? Maybe Stephen can have a look and figure out
what needs to be done to trigger the condition more reliable.

> Let me leave nit comments in line. If this patch does not respin, I can fold in
> the changes for the nit comments.

Sure, just fix it up if you decide to apply it.

Thanks!
Daniel