On Tue, Feb 13, 2018 at 03:54:04PM +0100, Christoph Hellwig wrote:
Looks good:
Reviewed-by: Christoph Hellwig hch@lst.de
Can you wire up your test cases for blktests?
Is blktests really the right place for this test? This failure is highly dependent on the configuration of the filesystem that is holding the file that we are using for the loopback device. It doesn't seem like blktests has support for mount options (dax), etc?
Because of the interaction with the underlying filesystem this seems like a better fit with xfstests to me, but I don't know if we need to add tests there because we already have pretty good coverage of loopback device failures. That's how we found this - this bug causes all these tests to fail: xfs/074 xfs/078 xfs/216 xfs/217 xfs/250