Subject: [PATCH 3/4] Grow_reshape: need sub-array component size indata migration.

From a187f2281d4053b02b1e0e3e99cdd8f406db38e3 Mon Sep 17 00:00:00 2001
From: Marcin Labun
Date: Wed, 5 Jan 2011 17:38:00 +0100
Subject: [PATCH 3/4] Grow_reshape: need sub-array component size in data migration.

Signed-off-by: Marcin Labun
---
Grow.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/Grow.c b/Grow.c
index 1998750..9db4594 100644
--- a/Grow.c
+++ b/Grow.c
@@ -1406,7 +1406,7 @@ int Grow_reshape(char *devname, int fd, int quiet, char *backup_file,
array.spare_disks);
return 1;
}
- sra = sysfs_read(fd, 0, GET_LEVEL | GET_DISKS | GET_DEVS | GET_STATE | GET_VERSION);
+ sra = sysfs_read(fd, 0, GET_LEVEL | GET_DISKS | GET_DEVS | GET_STATE | GET_VERSION | GET_COMPONENT);
if (sra) {
if (st->ss->external && subarray == NULL) {
array.level = LEVEL_CONTAINER;
--
1.6.4.2

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

Re: Subject: [PATCH 3/4] Grow_reshape: need sub-array componentsize in data migration.

On Tue, 11 Jan 2011 15:24:49 +0000 "Labun, Marcin"
wrote:

> >From a187f2281d4053b02b1e0e3e99cdd8f406db38e3 Mon Sep 17 00:00:00 2001
> From: Marcin Labun
> Date: Wed, 5 Jan 2011 17:38:00 +0100
> Subject: [PATCH 3/4] Grow_reshape: need sub-array component size in data migration.
>
> Signed-off-by: Marcin Labun
> ---
> Grow.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/Grow.c b/Grow.c
> index 1998750..9db4594 100644
> --- a/Grow.c
> +++ b/Grow.c
> @@ -1406,7 +1406,7 @@ int Grow_reshape(char *devname, int fd, int quiet, char *backup_file,
> array.spare_disks);
> return 1;
> }
> - sra = sysfs_read(fd, 0, GET_LEVEL | GET_DISKS | GET_DEVS | GET_STATE | GET_VERSION);
> + sra = sysfs_read(fd, 0, GET_LEVEL | GET_DISKS | GET_DEVS | GET_STATE | GET_VERSION | GET_COMPONENT);
> if (sra) {
> if (st->ss->external && subarray == NULL) {
> array.level = LEVEL_CONTAINER;


I cannot see where sra->component_size is used.
Please be more explicit. (Please always be more explicit).

NeilBrown

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