linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Boaz Harrosh <bharrosh@panasas.com>
To: Benny Halevy <bhalevy@tonian.com>,
	Sorin Faibish <sfaibish@emc.com>,
	peter honeyman <honey@citi.umich.edu>
Cc: <tao.peng@emc.com>, <torvalds@linux-foundation.org>,
	<rdunlap@xenotime.net>, <bergwolf@gmail.com>,
	<Trond.Myklebust@netapp.com>, <linux-raid@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>, <linux-nfs@vger.kernel.org>,
	"Welch, Brent" <welch@panasas.com>
Subject: [PATCH] pnfs: Automatically select blocks & objects layouts
Date: Thu, 11 Aug 2011 14:10:42 -0700	[thread overview]
Message-ID: <4E444552.9030105@panasas.com> (raw)
In-Reply-To: <4E4422FD.3060706@panasas.com>

What about the below? based on v3.1-rc1

----
From: Boaz Harrosh <bharrosh@panasas.com>
Subject: [PATCH] pnfs: Automatically select blocks & objects layouts

Just like files-layout, blocks & objects layouts are part of the
NFS 4.1 protocol and should be automatically selected if NFS_4_1
is selected. The small problem is that these depend on other
Kernel support being present, while files only depends on NFS
itself.

This patch removes from the user choice the presence of objects
and blocks layout. But makes sure these are selected only if
the depended subsystems are present in the Kernel.

Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
---
 fs/nfs/Kconfig |   14 ++++----------
 1 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/fs/nfs/Kconfig b/fs/nfs/Kconfig
index be02077..1b951cd 100644
--- a/fs/nfs/Kconfig
+++ b/fs/nfs/Kconfig
@@ -79,9 +79,6 @@ 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.
@@ -93,16 +90,13 @@ config PNFS_FILE_LAYOUT
 
 config PNFS_BLOCK
 	tristate
+	depends on NFS_FS && NFS_V4_1 && MD && BLK_DEV_DM
+	default m
 
 config PNFS_OBJLAYOUT
-	tristate "Provide support for the pNFS Objects Layout Driver for NFSv4.1 pNFS (EXPERIMENTAL)"
+	tristate
 	depends on NFS_FS && NFS_V4_1 && SCSI_OSD_ULD
-	help
-	  Say M here if you want your pNFS client to support the Objects Layout Driver.
-	  Requires the SCSI osd initiator library (SCSI_OSD_INITIATOR) and
-	  upper level driver (SCSI_OSD_ULD).
-
-	  If unsure, say N.
+	default m
 
 config ROOT_NFS
 	bool "Root file system on NFS"
-- 
1.7.6



  reply	other threads:[~2011-08-11 21:11 UTC|newest]

Thread overview: 30+ 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:58     ` Randy Dunlap
2011-08-11  1:10       ` Linus Torvalds
2011-08-11  3:26         ` tao.peng
2011-08-11  6:06           ` Christoph Hellwig
2011-08-11  6:21             ` tao.peng
2011-08-11 10:11           ` Benny Halevy
2011-08-11 18:44             ` Boaz Harrosh
2011-08-11 21:10               ` Boaz Harrosh [this message]
2011-08-11 21:29                 ` [PATCH v2] pnfs: Automatically select blocks & objects layouts Boaz Harrosh
2011-08-11 21:43                   ` Randy Dunlap
2011-08-11 21:53                     ` Boaz Harrosh
2011-08-11 22:13                 ` [PATCH] " Jim Rees
2011-08-11 23:47                   ` Peng Tao
2011-08-11 15:41           ` Linux 3.1-rc1 (nfs/pnfs and drivers/md) 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=4E444552.9030105@panasas.com \
    --to=bharrosh@panasas.com \
    --cc=Trond.Myklebust@netapp.com \
    --cc=bergwolf@gmail.com \
    --cc=bhalevy@tonian.com \
    --cc=honey@citi.umich.edu \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-nfs@vger.kernel.org \
    --cc=linux-raid@vger.kernel.org \
    --cc=rdunlap@xenotime.net \
    --cc=sfaibish@emc.com \
    --cc=tao.peng@emc.com \
    --cc=torvalds@linux-foundation.org \
    --cc=welch@panasas.com \
    /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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).