Date: Wed, 16 Mar 2016 21:37:21 -0700 From: Sean Fagan <sef@ixsystems.com> To: Xin LI <delphij@gmail.com> Cc: Alan Somers <asomers@freebsd.org>, Xin LI <delphij@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org> Subject: Re: svn commit: r287445 - in head: bin/setfacl lib/libc/posix1e share/man/man9 sys/cddl/compat/opensolaris/kern sys/cddl/contrib/opensolaris/uts/common/sys sys/kern sys/sys tools/regression/acltools Message-ID: <DF93572A-5BA0-40FA-8792-8E66C39D510E@ixsystems.com> In-Reply-To: <CAGMYy3tA7xPpo4UJT7jaKJgR52CcuB-9%2BRCWvw0bLMukSzCo%2BA@mail.gmail.com> References: <201509040014.t840ELno034811@repo.freebsd.org> <CAOtMX2j8v5hr_R37AOrrTWx=8_=qHyk4MbdhfPr57%2BvRMzW9dw@mail.gmail.com> <CAGMYy3tA7xPpo4UJT7jaKJgR52CcuB-9%2BRCWvw0bLMukSzCo%2BA@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
As I recall, from ZFS, but I=E2=80=99m not positive. Either that, or = something else was using 0x40. Sean. > On Mar 16, 2016, at 9:29 PM, Xin LI <delphij@gmail.com> wrote: >=20 > +sef who created the patch... >=20 > On Wed, Mar 16, 2016 at 3:40 PM, Alan Somers <asomers@freebsd.org> = wrote: >>=20 >>=20 >> On Thu, Sep 3, 2015 at 6:14 PM, Xin LI <delphij@freebsd.org> wrote: >>>=20 >>> Author: delphij >>> Date: Fri Sep 4 00:14:20 2015 >>> New Revision: 287445 >>> URL: https://svnweb.freebsd.org/changeset/base/287445 >>>=20 >>> Log: >>> Expose an interface to determine if an ACE is inherited. >>>=20 >>> Submitted by: sef >>> Reviewed by: trasz >>> MFC after: 2 weeks >>> Differential Revision: https://reviews.freebsd.org/D3540 >>>=20 >>> Modified: >>> head/bin/setfacl/setfacl.1 >>> head/lib/libc/posix1e/acl_add_flag_np.3 >>> head/lib/libc/posix1e/acl_support_nfs4.c >>> head/share/man/man9/acl.9 >>> head/sys/cddl/compat/opensolaris/kern/opensolaris_acl.c >>> head/sys/cddl/contrib/opensolaris/uts/common/sys/acl.h >>> head/sys/kern/subr_acl_nfs4.c >>> head/sys/sys/acl.h >>> head/tools/regression/acltools/tools-crossfs.test >>> head/tools/regression/acltools/tools-nfs4-psarc.test >>> head/tools/regression/acltools/tools-nfs4-trivial.test >>> head/tools/regression/acltools/tools-nfs4.test >>=20 >>=20 >> ... >>=20 >>>=20 >>> Modified: head/sys/sys/acl.h >>>=20 >>> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >>> --- head/sys/sys/acl.h Thu Sep 3 22:15:56 2015 (r287444) >>> +++ head/sys/sys/acl.h Fri Sep 4 00:14:20 2015 (r287445) >>> @@ -249,11 +249,12 @@ typedef void *acl_t; >>> #define ACL_ENTRY_INHERIT_ONLY 0x0008 >>> #define ACL_ENTRY_SUCCESSFUL_ACCESS 0x0010 >>> #define ACL_ENTRY_FAILED_ACCESS 0x0020 >>> +#define ACL_ENTRY_INHERITED 0x0080 >>>=20 >>> #define ACL_FLAGS_BITS = (ACL_ENTRY_FILE_INHERIT | >>> \ >>> ACL_ENTRY_DIRECTORY_INHERIT | ACL_ENTRY_NO_PROPAGATE_INHERIT | \ >>> ACL_ENTRY_INHERIT_ONLY | ACL_ENTRY_SUCCESSFUL_ACCESS | \ >>> - ACL_ENTRY_FAILED_ACCESS) >>> + ACL_ENTRY_FAILED_ACCESS | ACL_ENTRY_INHERITED) >>>=20 >>> /* >>> * Undefined value in ae_id field. ae_id should be set to this = value >>=20 >>=20 >> Out of curiosity, why did you choose 0x0080 for ACL_ENTRY_INHERITED = instead >> of 0x0040 ? >>=20 >=20 >=20 >=20 > --=20 > Xin LI <delphij@delphij.net> https://www.delphij.net/ > FreeBSD - The Power to Serve! Live free or die
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?DF93572A-5BA0-40FA-8792-8E66C39D510E>