From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [PATCH] fs: FS_POSIX_ACL does not depend on BLOCK Date: Sun, 2 Jan 2011 01:29:48 -0500 Message-ID: <20110102062948.GA582@infradead.org> References: <20110101221716.70ef7f5a.randy.dunlap@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-fsdevel@vger.kernel.org, akpm , viro To: Randy Dunlap Return-path: Received: from bombadil.infradead.org ([18.85.46.34]:51806 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750941Ab1ABG3u (ORCPT ); Sun, 2 Jan 2011 01:29:50 -0500 Content-Disposition: inline In-Reply-To: <20110101221716.70ef7f5a.randy.dunlap@oracle.com> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Sat, Jan 01, 2011 at 10:17:16PM -0800, Randy Dunlap wrote: > +config FS_POSIX_ACL > +# Posix ACL utility routines > +# (for now, ext2/ext3/ext4/jfs/jffs2/reiserfs/xfs/btrfs/gfs2/ocfs2/nfs4/9p) > +# > +# NOTE: you can implement Posix ACLs without these helpers. > +# Never use this symbol for ifdefs. > +# > + def_bool n > + I'd drop the list of filesystems, and move the comment above the symbol, e.g: # # Posix ACL utility routines # # Note: Posix ACLs can be implemented without these helpers. Never use # this symbol for ifdefs in core code. # config FS_POSIX_ACL def_bool n