- nfs-suppress-warnings-about-nfs4err_old_stateid-in-nfs4_handle_exception.patch removed from -mm tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The patch titled
     NFS: suppress warnings about NFS4ERR_OLD_STATEID in nfs4_handle_exception
has been removed from the -mm tree.  Its filename was
     nfs-suppress-warnings-about-nfs4err_old_stateid-in-nfs4_handle_exception.patch

This patch was dropped because it was withdrawn

------------------------------------------------------
Subject: NFS: suppress warnings about NFS4ERR_OLD_STATEID in nfs4_handle_exception
From: Jeff Layton <jlayton@xxxxxxxxxx>

In certain situations, a NFSv4 client can end up with a set of racing calls
that cause the server to return a NFS4ERR_OLD_STATEID error.  When this
occurs, the only real recourse that the client has is to retry the call. 
In the current code, when this occurs, it triggers a useless printk (in
nfs4_map_errors):

    nfs4_map_errors could not handle NFSv4 error 10024

Trond recently mentioned that we really don't need to worry users with that
info.  The following patch is based on a suggestion by Frank Filz and
supresses this error.  Changing the return code here to 0 should be safe
since all callers of nfs4_handle_exception are expected to retry the call
in this situation anyway.

Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxx>
Signed-off-by: Frank Filz <ffilz@xxxxxxxxxx>
Cc: Trond Myklebust <trond.myklebust@xxxxxxxxxx>
Cc: "J. Bruce Fields" <bfields@xxxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---

 fs/nfs/nfs4proc.c |    1 +
 1 files changed, 1 insertion(+)

diff -puN fs/nfs/nfs4proc.c~nfs-suppress-warnings-about-nfs4err_old_stateid-in-nfs4_handle_exception fs/nfs/nfs4proc.c
--- a/fs/nfs/nfs4proc.c~nfs-suppress-warnings-about-nfs4err_old_stateid-in-nfs4_handle_exception
+++ a/fs/nfs/nfs4proc.c
@@ -2774,6 +2774,7 @@ int nfs4_handle_exception(const struct n
 				break;
 		case -NFS4ERR_OLD_STATEID:
 			exception->retry = 1;
+			ret = 0;
 	}
 	/* We failed to handle the error */
 	return nfs4_map_errors(ret);
_

Patches currently in -mm which might be from jlayton@xxxxxxxxxx are

nfs-suppress-warnings-about-nfs4err_old_stateid-in-nfs4_handle_exception.patch
make-iunique-use-a-do-while-loop-rather-than-its-obscure-goto-loop.patch
make-static-counters-in-new_inode-and-iunique-be-32-bits.patch
change-libfs-sb-creation-routines-to-avoid-collisions-with-their-root-inodes.patch
rpc-add-wrapper-for-svc_reserve-to-account-for-checksum.patch
rpc-add-wrapper-for-svc_reserve-to-account-for-checksum-fix.patch

-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Kernel Newbies FAQ]     [Kernel Archive]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Photo]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]

  Powered by Linux