[PATCH] FIX: imsm: Do not change serial if disk failed

This patch rollback one change connected with mdadm-OROM
compatibility:
adding ':0' at the end of disk serial number if disk is
detected as failed.
Current mdadm's implementation does not distinguish two
cases when disk is marked as failed:
1. If disk is really failed- disconnected, broken
2. Just marked as failed by mdadm- using "-f" option

Second case is not yet fully handled and compatible with
IMSM standard.
Changing serial number of existing, operational disk causes
problems in "thunderdome" and "load_super" functions that use
serial numbers to disks comparisons and searching.
The change must be recalled until full support will be
developed.

Signed-off-by: Krzysztof Wojcik
---
super-intel.c | 5 -----
1 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/super-intel.c b/super-intel.c
index 2b41e08..a91bcc2 100644
--- a/super-intel.c
+++ b/super-intel.c
@@ -5232,11 +5232,6 @@ static int mark_failure(struct imsm_dev *dev, struct imsm_disk *disk, int idx)
if (is_failed(disk) && (ord & IMSM_ORD_REBUILD))
return 0;

- sprintf(buf, "%s:0", disk->serial);
- if ((len = strlen(buf)) >= MAX_RAID_SERIAL_LEN)
- shift = len - MAX_RAID_SERIAL_LEN + 1;
- strncpy((char *)disk->serial, &buf[shift], MAX_RAID_SERIAL_LEN);
-
disk->status |= FAILED_DISK;
set_imsm_ord_tbl_ent(map, slot, idx | IMSM_ORD_REBUILD);
if (map->failed_disk_num == 0xff)

--
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: [PATCH] FIX: imsm: Do not change serial if disk failed

On Wed, 23 Mar 2011 17:36:29 +0100 Krzysztof Wojcik
wrote:

> This patch rollback one change connected with mdadm-OROM
> compatibility:
> adding ':0' at the end of disk serial number if disk is
> detected as failed.
> Current mdadm's implementation does not distinguish two
> cases when disk is marked as failed:
> 1. If disk is really failed- disconnected, broken
> 2. Just marked as failed by mdadm- using "-f" option
>
> Second case is not yet fully handled and compatible with
> IMSM standard.
> Changing serial number of existing, operational disk causes
> problems in "thunderdome" and "load_super" functions that use
> serial numbers to disks comparisons and searching.
> The change must be recalled until full support will be
> developed.
>
> Signed-off-by: Krzysztof Wojcik
> ---
> super-intel.c | 5 -----
> 1 files changed, 0 insertions(+), 5 deletions(-)
>
> diff --git a/super-intel.c b/super-intel.c
> index 2b41e08..a91bcc2 100644
> --- a/super-intel.c
> +++ b/super-intel.c
> @@ -5232,11 +5232,6 @@ static int mark_failure(struct imsm_dev *dev, struct imsm_disk *disk, int idx)
> if (is_failed(disk) && (ord & IMSM_ORD_REBUILD))
> return 0;
>
> - sprintf(buf, "%s:0", disk->serial);
> - if ((len = strlen(buf)) >= MAX_RAID_SERIAL_LEN)
> - shift = len - MAX_RAID_SERIAL_LEN + 1;
> - strncpy((char *)disk->serial, &buf[shift], MAX_RAID_SERIAL_LEN);
> -
> disk->status |= FAILED_DISK;
> set_imsm_ord_tbl_ent(map, slot, idx | IMSM_ORD_REBUILD);
> if (map->failed_disk_num == 0xff)


Applied thanks.

That was my last remaining issue before I'm comfortable releasing
mdadm-3.2.1
Does Intel have anything concerns about releasing the current code as
mdadm-3.2.1?

Thanks,
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

RE: [PATCH] FIX: imsm: Do not change serial if disk failed

> -----Original Message-----
> From: linux-raid-owner@vger.kernel.org [mailto:linux-raid-
> owner@vger.kernel.org] On Behalf Of NeilBrown
> Sent: Thursday, March 24, 2011 12:18 AM
> To: Wojcik, Krzysztof
> Cc: linux-raid@vger.kernel.org; Neubauer, Wojciech; Kwolek, Adam;
> Williams, Dan J; Ciechanowski, Ed
> Subject: Re: [PATCH] FIX: imsm: Do not change serial if disk failed
>
> That was my last remaining issue before I'm comfortable releasing
> mdadm-3.2.1
> Does Intel have anything concerns about releasing the current code as
> mdadm-3.2.1?

Neil,

We have only two patches for test suite. Anna will sent them today.
After applying this, we are ready to 3.2.1 release.

Regards
Krzysztof

>
> Thanks,
> 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
--
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