From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boaz Harrosh Subject: Re: Wrong DIF guard tag on ext2 write Date: Mon, 07 Jun 2010 20:22:28 +0300 Message-ID: <4C0D2AD4.2020303@panasas.com> References: <20100531112817.GA16260@schmichrtp.mainz.de.ibm.com> <20100531144648.GA19068@schmichrtp.mainz.de.ibm.com> <20100602133748.GA11196@schmichrtp.mainz.de.ibm.com> <20100602232039.GA8287@dastard> <20100604023253.GF19651@dastard> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Cc: Dave Chinner , Christof Schmitt , linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org To: "Martin K. Petersen" Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On 06/07/2010 07:20 PM, Martin K. Petersen wrote: >>>>>> "Dave" == Dave Chinner writes: > >>> Didn't you use to wait_on_page_writeback() in page_mkwrite()? > > Dave> The generic implementation of ->page_mkwrite > Dave> (block_page_mkwrite()) which XFS uses has never had a > Dave> wait_on_page_writeback() call in it. There's no call in the > Dave> generic write paths, either, hence my comment that only direct IO > Dave> on XFS will work. > > I guess that wait_on_page_writeback() was something I added when I used > XFS for DIF testing. > Do you remember some performance numbers that show degradation / sameness? What type of work loads? Thanks Boaz