[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <x49y6c5tkj4.fsf@segfault.boston.devel.redhat.com>
Date: Tue, 17 Aug 2010 09:51:11 -0400
From: Jeff Moyer <jmoyer@...hat.com>
To: Vivek Goyal <vgoyal@...hat.com>
Cc: linux-kernel@...r.kernel.org, jaxboe@...ionio.com
Subject: Re: [PATCH 5/5] cfq-iosched: Documentation help for new tunables
Vivek Goyal <vgoyal@...hat.com> writes:
>> I'm not sure we need documentation of this tunable twice. Why not just
>> give guidance on when it should be set to 0 in the next section
>> (group_idle) and refer to cfq-iosched.txt?
>
> I have put one line saying for more details look at cfq-iosched.txt. I
> have still retained the slice_idle entry because at the end of the day
> this is the tunable I expect people to modify and not group_idle. Also
> notice that these are two different files (cfq-iosched.txt and
> blkio-controller.txt).
Yeah, I know it's in two places, and that means updating two places with
changes. However, I doubt we'll have to update this often, so I'm not
going to argue.
Acked-by: Jeff Moyer <jmoyer@...hat.com>
Cheers,
Jeff
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists