Messages in this thread | | | Date | Fri, 17 Aug 2018 13:58:12 +0100 | From | Al Viro <> | Subject | Re: Query on shrink list |
| |
On Fri, Aug 17, 2018 at 03:39:22PM +0530, Mukesh Ojha wrote: > Hi Al Viro, > > Is there is reason we have kept data->found++, if the dentry already there > in shrink list ? > > static enum d_walk_ret select_collect( > ... > if (dentry->d_flags & DCACHE_SHRINK_LIST) { > data->found++; > } else { > .. > > If the dentry is already there on shrink list, does it not mean that > data->found is already non-zero ?
Nope. It can be on *another* shrink list - if two processes are doing that...
> Can't we just go out from here directly?
| |