From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vyacheslav Dubeyko Subject: Re: [PATCH v3 2/3] hfsplus: add implementation of the ACLs support Date: Tue, 05 Mar 2013 10:52:56 +0400 Message-ID: <1362466376.2572.25.camel@slavad-ubuntu> References: <1362324572.29435.42.camel@slavad-ubuntu-12.04> <20130304212638.GJ20389@fieldses.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: linux-fsdevel@vger.kernel.org, Andrew Morton , Christoph Hellwig , Al Viro , Hin-Tak Leung To: "J. Bruce Fields" Return-path: Received: from oproxy12-pub.bluehost.com ([50.87.16.10]:57301 "HELO oproxy12-pub.bluehost.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1751036Ab3CEGxM (ORCPT ); Tue, 5 Mar 2013 01:53:12 -0500 In-Reply-To: <20130304212638.GJ20389@fieldses.org> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Mon, 2013-03-04 at 16:26 -0500, J. Bruce Fields wrote: > On Sun, Mar 03, 2013 at 07:29:32PM +0400, Vyacheslav Dubeyko wrote: > > From: Vyacheslav Dubeyko > > Subject: [PATCH v3 2/3] hfsplus: add implementation of the ACLs support > > > > This patch adds implementation of the ACLs support for hfsplus driver. > > Much of this actually appears to have started as a cut-and-paste of > fs/nfsd/nfs4acl.c. > > Could you explain what you needed to change from the nfsd code? > > It looks like we should be able to share at least some of this code > instead of copying it. The posix<->NFSv4 ACL mapping is a bit tricky > and I'd prefer we not have to fix bugs in two different instances of it. > Yes, I totally agree with you. I used fs/nfsd/nfs4acl.c as a basis for implementation of ACLs support in hfsplus. But it was a first step because I can't used implemented in nfsd functionality without changes. The ACLs in hfsplus have some peculiarities (special raw ACLs representation and slightly special set of flags) and I needed to achieve first of all working implementation of ACLs support. Now it is done and I think over how to share common code between hfsplus and nfsd. Maybe it can be useful and for other drivers. As a preliminary vision, I think that we can have something like pair of files: include/linux/nfsv4_acl.h (declarations) and fs/nfsv4_acl.c (implementation). The implementation should share common code with possibility to implement specialized part functionality in the concrete driver. > Also: have you looked at the latest rich ACL patches? That would > provide a native interface to NFSv4-like ACLs, which would really be > much more useful. > Could you point out about what patches you are talking? Maybe these patches correct my vision of sharing code between hfsplus and nfsd. Thanks, Vyacheslav Dubeyko. > --b. >