Re: [PATCH 5/8] NFSv4.1: Move NFSPROC4_CLNT_BIND_CONN_TO_SESSION to the end of the operations

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

 



On May 25, 2012, at 6:03 PM, Trond Myklebust wrote:

> For backward compatibility with nfs-utils.

Can you expand on that a bit? I put them where they are because they are in order of operation id (bind_conn_to_session = 41, exchange_id = 42).

If we really must only add to the end of the list, we really should have a comment above nfs4_procedures, etc.

-dros 

> 
> Signed-off-by: Trond Myklebust <Trond.Myklebust@xxxxxxxxxx>
> Cc: Weston Andros Adamson <dros@xxxxxxxxxx>
> ---
> fs/nfs/nfs4xdr.c     |    4 ++--
> include/linux/nfs4.h |    2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/fs/nfs/nfs4xdr.c b/fs/nfs/nfs4xdr.c
> index a6b95b7..1d4d259 100644
> --- a/fs/nfs/nfs4xdr.c
> +++ b/fs/nfs/nfs4xdr.c
> @@ -7221,8 +7221,6 @@ struct rpc_procinfo	nfs4_procedures[] = {
> 	PROC(RELEASE_LOCKOWNER,	enc_release_lockowner,	dec_release_lockowner),
> 	PROC(SECINFO,		enc_secinfo,		dec_secinfo),
> #if defined(CONFIG_NFS_V4_1)
> -	PROC(BIND_CONN_TO_SESSION,
> -			enc_bind_conn_to_session, dec_bind_conn_to_session),
> 	PROC(EXCHANGE_ID,	enc_exchange_id,	dec_exchange_id),
> 	PROC(CREATE_SESSION,	enc_create_session,	dec_create_session),
> 	PROC(DESTROY_SESSION,	enc_destroy_session,	dec_destroy_session),
> @@ -7237,6 +7235,8 @@ struct rpc_procinfo	nfs4_procedures[] = {
> 	PROC(TEST_STATEID,	enc_test_stateid,	dec_test_stateid),
> 	PROC(FREE_STATEID,	enc_free_stateid,	dec_free_stateid),
> 	PROC(GETDEVICELIST,	enc_getdevicelist,	dec_getdevicelist),
> +	PROC(BIND_CONN_TO_SESSION,
> +			enc_bind_conn_to_session, dec_bind_conn_to_session),
> #endif /* CONFIG_NFS_V4_1 */
> };
> 
> diff --git a/include/linux/nfs4.h b/include/linux/nfs4.h
> index a2b71cb..54006a9 100644
> --- a/include/linux/nfs4.h
> +++ b/include/linux/nfs4.h
> @@ -593,7 +593,6 @@ enum {
> 	NFSPROC4_CLNT_SECINFO,
> 
> 	/* nfs41 */
> -	NFSPROC4_CLNT_BIND_CONN_TO_SESSION,
> 	NFSPROC4_CLNT_EXCHANGE_ID,
> 	NFSPROC4_CLNT_CREATE_SESSION,
> 	NFSPROC4_CLNT_DESTROY_SESSION,
> @@ -608,6 +607,7 @@ enum {
> 	NFSPROC4_CLNT_TEST_STATEID,
> 	NFSPROC4_CLNT_FREE_STATEID,
> 	NFSPROC4_CLNT_GETDEVICELIST,
> +	NFSPROC4_CLNT_BIND_CONN_TO_SESSION,
> };
> 
> /* nfs41 types */
> -- 
> 1.7.7.6
> 

Attachment: smime.p7s
Description: S/MIME cryptographic signature


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux