mdadm: Apply to revert the commit of a4dcdb23

mdadm: Apply to revert the commit of a4dcdb23

[Zhilong Liu]

Hi, Jes;
    Excuse me, there is one problematic commit for Detail.c,
a4dcdb23(Detail: determine array state from sysfs).
This commit does change the codes meaning and it didn't
run to the right path. This commit was result in a bug that
Detail() cannot work normally.

For example:
# ./mdadm -CR /dev/md0 -l1 -n2 /dev/loop[0-1] --assume-clean
# ./mdadm -D /dev/md0
mdadm: cannot get array detail for /dev/md0: No such file or directory

Thanks very much,
-Zhilong

Re: mdadm: Apply to revert the commit of a4dcdb23

[Jes Sorensen]

On 06/06/2017 05:25 AM, Zhilong Liu wrote:
> Hi, Jes;
>     Excuse me, there is one problematic commit for Detail.c,
> a4dcdb23(Detail: determine array state from sysfs).
> This commit does change the codes meaning and it didn't
> run to the right path. This commit was result in a bug that
> Detail() cannot work normally.
> 
> For example:
> # ./mdadm -CR /dev/md0 -l1 -n2 /dev/loop[0-1] --assume-clean
> # ./mdadm -D /dev/md0
> mdadm: cannot get array detail for /dev/md0: No such file or directory
> 
> Thanks very much,
> -Zhilong

Please be more specific and explain what you believe is wrong. I prefer 
a patch fixing the issue to just reverting things.

Jes

Re: mdadm: Apply to revert the commit of a4dcdb23

[Zhilong Liu]

On 06/07/2017 12:02 AM, Jes Sorensen wrote:
> On 06/06/2017 05:25 AM, Zhilong Liu wrote:
>> Hi, Jes;
>>     Excuse me, there is one problematic commit for Detail.c,
>> a4dcdb23(Detail: determine array state from sysfs).
>> This commit does change the codes meaning and it didn't
>> run to the right path. This commit was result in a bug that
>> Detail() cannot work normally.
>>
>> For example:
>> # ./mdadm -CR /dev/md0 -l1 -n2 /dev/loop[0-1] --assume-clean
>> # ./mdadm -D /dev/md0
>> mdadm: cannot get array detail for /dev/md0: No such file or directory
>>
>> Thanks very much,
>> -Zhilong
>
> Please be more specific and explain what you believe is wrong. I 
> prefer a patch fixing the issue to just reverting things.
>

I just re-checked the mailing-list, found the email that post the commit 
of 9b8fea914f82
from Tomasz in May.24th, it has already fixed this issue. please ignore 
this mail. Sorry
for that.

Thanks,
-Zhilong

> Jes
>
> -- 
> 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
>