* [PATCH] as-iosched: fix incorrect comments
@ 2007-12-05 10:06 Aaron Carroll
2007-12-05 10:11 ` Nick Piggin
0 siblings, 1 reply; 3+ messages in thread
From: Aaron Carroll @ 2007-12-05 10:06 UTC (permalink / raw)
To: Jens Axboe; +Cc: linux-kernel, npiggin, Peter Chubb
Two comments refer to deadlines applying to reads only. This is
not the case.
Signed-off-by: Aaron Carroll <aaronc@gelato.unsw.edu.au>
---
block/as-iosched.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/block/as-iosched.c b/block/as-iosched.c
index dc715a5..4513fc5 100644
--- a/block/as-iosched.c
+++ b/block/as-iosched.c
@@ -880,7 +880,7 @@ static void as_remove_queued_request(struct request_queue *q,
}
/*
- * as_fifo_expired returns 0 if there are no expired reads on the fifo,
+ * as_fifo_expired returns 0 if there are no expired requests on the fifo,
* 1 otherwise. It is ratelimited so that we only perform the check once per
* `fifo_expire' interval. Otherwise a large number of expired requests
* would create a hopeless seekstorm.
@@ -1159,7 +1159,7 @@ static void as_add_request(struct request_queue *q, struct request *rq)
as_add_rq_rb(ad, rq);
/*
- * set expire time (only used for reads) and add to fifo list
+ * set expire time and add to fifo list
*/
rq_set_fifo_time(rq, jiffies + ad->fifo_expire[data_dir]);
list_add_tail(&rq->queuelist, &ad->fifo_list[data_dir]);
--
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] as-iosched: fix incorrect comments
2007-12-05 10:06 [PATCH] as-iosched: fix incorrect comments Aaron Carroll
@ 2007-12-05 10:11 ` Nick Piggin
2007-12-05 10:16 ` Jens Axboe
0 siblings, 1 reply; 3+ messages in thread
From: Nick Piggin @ 2007-12-05 10:11 UTC (permalink / raw)
To: Aaron Carroll; +Cc: Jens Axboe, linux-kernel, Peter Chubb
On Wed, Dec 05, 2007 at 09:06:50PM +1100, Aaron Carroll wrote:
> Two comments refer to deadlines applying to reads only. This is
> not the case.
>
> Signed-off-by: Aaron Carroll <aaronc@gelato.unsw.edu.au>
Acked-by: Nick Piggin <npiggin@suse.de>
> ---
> block/as-iosched.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/block/as-iosched.c b/block/as-iosched.c
> index dc715a5..4513fc5 100644
> --- a/block/as-iosched.c
> +++ b/block/as-iosched.c
> @@ -880,7 +880,7 @@ static void as_remove_queued_request(struct request_queue *q,
> }
>
> /*
> - * as_fifo_expired returns 0 if there are no expired reads on the fifo,
> + * as_fifo_expired returns 0 if there are no expired requests on the fifo,
> * 1 otherwise. It is ratelimited so that we only perform the check once per
> * `fifo_expire' interval. Otherwise a large number of expired requests
> * would create a hopeless seekstorm.
> @@ -1159,7 +1159,7 @@ static void as_add_request(struct request_queue *q, struct request *rq)
> as_add_rq_rb(ad, rq);
>
> /*
> - * set expire time (only used for reads) and add to fifo list
> + * set expire time and add to fifo list
> */
> rq_set_fifo_time(rq, jiffies + ad->fifo_expire[data_dir]);
> list_add_tail(&rq->queuelist, &ad->fifo_list[data_dir]);
> --
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] as-iosched: fix incorrect comments
2007-12-05 10:11 ` Nick Piggin
@ 2007-12-05 10:16 ` Jens Axboe
0 siblings, 0 replies; 3+ messages in thread
From: Jens Axboe @ 2007-12-05 10:16 UTC (permalink / raw)
To: Nick Piggin; +Cc: Aaron Carroll, linux-kernel, Peter Chubb
On Wed, Dec 05 2007, Nick Piggin wrote:
> On Wed, Dec 05, 2007 at 09:06:50PM +1100, Aaron Carroll wrote:
> > Two comments refer to deadlines applying to reads only. This is
> > not the case.
> >
> > Signed-off-by: Aaron Carroll <aaronc@gelato.unsw.edu.au>
>
> Acked-by: Nick Piggin <npiggin@suse.de>
Goodie, I'll apply this one as well.
--
Jens Axboe
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2007-12-05 10:24 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-12-05 10:06 [PATCH] as-iosched: fix incorrect comments Aaron Carroll
2007-12-05 10:11 ` Nick Piggin
2007-12-05 10:16 ` Jens Axboe
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).