From owner-p4-projects@FreeBSD.ORG Sun Apr 5 20:24:28 2009 Return-Path: Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id E17DA1065673; Sun, 5 Apr 2009 20:24:27 +0000 (UTC) Delivered-To: perforce@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 9BA001065670 for ; Sun, 5 Apr 2009 20:24:27 +0000 (UTC) (envelope-from trasz@freebsd.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 6EDBB8FC1D for ; Sun, 5 Apr 2009 20:24:27 +0000 (UTC) (envelope-from trasz@freebsd.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id n35KORKM016668 for ; Sun, 5 Apr 2009 20:24:27 GMT (envelope-from trasz@freebsd.org) Received: (from perforce@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id n35KORa6016666 for perforce@freebsd.org; Sun, 5 Apr 2009 20:24:27 GMT (envelope-from trasz@freebsd.org) Date: Sun, 5 Apr 2009 20:24:27 GMT Message-Id: <200904052024.n35KORa6016666@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: perforce set sender to trasz@freebsd.org using -f From: Edward Tomasz Napierala To: Perforce Change Reviews Cc: Subject: PERFORCE change 160246 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 05 Apr 2009 20:24:30 -0000 http://perforce.freebsd.org/chv.cgi?CH=160246 Change 160246 by trasz@trasz_victim7 on 2009/04/05 20:24:19 Clean up a little. Note that this changes on-disk format. Affected files ... .. //depot/projects/soc2008/trasz_nfs4acl/sys/sys/acl.h#31 edit Differences ... ==== //depot/projects/soc2008/trasz_nfs4acl/sys/sys/acl.h#31 (text+ko) ==== @@ -52,7 +52,7 @@ #define OLDACL_MAX_ENTRIES 32 /* - * With 204 entries, "struct acl" is exactly one page big. + * With 204 entries, "struct acl_t_struct" is almost exactly one page big. * Note that with NFS4 ACLs, the maximum number of ACL entries one * may set on file or directory is about half of ACL_MAX_ENTRIES. * @@ -62,6 +62,8 @@ * The maximum number of POSIX.1e ACLs is controlled * by OLDACL_MAX_ENTRIES. Changing that one will break binary * compatibility with userland and change on-disk ACL layout. + * + * XXX: Recompute. */ #define ACL_MAX_ENTRIES 204 @@ -179,7 +181,8 @@ #define ACL_TYPE_NFS4 0x00000004 /* - * Possible flags in ae_perm field for POSIX.1e ACLs. + * Possible flags in ae_perm field for POSIX.1e ACLs. Note + * that ACL_EXECUTE may be used in both NFSv4 and POSIX.1e ACLs. */ #define ACL_EXECUTE 0x0001 #define ACL_WRITE 0x0002 @@ -190,27 +193,24 @@ /* * Possible flags in ae_perm field for NFSv4 ACLs. - * XXX: Change values of these to match rfc3530. */ -#define ACL_READ_DATA 0x00010000 -#define ACL_LIST_DIRECTORY 0x00010000 -#define ACL_WRITE_DATA 0x00020000 -#define ACL_ADD_FILE 0x00020000 -#define ACL_APPEND_DATA 0x00040000 -#define ACL_ADD_SUBDIRECTORY 0x00040000 -#define ACL_READ_NAMED_ATTRS 0x00080000 -#define ACL_WRITE_NAMED_ATTRS 0x00100000 -#ifdef XXX_conflicting_defines -#define ACL_EXECUTE 0x00200000 -#endif -#define ACL_DELETE_CHILD 0x00400000 -#define ACL_READ_ATTRIBUTES 0x00800000 -#define ACL_WRITE_ATTRIBUTES 0x01000000 -#define ACL_DELETE 0x02000000 -#define ACL_READ_ACL 0x04000000 -#define ACL_WRITE_ACL 0x08000000 -#define ACL_WRITE_OWNER 0x10000000 -#define ACL_SYNCHRONIZE 0x20000000 +#define ACL_READ_DATA 0x00000020 +#define ACL_LIST_DIRECTORY 0x00000020 +#define ACL_WRITE_DATA 0x00000040 +#define ACL_ADD_FILE 0x00000040 +#define ACL_APPEND_DATA 0x00000080 +#define ACL_ADD_SUBDIRECTORY 0x00000080 +#define ACL_READ_NAMED_ATTRS 0x00000100 +#define ACL_WRITE_NAMED_ATTRS 0x00000200 +/* ACL_EXECUTE is defined above. */ +#define ACL_DELETE_CHILD 0x00000400 +#define ACL_READ_ATTRIBUTES 0x00000800 +#define ACL_WRITE_ATTRIBUTES 0x00001000 +#define ACL_DELETE 0x00002000 +#define ACL_READ_ACL 0x00004000 +#define ACL_WRITE_ACL 0x00008000 +#define ACL_WRITE_OWNER 0x00010000 +#define ACL_SYNCHRONIZE 0x00020000 #define ACL_NFS4_PERM_BITS (ACL_READ_DATA | ACL_WRITE_DATA | \ ACL_APPEND_DATA | ACL_READ_NAMED_ATTRS | ACL_WRITE_NAMED_ATTRS | \