On 29/07/2019 21:08, NeilBrown wrote: >[...] >> + if (unlikely(test_bit(MD_BROKEN, &mddev->flags))) { >> + bio_io_error(bio); >> + return BLK_QC_T_NONE; >> + } > > I think this should only fail WRITE requests, not READ requests. > > Otherwise the patch is probably reasonable. > > NeilBrown Thanks for the feedback Neil! I thought about it; it seemed to me better to deny/fail the reads instead of returning "wrong" reads, since a file read in a raid0 will be incomplete if one member is missing. But it's fine for me to change that in the next iteration of this patch. Cheers, Guilherme