From mboxrd@z Thu Jan 1 00:00:00 1970 From: Al Viro Subject: Re: remove inode_setattr Date: Wed, 11 Aug 2010 02:00:57 +0100 Message-ID: <20100811010057.GC31363@ZenIV.linux.org.uk> References: <201008101859.o7AIx9BM026915@hera.kernel.org> <20100810155011.b9657102.akpm@linux-foundation.org> <20100811101751.1afdded8.sfr@canb.auug.org.au> <20100811104503.4082f68f.sfr@canb.auug.org.au> <20100811005216.GB31363@ZenIV.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from zeniv.linux.org.uk ([195.92.253.2]:56383 "EHLO ZenIV.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752370Ab0HKBBC (ORCPT ); Tue, 10 Aug 2010 21:01:02 -0400 Content-Disposition: inline In-Reply-To: <20100811005216.GB31363@ZenIV.linux.org.uk> Sender: linux-next-owner@vger.kernel.org List-ID: To: Stephen Rothwell Cc: Andrew Morton , Christoph Hellwig , Linus Torvalds , Eric Van Hensbergen , Sripathi Kodi , LKML , linux-next@vger.kernel.org On Wed, Aug 11, 2010 at 01:52:16AM +0100, Al Viro wrote: > On Wed, Aug 11, 2010 at 10:45:03AM +1000, Stephen Rothwell wrote: > > Hi All, > > > > On Wed, 11 Aug 2010 10:17:51 +1000 Stephen Rothwell wrote: > > > > > > Reported by me June 22 with a fixup, reminded Aug 4 and yesterday ... it > > > needed to be fixed up by Linus when he merged Al's tree ... > > > > I will apply this fix patch to linux-next today ... Al, Christoph, this > > is the fix I sent out earlier. Is it correct enough for Linus to apply > > it? > Whiskey Tango Fotrot, Over > > In Linus' tree we have > commit 1025774ce411f2bd4b059ad7b53f0003569b74fa > Author: Christoph Hellwig > Date: Fri Jun 4 11:30:02 2010 +0200 > ... > diff --git a/fs/9p/vfs_inode.c b/fs/9p/vfs_inode.c > index 4331b3b..4b3ad6a 100644 > --- a/fs/9p/vfs_inode.c > +++ b/fs/9p/vfs_inode.c > @@ -896,10 +896,19 @@ static int v9fs_vfs_setattr(struct dentry *dentry, struct D'oh. Different function, appeared there only recently. OK, I'll drop your fix into today's pull request (in about an hour). ACK.