From: <tao.peng-mb1K0bWo544@public.gmane.org> To: torvalds-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org, rdunlap-/UHa2rfvQTnk1uMJSBkQmQ@public.gmane.org Cc: bergwolf-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, Trond.Myklebust-HgOvQuBEEgTQT0dZR+AlfA@public.gmane.org, linux-raid-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Subject: RE: Linux 3.1-rc1 (nfs/pnfs and drivers/md) Date: Wed, 10 Aug 2011 23:26:47 -0400 [thread overview] Message-ID: <F19688880B763E40B28B2B462677FBF805BF0EAC8A@MX09A.corp.emc.com> (raw) In-Reply-To: <CA+55aFzcSLrhkexEn=J7BN8F3P3Vy+EAjwgGwMQPdeoYoQG+Bw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> Hi, Randy and Linus > -----Original Message----- > From: linux-nfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org [mailto:linux-nfs-owner-fy+rA21nqHI@public.gmane.orgrnel.org] > On Behalf Of Linus Torvalds > Sent: Thursday, August 11, 2011 9:10 AM > To: Randy Dunlap > Cc: Peng Tao; Trond Myklebust; linux-raid-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; Linux Kernel Mailing List; > linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > Subject: Re: Linux 3.1-rc1 (nfs/pnfs and drivers/md) > > On Wed, Aug 10, 2011 at 5:58 PM, Randy Dunlap <rdunlap-/UHa2rfvQTnk1uMJSBkQmQ@public.gmane.org> wrote: > > > > The blocklayout parts of NVS_V4_1 could depend on BLOCK. or all of NVS_V4_1 > > can depend on BLOCK. > > Absolutely. Replace "could" by "should". > > There's no way I'll take a patch to make NFS "select BLOCK". If the > block layer isn't enabled, then NFS obviously shouldn't be doing any > pnfs stuff. I see. How about the following patch? We used to do like following for PNFS_BLOCK but it was changed during code review. From 833b59c1d24b43a9a4d47f777abab2cd9d041c68 Mon Sep 17 00:00:00 2001 From: Peng Tao <bergwolf-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Date: Wed, 10 Aug 2011 18:29:21 -0400 Subject: [PATCH] NFS41: make PNFS_BLOCK selectable PNFS_BLOCK needs BLK_DEV_DM/MD, which is not a dependency for other pnfs layout drivers. Seperate it out so others can still build when BLK_DEV_DM/MD is not enabled. Also change select to depends on to avoid build failures. Reported-by: Randy Dunlap <rdunlap-/UHa2rfvQTnk1uMJSBkQmQ@public.gmane.org> Signed-off-by: Peng Tao <peng_tao-mb1K0bWo544@public.gmane.org> --- fs/nfs/Kconfig | 14 +++++++++----- 1 files changed, 9 insertions(+), 5 deletions(-) diff --git a/fs/nfs/Kconfig b/fs/nfs/Kconfig index be02077..0d30613 100644 --- a/fs/nfs/Kconfig +++ b/fs/nfs/Kconfig @@ -79,12 +79,9 @@ config NFS_V4_1 depends on NFS_FS && NFS_V4 && EXPERIMENTAL select SUNRPC_BACKCHANNEL select PNFS_FILE_LAYOUT - select PNFS_BLOCK - select MD - select BLK_DEV_DM help This option enables support for minor version 1 of the NFSv4 protocol - (RFC 5661 and RFC 5663) in the kernel's NFS client. + (RFC 5661) in the kernel's NFS client. If unsure, say N. @@ -92,7 +89,14 @@ config PNFS_FILE_LAYOUT tristate config PNFS_BLOCK - tristate + tristate "Provide support for the pNFS Block Layout Driver for NFSv4.1 pNFS (EXPERIMENTAL)" + depends on NFS_FS && NFS_V4_1 && BLK_DEV_DM + help + Say M here if you want your pNFS client to support the Block Layout Driver + (RFC 5663). Requires Multiple devices driver support (DM) and Device mapper + support (BLK_DEV_DM). + + If unsure, say N. config PNFS_OBJLAYOUT tristate "Provide support for the pNFS Objects Layout Driver for NFSv4.1 pNFS (EXPERIMENTAL)" -- 1.7.4.2 -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: <tao.peng@emc.com> To: <torvalds@linux-foundation.org>, <rdunlap@xenotime.net> Cc: <bergwolf@gmail.com>, <Trond.Myklebust@netapp.com>, <linux-raid@vger.kernel.org>, <linux-kernel@vger.kernel.org>, <linux-nfs@vger.kernel.org> Subject: RE: Linux 3.1-rc1 (nfs/pnfs and drivers/md) Date: Wed, 10 Aug 2011 23:26:47 -0400 [thread overview] Message-ID: <F19688880B763E40B28B2B462677FBF805BF0EAC8A@MX09A.corp.emc.com> (raw) In-Reply-To: <CA+55aFzcSLrhkexEn=J7BN8F3P3Vy+EAjwgGwMQPdeoYoQG+Bw@mail.gmail.com> Hi, Randy and Linus > -----Original Message----- > From: linux-nfs-owner@vger.kernel.org [mailto:linux-nfs-owner@vger.kernel.org] > On Behalf Of Linus Torvalds > Sent: Thursday, August 11, 2011 9:10 AM > To: Randy Dunlap > Cc: Peng Tao; Trond Myklebust; linux-raid@vger.kernel.org; Linux Kernel Mailing List; > linux-nfs@vger.kernel.org > Subject: Re: Linux 3.1-rc1 (nfs/pnfs and drivers/md) > > On Wed, Aug 10, 2011 at 5:58 PM, Randy Dunlap <rdunlap@xenotime.net> wrote: > > > > The blocklayout parts of NVS_V4_1 could depend on BLOCK. or all of NVS_V4_1 > > can depend on BLOCK. > > Absolutely. Replace "could" by "should". > > There's no way I'll take a patch to make NFS "select BLOCK". If the > block layer isn't enabled, then NFS obviously shouldn't be doing any > pnfs stuff. I see. How about the following patch? We used to do like following for PNFS_BLOCK but it was changed during code review. >From 833b59c1d24b43a9a4d47f777abab2cd9d041c68 Mon Sep 17 00:00:00 2001 From: Peng Tao <bergwolf@gmail.com> Date: Wed, 10 Aug 2011 18:29:21 -0400 Subject: [PATCH] NFS41: make PNFS_BLOCK selectable PNFS_BLOCK needs BLK_DEV_DM/MD, which is not a dependency for other pnfs layout drivers. Seperate it out so others can still build when BLK_DEV_DM/MD is not enabled. Also change select to depends on to avoid build failures. Reported-by: Randy Dunlap <rdunlap@xenotime.net> Signed-off-by: Peng Tao <peng_tao@emc.com> --- fs/nfs/Kconfig | 14 +++++++++----- 1 files changed, 9 insertions(+), 5 deletions(-) diff --git a/fs/nfs/Kconfig b/fs/nfs/Kconfig index be02077..0d30613 100644 --- a/fs/nfs/Kconfig +++ b/fs/nfs/Kconfig @@ -79,12 +79,9 @@ config NFS_V4_1 depends on NFS_FS && NFS_V4 && EXPERIMENTAL select SUNRPC_BACKCHANNEL select PNFS_FILE_LAYOUT - select PNFS_BLOCK - select MD - select BLK_DEV_DM help This option enables support for minor version 1 of the NFSv4 protocol - (RFC 5661 and RFC 5663) in the kernel's NFS client. + (RFC 5661) in the kernel's NFS client. If unsure, say N. @@ -92,7 +89,14 @@ config PNFS_FILE_LAYOUT tristate config PNFS_BLOCK - tristate + tristate "Provide support for the pNFS Block Layout Driver for NFSv4.1 pNFS (EXPERIMENTAL)" + depends on NFS_FS && NFS_V4_1 && BLK_DEV_DM + help + Say M here if you want your pNFS client to support the Block Layout Driver + (RFC 5663). Requires Multiple devices driver support (DM) and Device mapper + support (BLK_DEV_DM). + + If unsure, say N. config PNFS_OBJLAYOUT tristate "Provide support for the pNFS Objects Layout Driver for NFSv4.1 pNFS (EXPERIMENTAL)" -- 1.7.4.2
next prev parent reply other threads:[~2011-08-11 3:26 UTC|newest] Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top 2011-08-08 2:25 Linux 3.1-rc1 Linus Torvalds 2011-08-08 20:59 ` Linux 3.1-rc1 (i7core_edac build error) Randy Dunlap 2011-08-08 21:31 ` Borislav Petkov 2011-08-08 21:37 ` Randy Dunlap 2011-08-09 10:01 ` Mauro Carvalho Chehab 2011-08-10 12:43 ` [PATCH] EDAC: Correct Kconfig dependencies Borislav Petkov 2011-08-10 17:14 ` Randy Dunlap 2011-08-11 16:26 ` Borislav Petkov 2011-08-08 21:35 ` Linux 3.1-rc1 (staging/comedi) Randy Dunlap 2011-08-08 21:37 ` Randy Dunlap 2011-08-08 21:51 ` Greg KH 2011-08-08 23:32 ` Randy Dunlap 2011-08-10 18:22 ` [PATCH mainline] xen: xen-selfballoon.c needs more header files Randy Dunlap 2011-08-10 19:20 ` Konrad Rzeszutek Wilk 2011-08-10 19:23 ` Linux 3.1-rc1 (nfs/pnfs and drivers/md) Randy Dunlap 2011-08-11 0:35 ` Peng Tao 2011-08-11 0:35 ` Peng Tao 2011-08-11 0:58 ` Randy Dunlap [not found] ` <20110810175829.a834d80f.rdunlap-/UHa2rfvQTnk1uMJSBkQmQ@public.gmane.org> 2011-08-11 1:10 ` Linus Torvalds 2011-08-11 1:10 ` Linus Torvalds [not found] ` <CA+55aFzcSLrhkexEn=J7BN8F3P3Vy+EAjwgGwMQPdeoYoQG+Bw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2011-08-11 3:26 ` tao.peng-mb1K0bWo544 [this message] 2011-08-11 3:26 ` tao.peng 2011-08-11 6:06 ` Christoph Hellwig [not found] ` <20110811060633.GA11654-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org> 2011-08-11 6:21 ` tao.peng-mb1K0bWo544 2011-08-11 6:21 ` tao.peng [not found] ` <F19688880B763E40B28B2B462677FBF805BF0EAC8A-AYrsSIZi/B2B3McK65YKY9BPR1lH4CV8@public.gmane.org> 2011-08-11 10:11 ` Benny Halevy 2011-08-11 10:11 ` Benny Halevy 2011-08-11 18:44 ` Boaz Harrosh 2011-08-11 18:44 ` Boaz Harrosh [not found] ` <4E4422FD.3060706-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org> 2011-08-11 21:10 ` [PATCH] pnfs: Automatically select blocks & objects layouts Boaz Harrosh 2011-08-11 21:10 ` Boaz Harrosh 2011-08-11 21:29 ` [PATCH v2] " Boaz Harrosh 2011-08-11 21:29 ` Boaz Harrosh 2011-08-11 21:43 ` Randy Dunlap 2011-08-11 21:43 ` Randy Dunlap 2011-08-11 21:53 ` Boaz Harrosh 2011-08-11 21:53 ` Boaz Harrosh [not found] ` <4E444552.9030105-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org> 2011-08-11 22:13 ` [PATCH] " Jim Rees 2011-08-11 22:13 ` Jim Rees [not found] ` <20110811221305.GA8608-8f4Pc2RrbJmHXe+LvDLADg@public.gmane.org> 2011-08-11 23:47 ` Peng Tao 2011-08-11 23:47 ` Peng Tao 2011-08-11 15:41 ` Linux 3.1-rc1 (nfs/pnfs and drivers/md) Randy Dunlap 2011-08-11 15:41 ` Randy Dunlap
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=F19688880B763E40B28B2B462677FBF805BF0EAC8A@MX09A.corp.emc.com \ --to=tao.peng-mb1k0bwo544@public.gmane.org \ --cc=Trond.Myklebust-HgOvQuBEEgTQT0dZR+AlfA@public.gmane.org \ --cc=bergwolf-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-raid-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=rdunlap-/UHa2rfvQTnk1uMJSBkQmQ@public.gmane.org \ --cc=torvalds-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.