From mboxrd@z Thu Jan 1 00:00:00 1970 From: Al Viro Subject: Re: [patch 00/11] new truncate sequence Date: Tue, 22 Sep 2009 22:51:38 +0100 Message-ID: <20090922215138.GH14381@ZenIV.linux.org.uk> References: <20090820163504.131529718@suse.de> <20090922150444.GA14566@ZenIV.linux.org.uk> <20090922200042.GA27015@lst.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: npiggin@suse.de, Andrew Morton , linux-fsdevel@vger.kernel.org To: Christoph Hellwig Return-path: Received: from zeniv.linux.org.uk ([195.92.253.2]:55554 "EHLO ZenIV.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752867AbZIVVvg (ORCPT ); Tue, 22 Sep 2009 17:51:36 -0400 Content-Disposition: inline In-Reply-To: <20090922200042.GA27015@lst.de> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Tue, Sep 22, 2009 at 10:00:42PM +0200, Christoph Hellwig wrote: > On Tue, Sep 22, 2009 at 04:04:44PM +0100, Al Viro wrote: > > On Fri, Aug 21, 2009 at 02:35:04AM +1000, npiggin@suse.de wrote: > > > Hi, > > > > > > I have made a few small changes that were suggested, and converted > > > a few more filesystems over. I didn't spend so much time on converting > > > them this week so I think it should be best to just get it out now. > > > > Umm... who calls jfs_setsize()? > > Yeah, noticed this while looking over the series. The JFS patch seems > to be botched. It is... I really don't like some parts of that: * failing on foo_truncate() is all nice and proper, but what to do about the quota transfer that has already happened? Note that e.g. ext2_setattr() simply doesn't change uid/gid in case of ext2_setsize() failure. It does transfer quota, though. And yes, nfsd can produce calls with both ATTR_SIZE and ATTR_UID set. * way, _way_ too much boilerplate around the aforementioned quota transfers. * if we don't do quota handling for minix and sysv (fair enough, no quota support implemented for those), why the devil are we doing it in simple_setattr()? Anyway, #new-truncate in the usual VFS tree contains all "new truncate scheme" stuff I've collected from the lists so far. Additions are welcome...