Re: [PATCH] xfs: only SetPageUptodate if all buffers are uptodate

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

 



On Tue, Apr 19, 2011 at 06:40:28AM -0500, Ben Myers wrote:
> xfs_vm_writepage and xfs_page_convert set a page uptodate if it is determined
> that all of the buffer_heads attached to that page are uptodate.
> 
> Currently we use the flag variable 'uptodate'.  The flag is initially set = 1
> and it is cleared if a !buffer_uptodate buffer is encountered.  In addition we
> check that bh == head in order to ensure that all of the buffer_heads have been
> checked.  However, it is possible to break out of the buffer_head loop early
> having processed only the first buffer.  This leaves uptodate == 1 and bh ==
> head, so the uptodate bit can be set on a page even if not all of the buffers
> have been checked.  This can lead to data corruption on platforms with > 1
> buffer per page.

Dou have a testcase to reproduce this issue?

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs


[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux