NeilBrown <ne...@suse.com> writes: > Allow per-device "failfast" flag to be set when creating an > array or adding devices to an array. > > When re-adding a device which had the failfast flag, it can be removed > using --nofailfast. > > failfast status is printed in --detail and --examine output. > > Signed-off-by: NeilBrown <ne...@suse.com> > --- > > Hi Jes, > this patch adds mdadm support for the failfast functionality that > Shaohua recently included in his for-next. > Hopefully the man-page additions provide all necessary context. > If there is anything that seems to be missing, I'll be very happy to > add it.
Hi Neil, It looks reasonable. The only minor concern I have is over the use of hardcoded magic numbers like 'dv->failfast = 2' rather than using an enum or defines with descriptive names. Something that can be addressed later, so patch applied. Cheers, Jes
