Commit de6a78b6 authored Mar 18, 2020 by Ming Lei Committed by Jens Axboe Mar 18, 2020

block: Prevent hung_check firing during long sync IO

submit_bio_wait() can be called from ioctl(BLKSECDISCARD), which
may take long time to complete, as Salman mentioned, 4K BLKSECDISCARD
takes up to 100 second on some devices. Also any block I/O operation
that occurs after the BLKSECDISCARD is submitted will also potentially
be affected by the hung task timeouts.

Another report is that task hang can be observed when running mkfs
over raid10 which takes a small max discard sectors limit because
of chunk size.

So prevent hung_check from firing by taking same approach used
in blk_execute_rq(), and the wake-up interval is set as half the
hung_check timer period, which keeps overhead low enough.

Cc: Salman Qazi <sqazi@google.com>
Cc: Jesse Barnes <jsbarnes@google.com>
Cc: Bart Van Assche <bvanassche@acm.org>
Link: https://lkml.org/lkml/2020/2/12/1193


Reported-by: Salman Qazi <sqazi@google.com>
Reviewed-by: Jesse Barnes <jsbarnes@google.com>
Reviewed-by: Salman Qazi <sqazi@google.com>
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

parent d981cb5b

block/bio.c

+11 −1

Original line number	Diff line number	Diff line
		@@ -17,6 +17,7 @@
		#include <linux/cgroup.h>
		#include <linux/blk-cgroup.h>
		#include <linux/highmem.h>
		#include <linux/sched/sysctl.h>

		#include <trace/events/block.h>
		#include "blk.h"
		@@ -1019,11 +1020,20 @@ static void submit_bio_wait_endio(struct bio *bio)
		int submit_bio_wait(struct bio *bio)
		{
		DECLARE_COMPLETION_ONSTACK_MAP(done, bio->bi_disk->lockdep_map);
		unsigned long hang_check;

		bio->bi_private = &done;
		bio->bi_end_io = submit_bio_wait_endio;
		bio->bi_opf \|= REQ_SYNC;
		submit_bio(bio);

		/* Prevent hang_check timer from firing at us during very long I/O */
		hang_check = sysctl_hung_task_timeout_secs;
		if (hang_check)
		while (!wait_for_completion_io_timeout(&done,
		hang_check * (HZ/2)))
		;
		else
		wait_for_completion_io(&done);

		return blk_status_to_errno(bio->bi_status);

Admin message