xfs
[Top] [All Lists]

Re: [PATCH 14/18] nfsd: pNFS block layout driver

To: Christoph Hellwig <hch@xxxxxx>
Subject: Re: [PATCH 14/18] nfsd: pNFS block layout driver
From: Jeff Layton <jeff.layton@xxxxxxxxxxxxxxx>
Date: Thu, 8 Jan 2015 12:41:31 -0800
Cc: Jeff Layton <jeff.layton@xxxxxxxxxxxxxxx>, "J. Bruce Fields" <bfields@xxxxxxxxxxxx>, linux-nfs@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20150107102802.GA28783@xxxxxx>
References: <1420561721-9150-1-git-send-email-hch@xxxxxx> <1420561721-9150-15-git-send-email-hch@xxxxxx> <20150106171658.GD12067@xxxxxxxxxxxx> <20150106173957.GA16200@xxxxxx> <20150106193949.GD28003@xxxxxxxxxxxx> <20150106114205.4151269c@xxxxxxxxxxxxxxxxxxxxxxxxx> <20150107102802.GA28783@xxxxxx>
On Wed, 7 Jan 2015 11:28:02 +0100
Christoph Hellwig <hch@xxxxxx> wrote:

> On Tue, Jan 06, 2015 at 11:42:05AM -0800, Jeff Layton wrote:
> > Or maybe an "on" switch?
> > 
> > We have some patches (not posted currently) that add a "pnfs" export
> > option. Maybe we should add that and only enable pnfs on exports that
> > have that option present?
> 
> I would defintively prefer the off switch.  I can add one if people want
> it, but export options are a little annoying as they require support
> not only in the kernel but also in nfs-utils.

True, it is a pain, but I think it's realistic to expect someone who
wants to do pnfs to have an updated nfs-utils. It wouldn't take too
long for it to trickle out to the various distros and adding new export
options is fairly simple to do.

If we do want to go that route, it might be nice to do the option with
a list of layout types. For example:

    pnfs=block:file:flexfiles

...so we could potentially support more than one layout type per
export.

-- 
Jeff Layton <jlayton@xxxxxxxxxxxxxxx>

<Prev in Thread] Current Thread [Next in Thread>