This patchset is a respin of the second pile of cleanups for the cifsacl code. Most of the changes to the already-sent patches are pretty minor cleanups and such. The last two patches in the series are new. The first adds a timeout to the idmap keys so that they can eventually work their way out of the cache. The second fixes an existing bug that caused "ls -l" to show the ownership wrong when cifsacl is used. With this set, everything seems to work as expected. Steve, could you pull these into your for-next branch soon so we can get them some testing exposure? Jeff Layton (9): cifs: make cifs_copy_sid handle a source sid with variable size subauth arrays cifs: redefine NUM_SUBAUTH constant from 5 to 15 cifs: fix the format specifiers in sid_to_str cifs: remove uneeded __KERNEL__ block from cifsacl.h cifs: simplify id_to_sid and sid_to_id mapping code cifs: avoid extra allocation for small cifs.idmap keys cifs: don't override the uid/gid in getattr when cifsacl is enabled cifs: set a timeout on keys in cifs.idmap cache cifs: ensure we revalidate the inode after readdir if cifsacl is enabled fs/cifs/cifsacl.c | 573 ++++++++++++---------------------------------------- fs/cifs/cifsacl.h | 49 ++--- fs/cifs/cifsfs.c | 1 - fs/cifs/cifsproto.h | 1 - fs/cifs/inode.c | 7 +- fs/cifs/readdir.c | 10 + 6 files changed, 161 insertions(+), 480 deletions(-) -- 1.7.11.7 -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html