Re: [PATCH v4 0/4] stat: Modify info for bw and add iops stats

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 06/23/2017 04:27 PM, Andreas Herrmann wrote:
> Hi,
> 
> Next try. Changes to previous version:
> 
> [PATCH 1/4] stat: Print one-line iops stat			     
>  - Added client conversion and bump server version
> [PATCH 2/4] stat: Print number of samples in bw and iops stats  
> [PATCH 3/4] stat: Merge show_thread_status_terse_* functions    
> [PATCH 4/4] stat: Add iops stat and sample number information to
>  - Added explanation of v5 terse format to man page and HOWTO
>  - Continue to report TRIM status with terse version 5

Thanks, applied!

> One oddity I've spotted is that man page states
> 
>  "text description (if provided in config - appears on newline)"
> 
> But I think that this applies only to v2 terse format.
> 
> My merge of the terse output functions tried to preserve the code as
> is so no newline for v3 and above.
> 
> If the newline is the desired behaviour I'll send a follow-on patch to
> adapt the code to the man-page (and if not vice versa).

Only for v2, so I think you're fine.

-- 
Jens Axboe

--
To unsubscribe from this list: send the line "unsubscribe fio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel]     [Linux SCSI]     [Linux IDE]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux