On Wed, Sep 11, 2019 at 3:44 AM Yufen Yu <yuyufen@xxxxxxxxxx> wrote: > > Hi, > For this patchset , we add a new entry .disk_is_spare in super_type > to check if the disk is in 'spare' state. If a disk is in spare, > analyze_sbs() should skip the disk to be freshest disk. Otherwise, > it may cause md run fail. There is a fail example in the second patch. > I think we need go a different path. I am sorry that some of early comments are misleading. We can extend the output of load_super() to have "2" for spares. And this _should_ make the code simpler. Does this make sense? Thanks, Song