On Tue, Mar 15, 2016 at 01:46:16PM +1100, Dave Chinner wrote: > On Fri, Mar 11, 2016 at 11:40:22PM +0800, Eryu Guan wrote: > > This is a test that performs simple I/O on dm error device, which > > returns EIO on all I/O request. > > > > This is motivated by an ext4 bug that crashes kernel on error path when > > trying to update atime. Following kernel patch should fix the issue > > > > ext4: fix NULL pointer dereference in ext4_mark_inode_dirty() > > Why does this test require the loop device? Why can't you just > unmount the filesystem, run 'blkdev --flushbufs <dev>' to ensure > there are no cached buffers/pages on the block device, then mount > it again? Yes, 'blockdev --flushbufs <dev>' works, and I found that I only need to add a blockdev call before dropping caches. This makes the code much cleaner and easier to read, perhaps the first patch can be dropped as well. I'll send out v2 shortly. Thanks for the review! Eryu -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
