[illumos-Developer] [Ping?]: Code review: 742 Resurrect the ZFS "aclmode" property and 279, 664, 807

Gordon Ross gordon.w.ross at gmail.com
Mon May 9 11:25:54 PDT 2011


Any takers?  These changes are not terribly complicated.

For background, here's the original discussion of this feature:
http://lists.illumos.org/pipermail/developer/2011-February/001543.html

Thanks,
Gordon

---------- Forwarded message ----------
From: Gordon Ross <gordon.w.ross at gmail.com>
Date: Wed, Apr 27, 2011 at 2:29 PM
Subject: Code review: 742 Resurrect the ZFS "aclmode" property and 279, 664, 807
To: illumos-dev <developer at lists.illumos.org>


Hi all, sorry for the delay with this.  We've all been busy.
I'm resuming the integration work for "aclmode".

After some feedback indicating possible problems with
one part of the previous webrev, I've backed out:
 807 Trivial ACEs missing delete

Changes just for that backout can be reviewed here:
 http://www.yalms.org/cr/backout-807/

The updated webrev for the remainder is here:
 http://www.yalms.org/cr/aclmode2/

Comments in the next week or so appreciated.

Thanks,
Gordon

---------- Forwarded message ----------
From: Albert Lee <trisk at opensolaris.org>
Date: Sat, Mar 12, 2011 at 10:14 PM
Subject: [illumos-Developer] Review: 742 Resurrect the ZFS "aclmode"
property and 279, 664, 807
To: Discussions related to development of Illumos <developer at lists.illumos.org>


I'd like to request comments on:
http://pkgdev.openindiana.org/~trisk/aclmode/

279 Bug in the new ACL (post-PSARC/2010/029) semantics
664 Umask masking "deny" ACL entries.
742 Resurrect the ZFS "aclmode" property
807 Trivial ACEs missing delete

For background information, see (this also links to the related bugs):
https://www.illumos.org/issues/742

This reintroduces the "aclmode" ZFS property removed in
PSARC/2010/029. The new "aclmode" only applies to chmod(2) operations,
whereas prior to its removal in PSARC/2010/029 there were poorly
defined interactions with inherited ACEs for newly created files.

279 and 664 were introduced by changes for PSARC/2010/029. 807 affects
interaction with Windows CIFS clients.

-Albert



More information about the Developer mailing list