On Wednesday, June 21, 2017 3:18:20 PM IST Miklos Szeredi wrote: > On Wed, Jun 21, 2017 at 11:36 AM, Amir Goldstein <amir73il@xxxxxxxxx> wrote: > > On Wed, Jun 21, 2017 at 12:20 PM, Miklos Szeredi <miklos@xxxxxxxxxx> wrote: > >> On Wed, Jun 21, 2017 at 11:05 AM, Amir Goldstein <amir73il@xxxxxxxxx> wrote: > > > >>> > >>> Following up on your idea: > >>> - check in ovl_iterate() if version has changed and if dir became impure > >>> - if it did, populate od->cache, but keep the dir od->is_real > >>> - iterate upper cache entries and call ovl_cache_update_ino() > >>> - Then actor of real dir iterator can use the cache to ommit entries or use > >>> p->ino from cache if p->real_ino match real d_ino, but differs from p->ino. > >> > >> For non-merge dirs we can have a simplified cache just containing the > >> entries with origin, recreated when the version changes or updated in > >> rename, whichever is simpler. A non-merge dir will never become a > >> merge one, so we can keep the handling separate. > >> > > > > And use this cache to ommit/fix entries with actor? > > Right. I think fixing up is better, because to correctly omit entries > we'd need separate lists for each open directory. For fixing up we > can use a common one, just like for the merged dir. > Amir, Can I please take over the implementation of this feature. -- chandan -- To unsubscribe from this list: send the line "unsubscribe linux-unionfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html