diff mbox series

block: doc: add slice_idle_us to bfq documentation

Message ID 20190108215613.15277-1-jpittman@redhat.com (mailing list archive)
State New, archived
Headers show
Series block: doc: add slice_idle_us to bfq documentation | expand

Commit Message

John Pittman Jan. 8, 2019, 9:56 p.m. UTC
Of the tunables available for the bfq I/O scheduler,
the only one missing from the documentation in
'Documentation/block/bfq-iosched.txt' is slice_idle_us.
Add this tunable to the documentation and a short
explanation of its purpose.

Signed-off-by: John Pittman <jpittman@redhat.com>
---
 Documentation/block/bfq-iosched.txt | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Paolo Valente Jan. 9, 2019, 6:08 a.m. UTC | #1
> Il giorno 8 gen 2019, alle ore 22:56, John Pittman <jpittman@redhat.com> ha scritto:
> 
> Of the tunables available for the bfq I/O scheduler,
> the only one missing from the documentation in
> 'Documentation/block/bfq-iosched.txt' is slice_idle_us.
> Add this tunable to the documentation and a short
> explanation of its purpose.
> 
> Signed-off-by: John Pittman <jpittman@redhat.com>

Acked-by: Paolo Valente <paolo.valente@linaro.org>

Thank you for adding this missing piece,
Paolo

> ---
> Documentation/block/bfq-iosched.txt | 7 +++++++
> 1 file changed, 7 insertions(+)
> 
> diff --git a/Documentation/block/bfq-iosched.txt b/Documentation/block/bfq-iosched.txt
> index 8d8d8f06cab2..98a8dd5ee385 100644
> --- a/Documentation/block/bfq-iosched.txt
> +++ b/Documentation/block/bfq-iosched.txt
> @@ -357,6 +357,13 @@ video playing/streaming, a very low drop rate may be more important
> than maximum throughput. In these cases, consider setting the
> strict_guarantees parameter.
> 
> +slice_idle_us
> +-------------
> +
> +Controls the same tuning parameter as slice_idle, but in microseconds.
> +Either tunable can be used to set idling behavior.  Afterwards, the
> +other tunable will reflect the newly set value in sysfs.
> +
> strict_guarantees
> -----------------
> 
> -- 
> 2.17.2
>
Jens Axboe Jan. 9, 2019, 2:39 p.m. UTC | #2
On 1/8/19 2:56 PM, John Pittman wrote:
> Of the tunables available for the bfq I/O scheduler,
> the only one missing from the documentation in
> 'Documentation/block/bfq-iosched.txt' is slice_idle_us.
> Add this tunable to the documentation and a short
> explanation of its purpose.

Applied, but I fixed up your commit message. You should use 72
chars lengths.
John Pittman Jan. 9, 2019, 3:16 p.m. UTC | #3
Thanks; noted.


On Wed, Jan 9, 2019 at 9:39 AM Jens Axboe <axboe@kernel.dk> wrote:
>
> On 1/8/19 2:56 PM, John Pittman wrote:
> > Of the tunables available for the bfq I/O scheduler,
> > the only one missing from the documentation in
> > 'Documentation/block/bfq-iosched.txt' is slice_idle_us.
> > Add this tunable to the documentation and a short
> > explanation of its purpose.
>
> Applied, but I fixed up your commit message. You should use 72
> chars lengths.
>
> --
> Jens Axboe
>
diff mbox series

Patch

diff --git a/Documentation/block/bfq-iosched.txt b/Documentation/block/bfq-iosched.txt
index 8d8d8f06cab2..98a8dd5ee385 100644
--- a/Documentation/block/bfq-iosched.txt
+++ b/Documentation/block/bfq-iosched.txt
@@ -357,6 +357,13 @@  video playing/streaming, a very low drop rate may be more important
 than maximum throughput. In these cases, consider setting the
 strict_guarantees parameter.
 
+slice_idle_us
+-------------
+
+Controls the same tuning parameter as slice_idle, but in microseconds.
+Either tunable can be used to set idling behavior.  Afterwards, the
+other tunable will reflect the newly set value in sysfs.
+
 strict_guarantees
 -----------------