Tomasz Majchrzak <tomasz.majchrzak@xxxxxxxxx> writes: > On create set bad block support flag for each drive. On assmble also > provide a list of known bad blocks. Bad blocks are stored in metadata > per disk so they have to be checked against volume boundaries > beforehand. > > Signed-off-by: Tomasz Majchrzak <tomasz.majchrzak@xxxxxxxxx> > --- > super-intel.c | 59 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 59 insertions(+) Tomasz, Thanks for posting 1/9 v3 - I was in the process of applying this set, but something is causing a conflict. > diff --git a/super-intel.c b/super-intel.c > index 421bfbc..b2afdff 100644 > --- a/super-intel.c > +++ b/super-intel.c [snip] > @@ -7160,6 +7212,12 @@ static struct mdinfo *container_content_imsm(struct supertype *st, char *subarra > info_d->events = __le32_to_cpu(mpb->generation_num); > info_d->data_offset = pba_of_lba0(map); > info_d->component_size = blocks_per_member(map); > + > + info_d->bb.supported = 0; > + get_volume_badblocks(super->bbm_log, ord_to_idx(ord), > + info_d->data_offset, > + info_d->component_size, > + &info_d->bb); > } > /* now that the disk list is up-to-date fixup recovery_start */ > update_recovery_start(super, dev, this); This hunk is failing as my tree doesn't have the line above: info_d->component_size = blocks_per_member(map); I can merge this manually, but I prefer to be sure we are in sync just in case. Where did that line come from - did I miss an earlier patch? Cheers, Jes -- To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html