On Sat, Jul 23, 2011 at 8:37 AM, Christoph Hellwig [off-list ref] wrote:

Replace the ->check_acl method with a ->get_acl method that simply reads an
ACL from disk after having a cache miss.  This means we can replace the ACL
checking boilerplate code with a single implementation in namei.c.

Hmm. If we just had a rule for locking (i_mutex?), we could make the
cache update be in namei.c too..

Things that want to use timeouts etc to make for more complicated
permissions rules than the straightforward acl cache have to use
->permission anyway, so it would make sense to make the ->get_acl
method as trivial as possible for filesystems.

But regardless: your patch series looks fine as-is (as does Al's), I'm
just pointing out a possible further step.

So Ack.

                     Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html