* [PATCH] dm: fix parameter to blk_delay_queue()
@ 2016-07-15 13:27 Tahsin Erdogan
2016-07-15 13:59 ` Mike Snitzer
0 siblings, 1 reply; 3+ messages in thread
From: Tahsin Erdogan @ 2016-07-15 13:27 UTC (permalink / raw)
To: Alasdair Kergon, Mike Snitzer, Jens Axboe, dm-devel, Shaohua Li,
linux-raid
Cc: linux-kernel, Tahsin Erdogan
Second parameter to blk_delay_queue() must be in msec units not jiffies.
Signed-off-by: Tahsin Erdogan <tahsin@google.com>
---
drivers/md/dm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/md/dm.c b/drivers/md/dm.c
index 1b2f96205361..17c63265a205 100644
--- a/drivers/md/dm.c
+++ b/drivers/md/dm.c
@@ -2175,7 +2175,7 @@ static void dm_request_fn(struct request_queue *q)
md_in_flight(md) && rq->bio && rq->bio->bi_vcnt == 1 &&
md->last_rq_pos == pos && md->last_rq_rw == rq_data_dir(rq)) ||
(ti->type->busy && ti->type->busy(ti))) {
- blk_delay_queue(q, HZ / 100);
+ blk_delay_queue(q, 10);
return;
}
--
2.8.0.rc3.226.g39d4020
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: dm: fix parameter to blk_delay_queue()
2016-07-15 13:27 [PATCH] dm: fix parameter to blk_delay_queue() Tahsin Erdogan
@ 2016-07-15 13:59 ` Mike Snitzer
2016-07-15 14:18 ` Tahsin Erdogan
0 siblings, 1 reply; 3+ messages in thread
From: Mike Snitzer @ 2016-07-15 13:59 UTC (permalink / raw)
To: Tahsin Erdogan
Cc: Alasdair Kergon, Jens Axboe, dm-devel, Shaohua Li, linux-raid,
linux-kernel
On Fri, Jul 15 2016 at 9:27am -0400,
Tahsin Erdogan <tahsin@google.com> wrote:
> Second parameter to blk_delay_queue() must be in msec units not jiffies.
>
> Signed-off-by: Tahsin Erdogan <tahsin@google.com>
This needs to be rebased against linux-next (or linux-dm.git's
'for-next') because the code in question has been moved out to dm-rq.c
But I'll gladly take care of it.
Your change seems fine, and in fact my intent was always to have it be
10 msec, see commit: d548b34b062b60
Curious that the initial "HZ / 10" (from Jens' commit 7eaceaccab) wasn't
expressed in terms of msecs either.
Anyway, thanks for the fix!
Mike
> ---
> drivers/md/dm.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/md/dm.c b/drivers/md/dm.c
> index 1b2f96205361..17c63265a205 100644
> --- a/drivers/md/dm.c
> +++ b/drivers/md/dm.c
> @@ -2175,7 +2175,7 @@ static void dm_request_fn(struct request_queue *q)
> md_in_flight(md) && rq->bio && rq->bio->bi_vcnt == 1 &&
> md->last_rq_pos == pos && md->last_rq_rw == rq_data_dir(rq)) ||
> (ti->type->busy && ti->type->busy(ti))) {
> - blk_delay_queue(q, HZ / 100);
> + blk_delay_queue(q, 10);
> return;
> }
>
> --
> 2.8.0.rc3.226.g39d4020
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-07-15 14:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-07-15 13:27 [PATCH] dm: fix parameter to blk_delay_queue() Tahsin Erdogan
2016-07-15 13:59 ` Mike Snitzer
2016-07-15 14:18 ` Tahsin Erdogan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).