[PATCH 1/2] blkparse: remove duplicated entry for flag M

[PATCH 1/2] blkparse: remove duplicated entry for flag M

[Weiping Zhang]

remove dupliated entry 'M' for man page of blkparse.

Signed-off-by: Weiping Zhang <zhangweiping@didichuxing.com>
---
 doc/blkparse.1 | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/doc/blkparse.1 b/doc/blkparse.1
index be9b34b..b1cb6ef 100644
--- a/doc/blkparse.1
+++ b/doc/blkparse.1
@@ -243,10 +243,6 @@ Same as the back merge, except this i/o ends where a previously inserted
 requests starts.
 
 .HP 4
-\fBM --front or back merge\fR
-One of the above
-
-.HP 4
 \fBM -- front or back merge\fR
 One of the above.
 
-- 
2.9.4

Re: [PATCH 1/2] blkparse: remove duplicated entry for flag M

[Steffen Maier]

On 04/07/2018 11:12 AM, Weiping Zhang wrote:
> remove dupliated entry 'M' for man page of blkparse.
> 
> Signed-off-by: Weiping Zhang <zhangweiping@didichuxing.com>
> ---
>   doc/blkparse.1 | 4 ----
>   1 file changed, 4 deletions(-)
> 
> diff --git a/doc/blkparse.1 b/doc/blkparse.1
> index be9b34b..b1cb6ef 100644
> --- a/doc/blkparse.1
> +++ b/doc/blkparse.1
> @@ -243,10 +243,6 @@ Same as the back merge, except this i/o ends where a previously inserted
>   requests starts.
> 
>   .HP 4
> -\fBM --front or back merge\fR
> -One of the above
> -
> -.HP 4
>   \fBM -- front or back merge\fR
>   One of the above.
> 

I also found this (and other things) and plan to submit them later.

Additionally, I had the impression that 'M' is nowadays always a "back 
merge", not "front or back merge".

Anyway, dropping the duplicate seems right:

Reviewed-by: Steffen Maier <maier@linux.ibm.com>

-- 
Mit freundlichen Grüßen / Kind regards
Steffen Maier

Linux on z Systems Development

IBM Deutschland Research & Development GmbH
Vorsitzende des Aufsichtsrats: Martina Koederitz
Geschaeftsfuehrung: Dirk Wittkopp
Sitz der Gesellschaft: Boeblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294

Re: [PATCH 1/2] blkparse: remove duplicated entry for flag M

[Jens Axboe]

On 4/9/18 8:33 AM, Steffen Maier wrote:
> 
> On 04/07/2018 11:12 AM, Weiping Zhang wrote:
>> remove dupliated entry 'M' for man page of blkparse.
>>
>> Signed-off-by: Weiping Zhang <zhangweiping@didichuxing.com>
>> ---
>>   doc/blkparse.1 | 4 ----
>>   1 file changed, 4 deletions(-)
>>
>> diff --git a/doc/blkparse.1 b/doc/blkparse.1
>> index be9b34b..b1cb6ef 100644
>> --- a/doc/blkparse.1
>> +++ b/doc/blkparse.1
>> @@ -243,10 +243,6 @@ Same as the back merge, except this i/o ends where a previously inserted
>>   requests starts.
>>
>>   .HP 4
>> -\fBM --front or back merge\fR
>> -One of the above
>> -
>> -.HP 4
>>   \fBM -- front or back merge\fR
>>   One of the above.
>>
> 
> I also found this (and other things) and plan to submit them later.
> 
> Additionally, I had the impression that 'M' is nowadays always a "back 
> merge", not "front or back merge".

'M' is a back merge, 'F' is a front merge.

-- 
Jens Axboe