Re: Review: ensure EOF writes into existing extents update filesize

xfs

Re: Review: ensure EOF writes into existing extents update filesize

To:	David Chinner <dgc@xxxxxxx>
Subject:	Re: Review: ensure EOF writes into existing extents update filesize
From:	Chris Wedgwood <cw@xxxxxxxx>
Date:	Sun, 20 May 2007 16:40:48 -0700
Cc:	xfs-dev <xfs-dev@xxxxxxx>, xfs-oss <xfs@xxxxxxxxxxx>
In-reply-to:	<20070520233417.GA85884050@xxxxxxx>
References:	<20070520233417.GA85884050@xxxxxxx>
Sender:	xfs-bounce@xxxxxxxxxxx

On Mon, May 21, 2007 at 09:34:17AM +1000, David Chinner wrote:

> In that case, we use a read mapping of the extent, and this also
> happens to use the read I/O completion handler instead of the
> write I/O completion handle. Hence the file size was not updated
> on I/O completion.
>
> Comments?

I've had this (well, the first version you sent to Jeremy) for a few
days on a testing machine without any obvious ill effects.

Once people are happy with this it should be checked to see if it
needs to go into -stable.

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
Review: ensure EOF writes into existing extents update filesize, David Chinner Re: Review: ensure EOF writes into existing extents update filesize, Chris Wedgwood <= Re: Review: ensure EOF writes into existing extents update filesize, David Chinner Re: Review: ensure EOF writes into existing extents update filesize, Timothy Shimmin Re: Review: ensure EOF writes into existing extents update filesize, David Chinner Re: Review: ensure EOF writes into existing extents update filesize, Nathan Scott Re: Review: ensure EOF writes into existing extents update filesize, David Chinner Re: Review: ensure EOF writes into existing extents update filesize, Nathan Scott Re: Review: ensure EOF writes into existing extents update filesize, David Chinner Re: Review: ensure EOF writes into existing extents update filesize, David Chinner Re: Review: ensure EOF writes into existing extents update filesize, Nathan Scott

Previous by Date:	Review: ensure EOF writes into existing extents update filesize, David Chinner
Next by Date:	Re: Review: ensure EOF writes into existing extents update filesize, David Chinner
Previous by Thread:	Review: ensure EOF writes into existing extents update filesize, David Chinner
Next by Thread:	Re: Review: ensure EOF writes into existing extents update filesize, David Chinner
Indexes:	[Date] [Thread] [Top] [All Lists]