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

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



Dne 18.3.2014 10:11, Hannes Reinecke napsal(a):
On 03/18/2014 09:57 AM, Zdenek Kabelac wrote:
Dne 18.3.2014 09:24, Hannes Reinecke napsal(a):
From: Thomas Fehr <fehr suse de>

The size is displayed in sectors, not kilobytes.

Signed-off-by: Thomas Fehr <fehr suse de>
Acked-by: Hannes Reinecke <hare 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.


Ahh - and I've missed it's for section  '-c, --colon'  and I guess it
still applies for those printed values - since the size is there in sectors - though people are most probably using 'pvs'.

Zdenek



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