[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CA+a=Yy5dgW99H8pTokUZDTJaS_uDS_vKfUeR+i8QNcCwtdFCAQ@mail.gmail.com>
Date:	Fri, 12 Aug 2011 07:47:20 +0800
From:	Peng Tao <bergwolf@...il.com>
To:	Jim Rees <rees@...ch.edu>, Boaz Harrosh <bharrosh@...asas.com>
Cc:	Benny Halevy <bhalevy@...ian.com>,
	Sorin Faibish <sfaibish@....com>,
	peter honeyman <honey@...i.umich.edu>, tao.peng@....com,
	torvalds@...ux-foundation.org, rdunlap@...otime.net,
	Trond.Myklebust@...app.com, linux-raid@...r.kernel.org,
	linux-kernel@...r.kernel.org, linux-nfs@...r.kernel.org,
	"Welch, Brent" <welch@...asas.com>
Subject: Re: [PATCH] pnfs: Automatically select blocks & objects layouts
On Fri, Aug 12, 2011 at 6:13 AM, Jim Rees <rees@...ch.edu> wrote:
> Boaz Harrosh wrote:
>
>  What about the below? based on v3.1-rc1
>
>  ----
>  From: Boaz Harrosh <bharrosh@...asas.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@...asas.com>
>
> Acked-by: Jim Rees <rees@...ch.edu>
>
> I think this is what we want.  There is no point allowing the user to pick
> and choose various layout drivers.  If they have selected nfs4.1 they should
> get all the layout drivers their kernel will support, based on the
> dependencies.
>
I agree automatically selecting is more reasonable.
Acked-by: Peng Tao <peng_tao@....com>
-- 
Thanks,
-Bergwolf
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
Powered by blists - more mailing lists