Oh... I accidently uploaded an unrelated diff into this review,

and I don't know of any way to undo that, so I've opened a new one:

https://www.illumos.org/rb/r/518/
7444 fs/xattr.c should be more transparent (zfs_acl_test)
The function xattr_dir_access() was missing some logic

that was there in the earlier version.
Also presented as a github compare:

https://github.com/illumos/illumos-gate/compare/master...gwr:il-7444e
Reviewed by: Yuri Pankov yuri.pankov@nexenta.com

Reviewed by: Dan Fields dan.fields@nexenta.com

Reviewed by: Roman Strashkin roman.strashkin@nexenta.com

Re-ran the test case Yuri described. OK now.