[lvm-devel] [PATCH 1/2] Fix pvdisplay to refer to sectors, not KB

[Hannes Reinecke]

On 03/18/2014 09:57 AM, Zdenek Kabelac wrote:
> Dne 18.3.2014 09:24, Hannes Reinecke napsal(a):
>> From: Thomas Fehr <fehr at suse.de>
>>
>> The size is displayed in sectors, not kilobytes.
>>
>> Signed-off-by: Thomas Fehr <fehr at suse.de>
>> Acked-by: Hannes Reinecke <hare at suse.de>
>> ---
>>   man/pvdisplay.8.in | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/man/pvdisplay.8.in b/man/pvdisplay.8.in
>> index 03c12c4..13c26b5 100644
>> --- a/man/pvdisplay.8.in
>> +++ b/man/pvdisplay.8.in
>> @@ -65,7 +65,7 @@ The values are:
>>
>>   * physical volume device name
>>   * volume group name
>> -* physical volume size in kilobytes
>> +* physical volume size in sectors
>>   * internal physical volume number (obsolete)
>>   * physical volume status
>>   * physical volume (not) allocatable
> 
> Hi
> 
> Against which version of lvm2 is this patch ?
> 
> Since my upstream pvdisplay shows  size in  'human readable' format
> i.e.:
> (So definitely not in sector - unless used with option  '--units s')
> 
> --- Physical volume ---
> PV Name               /dev/loop0
> VG Name               vg
> PV Size               976,56 GiB / not usable 0
> Allocatable           yes
> PE Size               32,00 KiB
> Total PE              31999968
> Free PE               31999872
> Allocated PE          96
> PV UUID               5DOVI6-ioCm-LJvd-JbwQ-qJc3-FWO4-yQCjGY
> 
Did I mention that these patches are ancient?
(Originally developed in 2007 ...)

I'm just doing house-keeping for our lvm2 package at the moment.
So I'm not at all surprised this patch is obsolete.

Thanks for the info; please disregard this patch.

Cheers,

Hannes
-- 
Dr. Hannes Reinecke		      zSeries & Storage
hare at suse.de			      +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)