On Wed, Jul 12, 2017 at 02:13:38PM +0200, Carlos Maiolino wrote:
> Ugh, sorry, I missed these comments during my past reply
>
> > > +
> > > +_mkfs_dev /dev/mapper/$vgname-$lvname >>$seqres.full 2>&1
> >
> > Sorry, I should've asked earlier in your RFC patch.. can all these setup
> > work be done with helpers in common/dmthin? e.g. _dmthin_init,
> > _dmthin_grow, the backing dev size, virtual size etc. are handled
> > properly in dmthin helpers. You could take generic/347 as an example.
> >
>
> I could do that if strictly required, but I'll also need to add a new dmthin
> helper to create snapshots, once this tests requires a snapshot. IIRC, LVM uses
> some 'tricks' to create snapshots (see device-mapper doc in kernel source), so
> this might add unneeded complexity to the test.
>
> I can use the helpers in common/dmthin for sure, but if this is not strictly
> required, I'd just stay with LVM instead of using device-mapper directly.
I don't think it's strictly required, if there're some good reasons. But
I really like to see the reasons are explained in the test as comments.
Otherwise it just makes me wondering why snapshot is needed, because
it's not mentioned in the commit log nor in the test description.
Thanks,
Eryu
P.S.
I played with this test a bit today and tried to get rid of snapshot but
failed. The difference I can see between using snapshot or not is that,
snapshot somehow makes metadata I/O error happens on block 0x0, while
without it the error happens on other blocks, e.g. block 0xc100.
XFS (dm-4): metadata I/O error: block 0x0 ("xfs_buf_iodone_callback_error") error 28 numblks 1
vs
XFS (dm-4): metadata I/O error: block 0xc100 ("xfs_buf_iodone_callback_error") error 28 numblks 32
--
To unsubscribe from this list: send the line "unsubscribe fstests" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
