2010/11/13 Jeff Layton <jlayton@xxxxxxxxx>: > > I don't see why the above is necessary. cifs_revalidate_dentry should > have taken care of invalidating the mapping when it revalidated the > dentry. > > Also, you're leaving invalid_mapping set so you're almost certainly > going to end up invalidating the mapping unnecessarily again. > > Finally, you're not considering the fscache case here. I think you need > to look closely at how dentry/inode revalidation is handled -- in > particular cifs_revalidate_dentry and cifs_invalidate_mapping. > Jeff, you are right about leaving invalid_mapping set - I noticed it after some time I posted this patch. So - I reposted it in the list again. Let's consider this version as invalid. -- Best regards, Pavel Shilovsky. -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
