Re: [PATCH v2 3/3] DAX: mmap write readonly file

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]



Eryu Guan <eguan@xxxxxxxxxx> writes:

> On Mon, Apr 10, 2017 at 02:05:53PM +0800, Xiong Zhou wrote:
>> Regression case that one can write to read-only
>> file in a DAX mountpoint.
>> 
>> Signed-off-by: Xiong Zhou <xzhou@xxxxxxxxxx>
>> ---
>> 
>> v2:
>>   compile test programme manually in this test because default
>> cc option -O2 prevents this issue reproduction;
>
> Hmm, this looks.. ugly to me :) Better to find out the exact reason that
> prevents the bug from reproducing and update the c program accordingly.

The compiler probably optimizes this bit out:

+	/* fault in the page */
+	foo = *buf;

Cheers,
Jeff
--
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



[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux