[LTP] [PATCH] fanotify10: Calling drop_cache three times to ensure the inode is evicted

Petr Vorel pvorel@suse.cz
Fri Oct 18 19:18:02 CEST 2024


Hi Jan, Zizhi Wo,

...
> > NUMA0: parent inode	exist		exist		free
> > NUMA1: parent dentry	exist		free		free
> > NUMA2: child inode	exist		free		free
> > NUMA3: child dentry	free		free		free

> Well, this is right but there's also the while ((freed >> shift++) > 1)
> loop in drop_slab() which should generally make us loop as long as there's
> something to reclaim. But yes, if in theory the only thing we can reclaim
> is the child dentry in the first round, then what you suggest may happen.

> > Due to the release of the dependency chain, the drop_cache cleanup also
> > takes several times. Therefore, to be safe, three drop_cache operations are
> > needed to handle the two-level directory structure.

> OK, I'm willing to give this one last try. If it doesn't work out, I'd just
> drop these tests until we can find a more reliable way of testing this.

> Feel free to add:

> Reviewed-by: Jan Kara <jack@suse.cz>

Thanks you both for yet another fix and review, both is very much appreciated.
Hopefully this will finally work.

Kind regards,
Petr

> 								Honza
...


More information about the ltp mailing list