On Fri, Aug 20, 2021 at 04:19:22PM +0800, Guoqing Jiang wrote: > How about this? > > +???????????????????????????? /* > +?????????????????????????????? * The write-behind io is only attempted on drives marked as > +?????????????????????????????? * write-mostly, which means we will allocate write behind > +?????????????????????????????? * bio later. > +?????????????????????????????? */ > ?????????????????????????????? if (test_bit(WriteMostly, &mirror->rdev->flags)) > ?????????????????????????????????????????????? write_behind = true; Except for the weird formatting of the whitespaces this looks good.