Commit f793aa53 authored Apr 16, 2014 by Jens Axboe

block: relax when to modify the timeout timer



Since we are now, by default, applying timer slack to expiry times,
the logic for when to modify a timer in the block code is suboptimal.
The block layer keeps a forward rolling timer per queue for all
requests, and modifies this timer if a request has a shorter timeout
than what the current expiry time is. However, this breaks down
when our rounded timer values get applied slack. Then each new
request ends up modifying the timer, since we're still a little
in front of the timer + slack.

Fix this by allowing a tolerance of HZ / 2, the timeout handling
doesn't need to be very precise. This drastically cuts down
the number of timer modifications we have to make.

Signed-off-by: Jens Axboe <axboe@fb.com>

parent 12120077

block/blk-timeout.c

+13 −2

Original line number	Diff line number	Diff line
		@@ -199,8 +199,19 @@ void __blk_add_timer(struct request req, struct list_head timeout_list)
		expiry = round_jiffies_up(req->deadline);

		if (!timer_pending(&q->timeout) \|\|
		time_before(expiry, q->timeout.expires))
		time_before(expiry, q->timeout.expires)) {
		unsigned long diff = q->timeout.expires - expiry;

		/*
		* Due to added timer slack to group timers, the timer
		* will often be a little in front of what we asked for.
		* So apply some tolerance here too, otherwise we keep
		* modifying the timer because expires for value X
		* will be X + something.
		*/
		if (diff >= HZ / 2)
		mod_timer(&q->timeout, expiry);
		}

		}

Admin message