On Wed, Sep 17, 2014 at 12:35:29AM +0100, Alasdair G Kergon wrote: > > > - ret = read_disk_sb(rdev, rdev->sb_size); And if that needed changing to 4096, why does this similar instance elsewhere one not also need changing? static void attempt_restore_of_faulty_devices(struct raid_set *rs) { ... sync_page_io(r, 0, r->sb_size, r->sb_page, READ, 1)) { Alasdair -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel