All of lore.kernel.org
 help / color / mirror / Atom feed
* [patch] block: fix up io_poll documentation
@ 2017-01-03 22:51 Jeff Moyer
  2017-01-03 23:46 ` Jens Axboe
  0 siblings, 1 reply; 3+ messages in thread
From: Jeff Moyer @ 2017-01-03 22:51 UTC (permalink / raw)
  To: axboe, linux-block


/sys/block/<dev>/queue/io_poll is a boolean.  Fix the docs.

Signed-off-by: Jeff Moyer <jmoyer@redhat.com>

diff --git a/Documentation/block/queue-sysfs.txt b/Documentation/block/queue-sysfs.txt
index 5164215..c0a3bb5 100644
--- a/Documentation/block/queue-sysfs.txt
+++ b/Documentation/block/queue-sysfs.txt
@@ -54,9 +54,9 @@ This is the hardware sector size of the device, in bytes.
 
 io_poll (RW)
 ------------
-When read, this file shows the total number of block IO polls and how
-many returned success.  Writing '0' to this file will disable polling
-for this device.  Writing any non-zero value will enable this feature.
+When read, this file shows whether polling is enabled (1) or disabled
+(0).  Writing '0' to this file will disable polling for this device.
+Writing any non-zero value will enable this feature.
 
 io_poll_delay (RW)
 ------------------

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [patch] block: fix up io_poll documentation
  2017-01-03 22:51 [patch] block: fix up io_poll documentation Jeff Moyer
@ 2017-01-03 23:46 ` Jens Axboe
  2017-01-04 14:49   ` Jeff Moyer
  0 siblings, 1 reply; 3+ messages in thread
From: Jens Axboe @ 2017-01-03 23:46 UTC (permalink / raw)
  To: Jeff Moyer, linux-block

On 01/03/2017 03:51 PM, Jeff Moyer wrote:
> 
> /sys/block/<dev>/queue/io_poll is a boolean.  Fix the docs.
> 
> Signed-off-by: Jeff Moyer <jmoyer@redhat.com>
> 
> diff --git a/Documentation/block/queue-sysfs.txt b/Documentation/block/queue-sysfs.txt
> index 5164215..c0a3bb5 100644
> --- a/Documentation/block/queue-sysfs.txt
> +++ b/Documentation/block/queue-sysfs.txt
> @@ -54,9 +54,9 @@ This is the hardware sector size of the device, in bytes.
>  
>  io_poll (RW)
>  ------------
> -When read, this file shows the total number of block IO polls and how
> -many returned success.  Writing '0' to this file will disable polling
> -for this device.  Writing any non-zero value will enable this feature.
> +When read, this file shows whether polling is enabled (1) or disabled
> +(0).  Writing '0' to this file will disable polling for this device.
> +Writing any non-zero value will enable this feature.

It seems to be describing the mq hw queue io_poll file description when
read, for the queue/ one it's different. I'll apply the patch, thanks.

-- 
Jens Axboe

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [patch] block: fix up io_poll documentation
  2017-01-03 23:46 ` Jens Axboe
@ 2017-01-04 14:49   ` Jeff Moyer
  0 siblings, 0 replies; 3+ messages in thread
From: Jeff Moyer @ 2017-01-04 14:49 UTC (permalink / raw)
  To: Jens Axboe; +Cc: linux-block

Jens Axboe <axboe@kernel.dk> writes:

> On 01/03/2017 03:51 PM, Jeff Moyer wrote:
>> 
>> /sys/block/<dev>/queue/io_poll is a boolean.  Fix the docs.
>> 
>> Signed-off-by: Jeff Moyer <jmoyer@redhat.com>
>> 
>> diff --git a/Documentation/block/queue-sysfs.txt b/Documentation/block/queue-sysfs.txt
>> index 5164215..c0a3bb5 100644
>> --- a/Documentation/block/queue-sysfs.txt
>> +++ b/Documentation/block/queue-sysfs.txt
>> @@ -54,9 +54,9 @@ This is the hardware sector size of the device, in bytes.
>>  
>>  io_poll (RW)
>>  ------------
>> -When read, this file shows the total number of block IO polls and how
>> -many returned success.  Writing '0' to this file will disable polling
>> -for this device.  Writing any non-zero value will enable this feature.
>> +When read, this file shows whether polling is enabled (1) or disabled
>> +(0).  Writing '0' to this file will disable polling for this device.
>> +Writing any non-zero value will enable this feature.
>
> It seems to be describing the mq hw queue io_poll file description when
> read, for the queue/ one it's different. I'll apply the patch, thanks.

Right.  It's probably time to create an mq-sysfs.txt file.

-Jeff

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-01-04 14:49 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-03 22:51 [patch] block: fix up io_poll documentation Jeff Moyer
2017-01-03 23:46 ` Jens Axboe
2017-01-04 14:49   ` Jeff Moyer

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.