On Monday June 19, [EMAIL PROTECTED] wrote:
> > Neil Brown wrote:
> > I am pleased to announce the availability of
> >    mdadm version 2.5.1
> 
> What the heck, here's another one. :) This one is slightly more serious. 
> We're getting a device of "0:0" in "Fail" events from the mdadm monitor 
> sometimes now (due to the change in map_dev, which allows it to 
> sometimes return "0:0" instead of just NULL for an unknown device).
> 

Thanks for this and the other two.  They are now in .git

> The patch fixes my issue. I don't know if there are more.
> 

I chose to do this differently - map_dev will now return NULL for 0,0
and all users can cope with a NULL.

NeilBrown



> Thanks,
> Paul
> --- mdadm-2.5.1/Monitor.c     Thu Jun  1 21:33:41 2006
> +++ mdadm-2.5.1-new/Monitor.c Mon Jun 19 14:51:31 2006
> @@ -328,7 +328,7 @@ int Monitor(mddev_dev_t devlist,
>                                       }
>                                       disc.major = disc.minor = 0;
>                               }
> -                             if (dv == NULL && st->devid[i])
> +                             if ((dv == NULL || strcmp(dv, "0:0") == 0) && 
> st->devid[i])
>                                       dv = map_dev(major(st->devid[i]),
>                                                    minor(st->devid[i]), 1);
>                               change = newstate ^ st->devstate[i];
-
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to