On Mon, Jan 17, 2011 at 08:04:02PM +0300, Vitaliy Gusev wrote:
> diff --git a/fs/nfs/namespace.c b/fs/nfs/namespace.c
> index db6aa36..c5f9162 100644
> --- a/fs/nfs/namespace.c
> +++ b/fs/nfs/namespace.c
> @@ -177,10 +177,16 @@ out_follow:
> const struct inode_operations nfs_mountpoint_inode_operations = {
> .follow_link = nfs_follow_mountpoint,
> .getattr = nfs_getattr,
> +#ifdef CONFIG_NFS_V4
> + .lookup = nfs_atomic_lookup,
> +#endif
> };
>
> const struct inode_operations nfs_referral_inode_operations = {
> .follow_link = nfs_follow_mountpoint,
> +#ifdef CONFIG_NFS_V4
> + .lookup = nfs_atomic_lookup,
> +#endif
> };
That looks very wrong. You are papering over the real problem here - ask
yourself what would that ->lookup() be expected to do?
The thing is, we shouldn't end up trying to call ->lookup() for those.
That's the real issue...
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
