On 19:33, Andre Noll wrote:
> This is twice as much as it should be due to a bug in mdadm which bites
> only for version1 superblocks. The patch below should fix it. However,
> this might not be the most elegant solution because the real bug
> is IMHO that get_component_size() multiplies the value from sysfs
> (which is always in 1K units) by two, so it returns 2K units which
> looks a bit weird.
This was of course a braino: Multiplying a value in 1K units by two
yields 512byte units rather than 2K units, which is exactly what the
comment to get_component_size() says ;)
That being said, I think my patch is correct and dividing the result
by two in Detail() seems the best way to deal with the situation.
So here's the patch again, this time with proper log message.
Andre
commit 2e9fd78bd09bf332ac86f2d288a0e7c3c1c3df4f
Author: Andre Noll <maan@xxxxxxxxxxxxxxx>
Date: Tue Jan 27 08:49:15 2009 +0100
Fix the used device size in mdadm -D output.
As get_component_size() returns the number of used sectors of a
device while mdadm is supposed to print this size in kilobytes,
we have to divide the return value of get_component_size() by two.
Signed-off-by: Andre Noll <maan@xxxxxxxxxxxxxxx>
diff --git a/Detail.c b/Detail.c
index 3cee66f..cef1abb 100644
--- a/Detail.c
+++ b/Detail.c
@@ -195,7 +195,7 @@ int Detail(char *dev, int brief, int export, int test, char *homehost)
if (array.level >= 1) {
if (array.major_version != 0 &&
(larray_size >= 0xFFFFFFFFULL|| array.size == 0)) {
- unsigned long long dsize = get_component_size(fd);
+ unsigned long long dsize = get_component_size(fd) / 2; /* KB */
if (dsize > 0)
printf(" Used Dev Size : %llu%s\n",
dsize,
--
The only person who always got his work done by Friday was Robinson Crusoe
Attachment:
signature.asc
Description: Digital signature
