[<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