On Mon, Feb 26, 2018 at 7:14 PM, Eric W. Biederman
<ebiederm@xxxxxxxxxxxx> wrote:
>
> As such I believe that usage of forget_cached_acl should be subsumed by
> using ACL_NOT_CACHED. If not we should really come up with a different
> helper function name to call from ->get_acl. Preferably one that does
> "cmpxchng(p, sentinel, ACL_NOT_CACHED)" so that we remove the races.
You make your bias very clear, by simply trying to hide the other case.
But for chrissake, that's not the state right now. That other case
exists. You can't - and shouldn't - try to just hide it.
Besides, that "forget_cached_acl()" approach actually has a valid use
case. Maybe you _do_ want to cache ACL's, but with a timeout or
revalidation.
ACL_DONT_CACHE really is a big hammer that makes caching not work at
all. It's not necessarily the right thing to do at all.
Linus
