* [PATCH v4] mdadm: replace container level checking with inline
@ 2022-09-02 6:49 Kinga Tanska
2022-09-03 7:11 ` Coly Li
2022-09-14 14:49 ` Coly Li
0 siblings, 2 replies; 4+ messages in thread
From: Kinga Tanska @ 2022-09-02 6:49 UTC (permalink / raw)
To: linux-raid; +Cc: jes, colyli
To unify all containers checks in code, is_container() function is
added and propagated.
Signed-off-by: Kinga Tanska <kinga.tanska@intel.com>
---
Assemble.c | 7 +++----
Create.c | 6 +++---
Grow.c | 6 +++---
Incremental.c | 4 ++--
mdadm.h | 14 ++++++++++++++
super-ddf.c | 6 +++---
super-intel.c | 4 ++--
super0.c | 2 +-
super1.c | 2 +-
sysfs.c | 2 +-
10 files changed, 33 insertions(+), 20 deletions(-)
diff --git a/Assemble.c b/Assemble.c
index 1dd82a8c..8b0af0c9 100644
--- a/Assemble.c
+++ b/Assemble.c
@@ -1120,7 +1120,7 @@ static int start_array(int mdfd,
i/2, mddev);
}
- if (content->array.level == LEVEL_CONTAINER) {
+ if (is_container(content->array.level)) {
sysfs_rules_apply(mddev, content);
if (c->verbose >= 0) {
pr_err("Container %s has been assembled with %d drive%s",
@@ -1549,8 +1549,7 @@ try_again:
*/
trustworthy = LOCAL;
- if (name[0] == 0 &&
- content->array.level == LEVEL_CONTAINER) {
+ if (!name[0] && is_container(content->array.level)) {
name = content->text_version;
trustworthy = METADATA;
}
@@ -1809,7 +1808,7 @@ try_again:
}
#endif
}
- if (c->force && !clean && content->array.level != LEVEL_CONTAINER &&
+ if (c->force && !clean && !is_container(content->array.level) &&
!enough(content->array.level, content->array.raid_disks,
content->array.layout, clean, avail)) {
change += st->ss->update_super(st, content, "force-array",
diff --git a/Create.c b/Create.c
index e06ec2ae..953e7372 100644
--- a/Create.c
+++ b/Create.c
@@ -487,7 +487,7 @@ int Create(struct supertype *st, char *mddev,
st->minor_version >= 1)
/* metadata at front */
warn |= check_partitions(fd, dname, 0, 0);
- else if (s->level == 1 || s->level == LEVEL_CONTAINER ||
+ else if (s->level == 1 || is_container(s->level) ||
(s->level == 0 && s->raiddisks == 1))
/* partitions could be meaningful */
warn |= check_partitions(fd, dname, freesize*2, s->size*2);
@@ -997,7 +997,7 @@ int Create(struct supertype *st, char *mddev,
* again returns container info.
*/
st->ss->getinfo_super(st, &info_new, NULL);
- if (st->ss->external && s->level != LEVEL_CONTAINER &&
+ if (st->ss->external && !is_container(s->level) &&
!same_uuid(info_new.uuid, info.uuid, 0)) {
map_update(&map, fd2devnm(mdfd),
info_new.text_version,
@@ -1040,7 +1040,7 @@ int Create(struct supertype *st, char *mddev,
map_unlock(&map);
free(infos);
- if (s->level == LEVEL_CONTAINER) {
+ if (is_container(s->level)) {
/* No need to start. But we should signal udev to
* create links */
sysfs_uevent(&info, "change");
diff --git a/Grow.c b/Grow.c
index 0f07a894..e362403a 100644
--- a/Grow.c
+++ b/Grow.c
@@ -2175,7 +2175,7 @@ size_change_error:
devname, s->size);
}
changed = 1;
- } else if (array.level != LEVEL_CONTAINER) {
+ } else if (!is_container(array.level)) {
s->size = get_component_size(fd)/2;
if (s->size == 0)
s->size = array.size;
@@ -2231,7 +2231,7 @@ size_change_error:
info.component_size = s->size*2;
info.new_level = s->level;
info.new_chunk = s->chunk * 1024;
- if (info.array.level == LEVEL_CONTAINER) {
+ if (is_container(info.array.level)) {
info.delta_disks = UnSet;
info.array.raid_disks = s->raiddisks;
} else if (s->raiddisks)
@@ -2344,7 +2344,7 @@ size_change_error:
printf("layout for %s set to %d\n",
devname, array.layout);
}
- } else if (array.level == LEVEL_CONTAINER) {
+ } else if (is_container(array.level)) {
/* This change is to be applied to every array in the
* container. This is only needed when the metadata imposes
* restraints of the various arrays in the container.
diff --git a/Incremental.c b/Incremental.c
index 4d0cd9d6..5a5f4c4c 100644
--- a/Incremental.c
+++ b/Incremental.c
@@ -244,7 +244,7 @@ int Incremental(struct mddev_dev *devlist, struct context *c,
c->autof = ci->autof;
name_to_use = info.name;
- if (name_to_use[0] == 0 && info.array.level == LEVEL_CONTAINER) {
+ if (name_to_use[0] == 0 && is_container(info.array.level)) {
name_to_use = info.text_version;
trustworthy = METADATA;
}
@@ -472,7 +472,7 @@ int Incremental(struct mddev_dev *devlist, struct context *c,
/* 7/ Is there enough devices to possibly start the array? */
/* 7a/ if not, finish with success. */
- if (info.array.level == LEVEL_CONTAINER) {
+ if (is_container(info.array.level)) {
char devnm[32];
/* Try to assemble within the container */
sysfs_uevent(sra, "change");
diff --git a/mdadm.h b/mdadm.h
index 941a5f38..3673494e 100644
--- a/mdadm.h
+++ b/mdadm.h
@@ -1924,3 +1924,17 @@ enum r0layout {
* This is true for native and DDF, IMSM allows 16.
*/
#define MD_NAME_MAX 32
+
+/**
+ * is_container() - check if @level is &LEVEL_CONTAINER
+ * @level: level value
+ *
+ * return:
+ * 1 if level is equal to &LEVEL_CONTAINER, 0 otherwise.
+ */
+static inline int is_container(const int level)
+{
+ if (level == LEVEL_CONTAINER)
+ return 1;
+ return 0;
+}
diff --git a/super-ddf.c b/super-ddf.c
index 949e7d15..9d1e3b94 100644
--- a/super-ddf.c
+++ b/super-ddf.c
@@ -3325,7 +3325,7 @@ validate_geometry_ddf_container(struct supertype *st,
int fd;
unsigned long long ldsize;
- if (level != LEVEL_CONTAINER)
+ if (!is_container(level))
return 0;
if (!dev)
return 1;
@@ -3371,7 +3371,7 @@ static int validate_geometry_ddf(struct supertype *st,
if (level == LEVEL_NONE)
level = LEVEL_CONTAINER;
- if (level == LEVEL_CONTAINER) {
+ if (is_container(level)) {
/* Must be a fresh device to add to a container */
return validate_geometry_ddf_container(st, level, raiddisks,
data_offset, dev,
@@ -3488,7 +3488,7 @@ static int validate_geometry_ddf_bvd(struct supertype *st,
struct dl *dl;
unsigned long long maxsize;
/* ddf/bvd supports lots of things, but not containers */
- if (level == LEVEL_CONTAINER) {
+ if (is_container(level)) {
if (verbose)
pr_err("DDF cannot create a container within an container\n");
return 0;
diff --git a/super-intel.c b/super-intel.c
index 4d82af3d..b0565610 100644
--- a/super-intel.c
+++ b/super-intel.c
@@ -6727,7 +6727,7 @@ static int validate_geometry_imsm_container(struct supertype *st, int level,
struct intel_super *super = NULL;
int rv = 0;
- if (level != LEVEL_CONTAINER)
+ if (!is_container(level))
return 0;
if (!dev)
return 1;
@@ -7692,7 +7692,7 @@ static int validate_geometry_imsm(struct supertype *st, int level, int layout,
* if given unused devices create a container
* if given given devices in a container create a member volume
*/
- if (level == LEVEL_CONTAINER)
+ if (is_container(level))
/* Must be a fresh device to add to a container */
return validate_geometry_imsm_container(st, level, raiddisks,
data_offset, dev,
diff --git a/super0.c b/super0.c
index 37f595ed..93876e2e 100644
--- a/super0.c
+++ b/super0.c
@@ -1273,7 +1273,7 @@ static int validate_geometry0(struct supertype *st, int level,
if (get_linux_version() < 3001000)
tbmax = 2;
- if (level == LEVEL_CONTAINER) {
+ if (is_container(level)) {
if (verbose)
pr_err("0.90 metadata does not support containers\n");
return 0;
diff --git a/super1.c b/super1.c
index 58345e68..0b505a7e 100644
--- a/super1.c
+++ b/super1.c
@@ -2830,7 +2830,7 @@ static int validate_geometry1(struct supertype *st, int level,
unsigned long long overhead;
int fd;
- if (level == LEVEL_CONTAINER) {
+ if (is_container(level)) {
if (verbose)
pr_err("1.x metadata does not support containers\n");
return 0;
diff --git a/sysfs.c b/sysfs.c
index 0d98a65f..ca1d888f 100644
--- a/sysfs.c
+++ b/sysfs.c
@@ -763,7 +763,7 @@ int sysfs_add_disk(struct mdinfo *sra, struct mdinfo *sd, int resume)
rv = sysfs_set_num(sra, sd, "offset", sd->data_offset);
rv |= sysfs_set_num(sra, sd, "size", (sd->component_size+1) / 2);
- if (sra->array.level != LEVEL_CONTAINER) {
+ if (!is_container(sra->array.level)) {
if (sra->consistency_policy == CONSISTENCY_POLICY_PPL) {
rv |= sysfs_set_num(sra, sd, "ppl_sector", sd->ppl_sector);
rv |= sysfs_set_num(sra, sd, "ppl_size", sd->ppl_size);
--
2.26.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v4] mdadm: replace container level checking with inline
2022-09-02 6:49 [PATCH v4] mdadm: replace container level checking with inline Kinga Tanska
@ 2022-09-03 7:11 ` Coly Li
2022-09-29 15:05 ` Jes Sorensen
2022-09-14 14:49 ` Coly Li
1 sibling, 1 reply; 4+ messages in thread
From: Coly Li @ 2022-09-03 7:11 UTC (permalink / raw)
To: Kinga Tanska; +Cc: linux-raid, jes
> 2022年9月2日 14:49，Kinga Tanska <kinga.tanska@intel.com> 写道：
>
> To unify all containers checks in code, is_container() function is
> added and propagated.
>
> Signed-off-by: Kinga Tanska <kinga.tanska@intel.com>
Acked-by: Coly Li <colyli@suse.de>
But this patch has a minor conflict with Mateusz’s “Manage: Block unsafe member failing” series, it is simply because your patch landed a bit late than Mateusz’s.
I already rebased your patch in the mdadm-CI queue. After I finish to review all the pending patches, let’s see whether you should post a v6 version or I can post the rebased one for you.
Thanks.
Coly Li
> ---
> Assemble.c | 7 +++----
> Create.c | 6 +++---
> Grow.c | 6 +++---
> Incremental.c | 4 ++--
> mdadm.h | 14 ++++++++++++++
> super-ddf.c | 6 +++---
> super-intel.c | 4 ++--
> super0.c | 2 +-
> super1.c | 2 +-
> sysfs.c | 2 +-
> 10 files changed, 33 insertions(+), 20 deletions(-)
>
> diff --git a/Assemble.c b/Assemble.c
> index 1dd82a8c..8b0af0c9 100644
> --- a/Assemble.c
> +++ b/Assemble.c
>
[snipped]
> @@ -1809,7 +1808,7 @@ try_again:
> }
> #endif
> }
> - if (c->force && !clean && content->array.level != LEVEL_CONTAINER &&
> + if (c->force && !clean && !is_container(content->array.level) &&
> !enough(content->array.level, content->array.raid_disks,
> content->array.layout, clean, avail)) {
> change += st->ss->update_super(st, content, "force-array”,
The conflict is here, and the rebased change looks like this,
@@ -1807,7 +1806,7 @@ try_again:
}
#endif
}
- if (c->force && !clean && content->array.level != LEVEL_CONTAINER &&
+ if (c->force && !clean && !is_container(content->array.level) &&
!enough(content->array.level, content->array.raid_disks,
content->array.layout, clean, avail)) {
change += st->ss->update_super(st, content, UOPT_SPEC_FORCE_ARRAY,
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v4] mdadm: replace container level checking with inline
2022-09-02 6:49 [PATCH v4] mdadm: replace container level checking with inline Kinga Tanska
2022-09-03 7:11 ` Coly Li
@ 2022-09-14 14:49 ` Coly Li
1 sibling, 0 replies; 4+ messages in thread
From: Coly Li @ 2022-09-14 14:49 UTC (permalink / raw)
To: Kinga Tanska; +Cc: linux-raid, jes
> 2022年9月2日 14:49，Kinga Tanska <kinga.tanska@intel.com> 写道：
>
> To unify all containers checks in code, is_container() function is
> added and propagated.
>
> Signed-off-by: Kinga Tanska <kinga.tanska@intel.com>
Acked-by: Coly Li <colyli@suse.de>
BTW, this patch doesn’t apply due to conflict with other under-review patches. I rebased the conflict and push it to branch refs/heads/20220903-testing from the mdadm-CI repo. Could you please to check and confirm this version? If it is OK, you may use this version to submit to Jes with my Acked-by.
Thanks.
Coly Li
> ---
> Assemble.c | 7 +++----
> Create.c | 6 +++---
> Grow.c | 6 +++---
> Incremental.c | 4 ++--
> mdadm.h | 14 ++++++++++++++
> super-ddf.c | 6 +++---
> super-intel.c | 4 ++--
> super0.c | 2 +-
> super1.c | 2 +-
> sysfs.c | 2 +-
> 10 files changed, 33 insertions(+), 20 deletions(-)
>
> diff --git a/Assemble.c b/Assemble.c
> index 1dd82a8c..8b0af0c9 100644
> --- a/Assemble.c
> +++ b/Assemble.c
> @@ -1120,7 +1120,7 @@ static int start_array(int mdfd,
> i/2, mddev);
> }
>
> - if (content->array.level == LEVEL_CONTAINER) {
> + if (is_container(content->array.level)) {
> sysfs_rules_apply(mddev, content);
> if (c->verbose >= 0) {
> pr_err("Container %s has been assembled with %d drive%s",
> @@ -1549,8 +1549,7 @@ try_again:
> */
> trustworthy = LOCAL;
>
> - if (name[0] == 0 &&
> - content->array.level == LEVEL_CONTAINER) {
> + if (!name[0] && is_container(content->array.level)) {
> name = content->text_version;
> trustworthy = METADATA;
> }
> @@ -1809,7 +1808,7 @@ try_again:
> }
> #endif
> }
> - if (c->force && !clean && content->array.level != LEVEL_CONTAINER &&
> + if (c->force && !clean && !is_container(content->array.level) &&
> !enough(content->array.level, content->array.raid_disks,
> content->array.layout, clean, avail)) {
> change += st->ss->update_super(st, content, "force-array",
> diff --git a/Create.c b/Create.c
> index e06ec2ae..953e7372 100644
> --- a/Create.c
> +++ b/Create.c
> @@ -487,7 +487,7 @@ int Create(struct supertype *st, char *mddev,
> st->minor_version >= 1)
> /* metadata at front */
> warn |= check_partitions(fd, dname, 0, 0);
> - else if (s->level == 1 || s->level == LEVEL_CONTAINER ||
> + else if (s->level == 1 || is_container(s->level) ||
> (s->level == 0 && s->raiddisks == 1))
> /* partitions could be meaningful */
> warn |= check_partitions(fd, dname, freesize*2, s->size*2);
> @@ -997,7 +997,7 @@ int Create(struct supertype *st, char *mddev,
> * again returns container info.
> */
> st->ss->getinfo_super(st, &info_new, NULL);
> - if (st->ss->external && s->level != LEVEL_CONTAINER &&
> + if (st->ss->external && !is_container(s->level) &&
> !same_uuid(info_new.uuid, info.uuid, 0)) {
> map_update(&map, fd2devnm(mdfd),
> info_new.text_version,
> @@ -1040,7 +1040,7 @@ int Create(struct supertype *st, char *mddev,
> map_unlock(&map);
> free(infos);
>
> - if (s->level == LEVEL_CONTAINER) {
> + if (is_container(s->level)) {
> /* No need to start. But we should signal udev to
> * create links */
> sysfs_uevent(&info, "change");
> diff --git a/Grow.c b/Grow.c
> index 0f07a894..e362403a 100644
> --- a/Grow.c
> +++ b/Grow.c
> @@ -2175,7 +2175,7 @@ size_change_error:
> devname, s->size);
> }
> changed = 1;
> - } else if (array.level != LEVEL_CONTAINER) {
> + } else if (!is_container(array.level)) {
> s->size = get_component_size(fd)/2;
> if (s->size == 0)
> s->size = array.size;
> @@ -2231,7 +2231,7 @@ size_change_error:
> info.component_size = s->size*2;
> info.new_level = s->level;
> info.new_chunk = s->chunk * 1024;
> - if (info.array.level == LEVEL_CONTAINER) {
> + if (is_container(info.array.level)) {
> info.delta_disks = UnSet;
> info.array.raid_disks = s->raiddisks;
> } else if (s->raiddisks)
> @@ -2344,7 +2344,7 @@ size_change_error:
> printf("layout for %s set to %d\n",
> devname, array.layout);
> }
> - } else if (array.level == LEVEL_CONTAINER) {
> + } else if (is_container(array.level)) {
> /* This change is to be applied to every array in the
> * container. This is only needed when the metadata imposes
> * restraints of the various arrays in the container.
> diff --git a/Incremental.c b/Incremental.c
> index 4d0cd9d6..5a5f4c4c 100644
> --- a/Incremental.c
> +++ b/Incremental.c
> @@ -244,7 +244,7 @@ int Incremental(struct mddev_dev *devlist, struct context *c,
> c->autof = ci->autof;
>
> name_to_use = info.name;
> - if (name_to_use[0] == 0 && info.array.level == LEVEL_CONTAINER) {
> + if (name_to_use[0] == 0 && is_container(info.array.level)) {
> name_to_use = info.text_version;
> trustworthy = METADATA;
> }
> @@ -472,7 +472,7 @@ int Incremental(struct mddev_dev *devlist, struct context *c,
>
> /* 7/ Is there enough devices to possibly start the array? */
> /* 7a/ if not, finish with success. */
> - if (info.array.level == LEVEL_CONTAINER) {
> + if (is_container(info.array.level)) {
> char devnm[32];
> /* Try to assemble within the container */
> sysfs_uevent(sra, "change");
> diff --git a/mdadm.h b/mdadm.h
> index 941a5f38..3673494e 100644
> --- a/mdadm.h
> +++ b/mdadm.h
> @@ -1924,3 +1924,17 @@ enum r0layout {
> * This is true for native and DDF, IMSM allows 16.
> */
> #define MD_NAME_MAX 32
> +
> +/**
> + * is_container() - check if @level is &LEVEL_CONTAINER
> + * @level: level value
> + *
> + * return:
> + * 1 if level is equal to &LEVEL_CONTAINER, 0 otherwise.
> + */
> +static inline int is_container(const int level)
> +{
> + if (level == LEVEL_CONTAINER)
> + return 1;
> + return 0;
> +}
> diff --git a/super-ddf.c b/super-ddf.c
> index 949e7d15..9d1e3b94 100644
> --- a/super-ddf.c
> +++ b/super-ddf.c
> @@ -3325,7 +3325,7 @@ validate_geometry_ddf_container(struct supertype *st,
> int fd;
> unsigned long long ldsize;
>
> - if (level != LEVEL_CONTAINER)
> + if (!is_container(level))
> return 0;
> if (!dev)
> return 1;
> @@ -3371,7 +3371,7 @@ static int validate_geometry_ddf(struct supertype *st,
>
> if (level == LEVEL_NONE)
> level = LEVEL_CONTAINER;
> - if (level == LEVEL_CONTAINER) {
> + if (is_container(level)) {
> /* Must be a fresh device to add to a container */
> return validate_geometry_ddf_container(st, level, raiddisks,
> data_offset, dev,
> @@ -3488,7 +3488,7 @@ static int validate_geometry_ddf_bvd(struct supertype *st,
> struct dl *dl;
> unsigned long long maxsize;
> /* ddf/bvd supports lots of things, but not containers */
> - if (level == LEVEL_CONTAINER) {
> + if (is_container(level)) {
> if (verbose)
> pr_err("DDF cannot create a container within an container\n");
> return 0;
> diff --git a/super-intel.c b/super-intel.c
> index 4d82af3d..b0565610 100644
> --- a/super-intel.c
> +++ b/super-intel.c
> @@ -6727,7 +6727,7 @@ static int validate_geometry_imsm_container(struct supertype *st, int level,
> struct intel_super *super = NULL;
> int rv = 0;
>
> - if (level != LEVEL_CONTAINER)
> + if (!is_container(level))
> return 0;
> if (!dev)
> return 1;
> @@ -7692,7 +7692,7 @@ static int validate_geometry_imsm(struct supertype *st, int level, int layout,
> * if given unused devices create a container
> * if given given devices in a container create a member volume
> */
> - if (level == LEVEL_CONTAINER)
> + if (is_container(level))
> /* Must be a fresh device to add to a container */
> return validate_geometry_imsm_container(st, level, raiddisks,
> data_offset, dev,
> diff --git a/super0.c b/super0.c
> index 37f595ed..93876e2e 100644
> --- a/super0.c
> +++ b/super0.c
> @@ -1273,7 +1273,7 @@ static int validate_geometry0(struct supertype *st, int level,
> if (get_linux_version() < 3001000)
> tbmax = 2;
>
> - if (level == LEVEL_CONTAINER) {
> + if (is_container(level)) {
> if (verbose)
> pr_err("0.90 metadata does not support containers\n");
> return 0;
> diff --git a/super1.c b/super1.c
> index 58345e68..0b505a7e 100644
> --- a/super1.c
> +++ b/super1.c
> @@ -2830,7 +2830,7 @@ static int validate_geometry1(struct supertype *st, int level,
> unsigned long long overhead;
> int fd;
>
> - if (level == LEVEL_CONTAINER) {
> + if (is_container(level)) {
> if (verbose)
> pr_err("1.x metadata does not support containers\n");
> return 0;
> diff --git a/sysfs.c b/sysfs.c
> index 0d98a65f..ca1d888f 100644
> --- a/sysfs.c
> +++ b/sysfs.c
> @@ -763,7 +763,7 @@ int sysfs_add_disk(struct mdinfo *sra, struct mdinfo *sd, int resume)
>
> rv = sysfs_set_num(sra, sd, "offset", sd->data_offset);
> rv |= sysfs_set_num(sra, sd, "size", (sd->component_size+1) / 2);
> - if (sra->array.level != LEVEL_CONTAINER) {
> + if (!is_container(sra->array.level)) {
> if (sra->consistency_policy == CONSISTENCY_POLICY_PPL) {
> rv |= sysfs_set_num(sra, sd, "ppl_sector", sd->ppl_sector);
> rv |= sysfs_set_num(sra, sd, "ppl_size", sd->ppl_size);
> --
> 2.26.2
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v4] mdadm: replace container level checking with inline
2022-09-03 7:11 ` Coly Li
@ 2022-09-29 15:05 ` Jes Sorensen
0 siblings, 0 replies; 4+ messages in thread
From: Jes Sorensen @ 2022-09-29 15:05 UTC (permalink / raw)
To: Coly Li, Kinga Tanska; +Cc: linux-raid
On 9/3/22 03:11, Coly Li wrote:
>
>
>> 2022年9月2日 14:49，Kinga Tanska <kinga.tanska@intel.com> 写道：
>>
>> To unify all containers checks in code, is_container() function is
>> added and propagated.
>>
>> Signed-off-by: Kinga Tanska <kinga.tanska@intel.com>
>
> Acked-by: Coly Li <colyli@suse.de>
>
>
> But this patch has a minor conflict with Mateusz’s “Manage: Block unsafe member failing” series, it is simply because your patch landed a bit late than Mateusz’s.
> I already rebased your patch in the mdadm-CI queue. After I finish to review all the pending patches, let’s see whether you should post a v6 version or I can post the rebased one for you.
Applied!
I pulled in this one since the Block series also had a conflict and this
one applied cleanly.
Thanks,
Jes
> Thanks.
>
> Coly Li
>
>
>> ---
>> Assemble.c | 7 +++----
>> Create.c | 6 +++---
>> Grow.c | 6 +++---
>> Incremental.c | 4 ++--
>> mdadm.h | 14 ++++++++++++++
>> super-ddf.c | 6 +++---
>> super-intel.c | 4 ++--
>> super0.c | 2 +-
>> super1.c | 2 +-
>> sysfs.c | 2 +-
>> 10 files changed, 33 insertions(+), 20 deletions(-)
>>
>> diff --git a/Assemble.c b/Assemble.c
>> index 1dd82a8c..8b0af0c9 100644
>> --- a/Assemble.c
>> +++ b/Assemble.c
>>
> [snipped]
>
>> @@ -1809,7 +1808,7 @@ try_again:
>> }
>> #endif
>> }
>> - if (c->force && !clean && content->array.level != LEVEL_CONTAINER &&
>> + if (c->force && !clean && !is_container(content->array.level) &&
>> !enough(content->array.level, content->array.raid_disks,
>> content->array.layout, clean, avail)) {
>> change += st->ss->update_super(st, content, "force-array”,
>
>
> The conflict is here, and the rebased change looks like this,
>
> @@ -1807,7 +1806,7 @@ try_again:
> }
> #endif
> }
> - if (c->force && !clean && content->array.level != LEVEL_CONTAINER &&
> + if (c->force && !clean && !is_container(content->array.level) &&
> !enough(content->array.level, content->array.raid_disks,
> content->array.layout, clean, avail)) {
> change += st->ss->update_super(st, content, UOPT_SPEC_FORCE_ARRAY,
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-09-29 15:06 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-09-02 6:49 [PATCH v4] mdadm: replace container level checking with inline Kinga Tanska
2022-09-03 7:11 ` Coly Li
2022-09-29 15:05 ` Jes Sorensen
2022-09-14 14:49 ` Coly Li
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).