On Friday February 20, trond.myklebust@xxxxxxxxxx wrote: > On Sat, 2009-02-21 at 06:52 +1100, Neil Brown wrote: > > On Friday February 20, trond.myklebust@xxxxxxxxxx wrote: > > > It should work fine. The ctime tracks the mtime in all cases except when > > > you setacl, setfattr, chown, chgrp, chmod, or touch the directory. Those > > > should be very rare operations for pretty much any workload... > > > > > > > Does that mean you'll take the patch ?? > > Not as it stands. You really want to be calling > nfs_force_lookup_revalidate() instead of invalidating the directory > contents. Surely it is 'aswell' rather than 'instead' ?? I need invalid |= NFS_INO_INVALID_DATA; to flush the readdir cache, and nfs_force_lookup_revalidate(inode) to flush the lookup cache ?? I'll respin the patches later today. Thanks, NeilBrown -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
