On 22/02/13 09:41, Myklebust, Trond wrote:
> On Fri, 2013-02-22 at 09:34 -0500, Steve Dickson wrote:
>> This enable NFSv4.2 support. To enable this code the
>> CONFIG_NFS_V4_2 Kconfig define needs to be set and
>> the -o v4.2 mount option need to be used.
>>
>> Signed-off-by: Steve Dickson <steved@xxxxxxxxxx>
>> ---
>> fs/nfs/Kconfig | 12 +++++++++++-
>> fs/nfs/callback.c | 3 +++
>> fs/nfs/nfs4client.c | 5 +++++
>> fs/nfs/nfs4proc.c | 3 +++
>> fs/nfs/super.c | 7 ++++++-
>> include/linux/nfs4.h | 4 ++++
>> 6 files changed, 32 insertions(+), 2 deletions(-)
>>
>> diff --git a/fs/nfs/Kconfig b/fs/nfs/Kconfig
>> index 3861a1f..4190710 100644
>> --- a/fs/nfs/Kconfig
>> +++ b/fs/nfs/Kconfig
>> @@ -104,6 +104,16 @@ config NFS_V4_1
>>
>> If unsure, say N.
>>
>> +config NFS_V4_2
>> + bool "NFS client support for NFSv4.2"
>> + depends on NFS_V4_1
>> + select NFS_V4_SECURITY_LABEL
>> + help
>> + This option enables support for minor version 2 of the NFSv4 protocol
>> + in the kernel's NFS client.
>
> + Say Y here if you want enable fine-grained security label attributes
Ok...
>
>> +
>> + If unsure, say N.
>> +
>> config PNFS_FILE_LAYOUT
>> tristate
>> depends on NFS_V4_1
>> @@ -133,7 +143,7 @@ config NFS_V4_1_IMPLEMENTATION_ID_DOMAIN
>>
>> config NFS_V4_SECURITY_LABEL
>> bool "Provide Security Label support for NFSv4 client"
>> - depends on NFS_V4 && SECURITY
>> + depends on NFS_V4_2 && SECURITY
>> help
>
> You don't need the above select at all above if you change this to
>
> config NFS_V4_SECURITY_LABEL
> bool
> depends on NFS_V4_2 && SECURITY
> default y
Fine...
>
>>
>> diff --git a/fs/nfs/callback.c b/fs/nfs/callback.c
>> index 5088b57..4058ec8 100644
>> --- a/fs/nfs/callback.c
>> +++ b/fs/nfs/callback.c
>> @@ -281,6 +281,9 @@ static int nfs_callback_up_net(int minorversion, struct svc_serv *serv, struct n
>> case 1:
>> ret = nfs41_callback_up_net(serv, net);
>> break;
>> + case 2:
>
>
> Since this is identical to case 1:, they should be merged.
I guess wrong on how you wanted this handled... ;-)
steved.
>
>> + ret = nfs41_callback_up_net(serv, net);
>> + break;
>> default:
>> printk(KERN_ERR "NFS: unknown callback version: %d\n",
>> minorversion);
>> diff --git a/fs/nfs/nfs4client.c b/fs/nfs/nfs4client.c
>> index 2e9779b..2987fd6 100644
>> --- a/fs/nfs/nfs4client.c
>> +++ b/fs/nfs/nfs4client.c
>> @@ -66,6 +66,11 @@ struct nfs_client *nfs4_alloc_client(const struct nfs_client_initdata *cl_init)
>> if (err)
>> goto error;
>>
>> + if (cl_init->minorversion > NFS4_MAX_MINOR_VERSION) {
>> + err = -EINVAL;
>> + goto error;
>> + }
>> +
>> spin_lock_init(&clp->cl_lock);
>> INIT_DELAYED_WORK(&clp->cl_renewd, nfs4_renew_state);
>> rpc_init_wait_queue(&clp->cl_rpcwaitq, "NFS client");
>> diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c
>> index 30b3e44..2384f91 100644
>> --- a/fs/nfs/nfs4proc.c
>> +++ b/fs/nfs/nfs4proc.c
>> @@ -7136,6 +7136,9 @@ const struct nfs4_minor_version_ops *nfs_v4_minor_ops[] = {
>> #if defined(CONFIG_NFS_V4_1)
>> [1] = &nfs_v4_1_minor_ops,
>> #endif
>> +#if defined(CONFIG_NFS_V4_2)
>> + [2] = &nfs_v4_1_minor_ops,
>> +#endif
>> };
>>
>> const struct inode_operations nfs4_dir_inode_operations = {
>> diff --git a/fs/nfs/super.c b/fs/nfs/super.c
>> index 4e78f93..d35582c 100644
>> --- a/fs/nfs/super.c
>> +++ b/fs/nfs/super.c
>> @@ -269,7 +269,7 @@ static match_table_t nfs_local_lock_tokens = {
>>
>> enum {
>> Opt_vers_2, Opt_vers_3, Opt_vers_4, Opt_vers_4_0,
>> - Opt_vers_4_1,
>> + Opt_vers_4_1, Opt_vers_4_2,
>>
>> Opt_vers_err
>> };
>> @@ -280,6 +280,7 @@ static match_table_t nfs_vers_tokens = {
>> { Opt_vers_4, "4" },
>> { Opt_vers_4_0, "4.0" },
>> { Opt_vers_4_1, "4.1" },
>> + { Opt_vers_4_2, "4.2" },
>>
>> { Opt_vers_err, NULL }
>> };
>> @@ -1143,6 +1144,10 @@ static int nfs_parse_version_string(char *string,
>> mnt->version = 4;
>> mnt->minorversion = 1;
>> break;
>> + case Opt_vers_4_2:
>> + mnt->version = 4;
>> + mnt->minorversion = 2;
>> + break;
>> default:
>> return 0;
>> }
>> diff --git a/include/linux/nfs4.h b/include/linux/nfs4.h
>> index aab8bd8..e9c040a 100644
>> --- a/include/linux/nfs4.h
>> +++ b/include/linux/nfs4.h
>> @@ -394,11 +394,15 @@ enum lock_type4 {
>> #define NFS4_VERSION 4
>> #define NFS4_MINOR_VERSION 0
>>
>> +#if defined(CONFIG_NFS_V4_2)
>> +#define NFS4_MAX_MINOR_VERSION 2
>> +#else
>> #if defined(CONFIG_NFS_V4_1)
>> #define NFS4_MAX_MINOR_VERSION 1
>> #else
>> #define NFS4_MAX_MINOR_VERSION 0
>> #endif /* CONFIG_NFS_V4_1 */
>> +#endif /* CONFIG_NFS_V4_2 */
>>
>> #define NFS4_DEBUG 1
>>
>
--
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
