xfs
[Top] [All Lists]

Re: [patch 1/2] From: Bart Samwel <bart@xxxxxxxxx>

To: Bart Samwel <bart@xxxxxxxxx>
Subject: Re: [patch 1/2] From: Bart Samwel <bart@xxxxxxxxx>
From: Nathan Scott <nathans@xxxxxxx>
Date: Tue, 6 Apr 2004 09:44:23 +1000
Cc: XFS mailing list <linux-xfs@xxxxxxxxxxx>
In-reply-to: <1081194839.2996.37.camel@xxxxxxxxx>
References: <E1B9lJh-0001SC-T6@xxxxxxxxx> <20040405084050.B2647@xxxxxxxxxxxxx> <1081194839.2996.37.camel@xxxxxxxxx>
Sender: linux-xfs-bounce@xxxxxxxxxxx
User-agent: Mutt/1.5.3i
On Mon, Apr 05, 2004 at 09:54:00PM +0200, Bart Samwel wrote:
> On Mon, 2004-04-05 at 09:40, Christoph Hellwig wrote:
> > > * Switch from keeping track of "flush time" to keeping track of "creation 
> > > time"
> > >   for a pagebuf, so that changes in xfs_age_buffer can take effect 
> > > immediately
> > >   instead of only for pagebufs whose flush time is set anew.
> > 
> > looks good but doesn't apply against the XFS CVS tree which has some
> > additional changes vs linus' tree in that area.
> Attached patch is against the XFS CVS tree.

Minor thing in the original patch - could you name the field
pb_queuetime instead of creation time?

Also, could you resend (or send patch inline)...

> -- Binary/unsupported file stripped by Ecartis --
> -- Type: text/x-patch
> -- File: xfs-laptop-mode.patch

I think Russell has just switched this Ecartis behavior off, so
either should work.  The other patches all came through fine as
they were in the message body.

thanks!

-- 
Nathan


<Prev in Thread] Current Thread [Next in Thread>