From: Christoph Hellwig Subject: Re: [PATCH v18 09/22] richacl: Permission check algorithm Date: Fri, 11 Mar 2016 06:09:47 -0800 Message-ID: <20160311140947.GD14808@infradead.org> References: <1456733847-17982-1-git-send-email-agruenba@redhat.com> <1456733847-17982-10-git-send-email-agruenba@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Alexander Viro , "J. Bruce Fields" , linux-nfs@vger.kernel.org, Theodore Ts'o , linux-cifs@vger.kernel.org, linux-api@vger.kernel.org, Trond Myklebust , linux-kernel@vger.kernel.org, xfs@oss.sgi.com, Christoph Hellwig , Andreas Dilger , linux-fsdevel@vger.kernel.org, Jeff Layton , linux-ext4@vger.kernel.org, Anna Schumaker To: Andreas Gruenbacher Return-path: Content-Disposition: inline In-Reply-To: <1456733847-17982-10-git-send-email-agruenba@redhat.com> Sender: linux-fsdevel-owner@vger.kernel.org List-Id: linux-ext4.vger.kernel.org > fs/Makefile | 2 +- > fs/richacl_inode.c | 149 ++++++++++++++++++++++++++++++++++++++++++++++++ What's the point of a tiny separate file here? All richacls files together are still small, and it would be much preferably to have all that code together.