On 03/05/2015 11:24 AM, Boaz Harrosh wrote: > > [v1] > Without this patch, c/mtime is not updated correctly when mmap'ed page is > first read from and then written to. > > A new xfstest is submitted for testing this (generic/080) > > [v2] > Jan Kara has pointed out that if we add the > sb_start/end_pagefault pair in the new pfn_mkwrite we > are then fixing another bug where: A user could start > writing to the page while filesystem is frozen. > Thanks Jan. Just as curiosity, does the freezing code goes and turns all mappings into read-only, Also for pfn mapping? Do you think there is already an xfstest freezing test that should now fail, and will succeed after this patch (v2). Something like: * mmap-read/write before the freeze * freeze the fs * Another thread tries to mmap-write, should get stuck * unfreeze the fs * Now mmap-writer continues Thanks again Boaz > CC: Jan Kara <jack@xxxxxxx> > Signed-off-by: Yigal Korman <yigal@xxxxxxxxxxxxx> > Signed-off-by: Boaz Harrosh <boaz@xxxxxxxxxxxxx> <> -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html