On Tue, Sep 20, 2016 at 01:47:34PM -0700, Dave Jiang wrote: > Change dm-flakey to implement direct_access function, > flakey_direct_access(), which maps sector and calls direct_access > function of its physical target device. This pretty much is a copy > from DM Linear. > > This allows fs with DAX to pass in xfstests. They don't fail. They simply don't run like this: generic/321 1s ... [not run] Cannot run tests with DAX on dmflakey devices generic/322 0s ... [not run] Cannot run tests with DAX on dmflakey devices > It does not actually introduce > data corruption and dropping writes unlike the non-DAX path. The main > reason is due to the I/O path being out of dmflakey's control once > direct_access() has been called. The the existing implementation cannot > be adapted to DAX I/O path. The error injection will be introduced at a > later date with more thought. Then introduce dmflakey DAX support at that time. Running tests that are supposed to exercise failure paths when those failure paths cannot be triggered is fundamentally useless. It leads a false indication that we are actually testing failures, when in fact we are doing nothing of the sort. Right now the test harness execution clearly documents that there is no failure testing being run on DAX devices, and that's the way it should remain until dmflakey, dmerror, etc can inject the required errors into the DAX paths.... Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
