From: Andrew Morton Subject: Re: [PATCHv4 01/17] VFS: introduce helpers for the s_dirty flag Date: Wed, 9 Jun 2010 09:31:57 -0700 Message-ID: <20100609093157.cd040633.akpm@linux-foundation.org> References: <1274795352-3551-1-git-send-email-dedekind1@gmail.com> <1274795352-3551-2-git-send-email-dedekind1@gmail.com> <20100528132318.0783675a.akpm@linux-foundation.org> <20100528211432.GM31073@ZenIV.linux.org.uk> <20100528141755.42958d81.akpm@linux-foundation.org> <20100609154448.GC6162@thunk.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Al Viro , Artem Bityutskiy , LKML , Jens Axboe , linux-fsdevel@vger.kernel.org, Roman Zippel , "Tigran A. Aivazian" , Chris Mason , Boaz Harrosh , linux-ext4@vger.kernel.org, OGAWA Hirofumi , David Woodhouse , reiserfs-devel@vger.kernel.org, Jan Kara , Evgeniy Dushistov To: tytso@mit.edu Return-path: In-Reply-To: <20100609154448.GC6162@thunk.org> Sender: linux-fsdevel-owner@vger.kernel.org List-Id: linux-ext4.vger.kernel.org On Wed, 9 Jun 2010 11:44:49 -0400 tytso@mit.edu wrote: > On Fri, May 28, 2010 at 02:17:55PM -0700, Andrew Morton wrote: > > > > > > The real issue is that it's almost certainly an overdesign. Let's > > > get rid of the bogus uses first and figure out what's happening in > > > what remains, OK? > > > > That would be good. > > Can we figure out what the new names will be for these accessor > functions, sb_mark_dirty(), sb_mark_clean(), sb_is_dirty(). > and then pursuade Linus to be willing to add patch #1 in > this series to add these accessor functions (without any users for > these functions, that would wait until the next merge window) to > 2.6.35-rc3 or -rc4, please? I expect he'd be OK with that. > It will make life much easier for fs maintainers to merge the patches, > especially if they've done some cleanup to reduce the bogus places > where s_dirt was getting set in the first place. For that reason.