Re: [PATCH v9] block: cancel all throttled bios in del_gendisk()
From: Ming Lei
Date: Mon Feb 28 2022 - 01:12:14 EST
Hi Christoph,
On Sun, Feb 27, 2022 at 09:16:54AM -0800, Christoph Hellwig wrote:
> On Thu, Feb 10, 2022 at 07:56:37PM +0800, Yu Kuai wrote:
> > Throttled bios can't be issued after del_gendisk() is done, thus
> > it's better to cancel them immediately rather than waiting for
> > throttle is done.
> >
> > For example, if user thread is throttled with low bps while it's
> > issuing large io, and the device is deleted. The user thread will
> > wait for a long time for io to return.
>
> FYI, this crashed left rigt and center when running xfstests with
> traces pointing to throtl_pending_timer_fn.
Can you share the exact xfstests test(fs, test)? Or panic log?
I can't reproduce it when running './check -g auto' on XFS, meantime
tracking throtl_pending_timer_fn().
Thanks,
Ming