From mboxrd@z Thu Jan  1 00:00:00 1970
From: Christoph Hellwig <hch@infradead.org>
Subject: Re: [RFC] POSIX ACL kernel infrastructure
Date: Fri, 9 Aug 2002 12:15:02 +0100
Sender: linux-fsdevel-owner@vger.kernel.org
Message-ID: <20020809121502.A8178@infradead.org>
References: <200208041546.35234.agruen@suse.de> <20020805132832.B5301@infradead.org> <20020809020223.GF731@frodo> <200208091253.55935.agruen@suse.de>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: Nathan Scott <nathans@sgi.com>,
	Christoph Hellwig <hch@infradead.org>,
	Linux-FSDevel <linux-fsdevel@vger.kernel.org>,
	Olaf Kirch <okir@suse.de>, Chris Mason <mason@suse.de>,
	Jeff Mahoney <jeffm@suse.com>
Return-path: <linux-fsdevel-owner@vger.kernel.org>
To: Andreas Gruenbacher <agruen@suse.de>
Content-Disposition: inline
In-Reply-To: <200208091253.55935.agruen@suse.de>; from agruen@suse.de on Fri, Aug 09, 2002 at 12:53:55PM +0200
List-Id: linux-fsdevel.vger.kernel.org

On Fri, Aug 09, 2002 at 12:53:55PM +0200, Andreas Gruenbacher wrote:
> The kernel NFS daemon introduces a number of interesting problems in the 
> kernel that no other component requires; the ACL permission masking hack is 
> one of them. The hack affects NFSv2 and NFSv3 
> 
> It is really important that encode_fattr and encode_fattr3, the functions that 
> include the hack, are efficient; they are used in all the essential NFS RPCs.

IF it's so essential why are you doing another IOP call then instead of adding
inkernel A retrieving to ->getattr and struct kstat?  encode_fattr* has to
call it anyway and that's the logical place for ACLs..