First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 144845
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Gentoo's Team for Core System packages <base-system@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Christophe Saout <christophe@saout.de>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
acl-2.2.29-CITI_NFS4_ALL-3.diff The CITI acl-2.2.29-CITI_NFS4_ALL-3.dif minus the debian/changelog reject patch Christophe Saout 2006-08-23 04:37 0000 168.53 KB Details | Diff
acl-2.2.39-acl_extended_file-nfs4.diff Let "ls -l" indicate additonal ACLs for NFSv4 too. patch Christophe Saout 2006-08-23 04:38 0000 1.30 KB Details | Diff
acl-2.2.39.ebuild.diff acl-2.2.39.ebuild patch adding nfsv4 USE flag patch Christophe Saout 2006-08-23 04:39 0000 823 bytes Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 144845 depends on: Show dependency tree
Show dependency graph
Bug 144845 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)







View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2006-08-23 04:35 0000
NFSv4 uses its own ACL scheme which differs from the POSIX ACL scheme used by
other Linux filesystems.

While the NFSv4 kernel server maps the POSIX acls of the underlying filesystem
to NFSv4 ACLs over the wire, the client exports the whole NFSv4 ACLs to
userspace via its own "system.nfs4_acl" xattr, not via the POSIX xattr.

Therefore the raw libacl doesn' understand these ACLs.

The NFSv4 developers at University of Michigan
http://www.citi.umich.edu/projects/nfsv4/ provide a patch for libacl at
http://www.citi.umich.edu/projects/nfsv4/linux/acl-patches/2.2.29-3/acl-2.2.29-CITI_NFS4_ALL-3.dif

When ignoring the reject for debian/changelog this patch applies cleanly to the
current acl-2.2.39.

This patch is relatively pain-free. It tries to look up NFSv4 ACLs first (and
if it findes them, maps them back to POSIX acls) and falls back to trying POSIX
ACLs if the filesystem is not NFSv4.

I've written a small additional patch since the patch missed to extend the
acl_extended_file function (which is neat, because e.g. "ls -l" uses to detect
whether additional ACLs are set on a directory entry and displays the very
useful "+" sign after the default Unix permissions).

As you know, the developers are working in intergrating NFSv4 stuff into the
core utilities. Unfortunately this is proceeding very slowly and Gentoo has
been patching NFSv4 stuff around here and there. So it would be nice to see
NFSv4 ACL  support included.

I've attached a minimal proposal for the acl-2.2.39.ebuild which adds a nfsv4
USE flag. It adds a libnfsidmap dependency (I'm not sure whether to use nfsv4
or nonfsv4 like nfs-utils does).

------- Comment #1 From Christophe Saout 2006-08-23 04:37:24 0000 -------
Created an attachment (id=94923) [edit]
The CITI acl-2.2.29-CITI_NFS4_ALL-3.dif minus the debian/changelog reject

------- Comment #2 From Christophe Saout 2006-08-23 04:38:16 0000 -------
Created an attachment (id=94924) [edit]
Let "ls -l" indicate additonal ACLs for NFSv4 too.

------- Comment #3 From Christophe Saout 2006-08-23 04:39:07 0000 -------
Created an attachment (id=94925) [edit]
acl-2.2.39.ebuild patch adding nfsv4 USE flag

------- Comment #4 From SpanKY 2006-08-23 17:57:42 0000 -------
your custom patch should be e-mailed to the nfs list:
nfs@lists.sourceforge.net

------- Comment #5 From Christophe Saout 2006-08-24 04:07:26 0000 -------
Yes, I've contacted nfsv4@linux-nfs.org though which seems more appropriate for
this type of stuff, the other mailing list is more about the kernel anf
nfs-utils.

------- Comment #6 From Christophe Saout 2006-09-01 13:39:31 0000 -------
Note that I sent the patch upstream and got a "thanks". He told me they were
planning better ACL support in general but it would be a good think to keep the
old way (the patches they currently provide). I haven't seen them release a new
official ACL patch though.

http://linux-nfs.org/pipermail/nfsv4/2006-August/004870.html

------- Comment #7 From Christophe Saout 2006-09-02 03:42:20 0000 -------
Update:

The patch has been included in their GIT repository, which also seems to be
more up-to-date than the patches on the CITI homepage:

http://linux-nfs.org/cgi-bin/gitweb.cgi?p=bfields-acl.git;a=summary

------- Comment #8 From SpanKY 2006-09-23 08:41:48 0000 -------
in portage then

First Last Prev Next    No search results available      Search page      Enter new bug