Mariusz Dabrowski <mariusz.dabrowski@xxxxxxxxx> writes:
> mdadm recognizes devices with partition table as part of an RAID array
> and invalid warning message is displayed. After this fix proper warning
> messages are being displayed for MBR/GPT disks and devices with RAID
> metadata.
>
> Signed-off-by: Mariusz Dabrowski <mariusz.dabrowski@xxxxxxxxx>
> ---
> util.c | 27 ++++++++++++++++-----------
> 1 file changed, 16 insertions(+), 11 deletions(-)
Looks good, applied!
Thanks,
Jes
> diff --git a/util.c b/util.c
> index 8b52242..a238a21 100644
> --- a/util.c
> +++ b/util.c
> @@ -710,17 +710,22 @@ int check_raid(int fd, char *name)
>
> if (!st)
> return 0;
> - st->ss->load_super(st, fd, name);
> - /* Looks like a raid array .. */
> - pr_err("%s appears to be part of a raid array:\n",
> - name);
> - st->ss->getinfo_super(st, &info, NULL);
> - st->ss->free_super(st);
> - crtime = info.array.ctime;
> - level = map_num(pers, info.array.level);
> - if (!level) level = "-unknown-";
> - cont_err("level=%s devices=%d ctime=%s",
> - level, info.array.raid_disks, ctime(&crtime));
> + if (st->ss->add_to_super != NULL) {
> + st->ss->load_super(st, fd, name);
> + /* Looks like a raid array .. */
> + pr_err("%s appears to be part of a raid array:\n", name);
> + st->ss->getinfo_super(st, &info, NULL);
> + st->ss->free_super(st);
> + crtime = info.array.ctime;
> + level = map_num(pers, info.array.level);
> + if (!level)
> + level = "-unknown-";
> + cont_err("level=%s devices=%d ctime=%s",
> + level, info.array.raid_disks, ctime(&crtime));
> + } else {
> + /* Looks like GPT or MBR */
> + pr_err("partition table exists on %s\n", name);
> + }
> return 1;
> }
--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
