From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Howells Subject: Re: [PATCH] Rearrange i_flags to be consistent with FS_IOC_GETFLAGS Date: Tue, 06 Jul 2010 14:40:23 +0100 Message-ID: <29431.1278423623@redhat.com> References: <20100706001032.GA25018@dastard> <20100705154319.31193.56706.stgit@warthog.procyon.org.uk> Cc: dhowells@redhat.com, viro@ZenIV.linux.org.uk, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, linux-ext4@vger.kernel.org To: Dave Chinner Return-path: In-Reply-To: <20100706001032.GA25018@dastard> Sender: linux-ext4-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org Dave Chinner wrote: > Further, it opens up the possibility that further down the track > someone modifies S_* flag values (say when adding some XFS > functionality) and corrupts extN filesystems all over the place. > There isn't even compiler guards to catch someone modifying the S_* > flags in a way that makes it incompatible with the extN on-disk > definitions. That occurred to me after I sent the patch. I can add some preprocessor guards for this. David