From: "J. Bruce Fields" <bfields@fieldses.org>
To: Christoph Hellwig <hch@lst.de>
Cc: trond.myklebust@primarydata.com, linux-nfs@vger.kernel.org
Subject: Re: pNFS SCSI layout support
Date: Tue, 1 Mar 2016 15:49:27 -0500 [thread overview]
Message-ID: <20160301204927.GE23792@fieldses.org> (raw)
In-Reply-To: <20160301203012.GA6033@lst.de>
On Tue, Mar 01, 2016 at 09:30:12PM +0100, Christoph Hellwig wrote:
> On Tue, Mar 01, 2016 at 02:24:08PM -0500, J. Bruce Fields wrote:
> > I still wonder whether this is the best behavior. How about also adding
> > the ability to configure out the block layout? I think we'd want to be
> > sure it's completely off in production.
>
> Being able to configure block layout eventually might be useful. Do
> you simply want separate compile time options? It's not like a whole
> lot of code will go away, it's mostly just entry points.
Yeah, I wasn't thinking about the size of the code so much as
supportability.
> > Also todo?:
> > - wireshark support
>
> I've mostly stayed clear of wiresharė.
No problem, just noting it so I don't forget.
> > - testing, including of fencing and reboot recovery.
>
> I've done a fair amount of testing, and the interesting part is
> indeed fencing. The normal path is not very different from the block
> layout driver.
>
> > (And:
> > currently I just share a file between two vm's to use as my
> > shared block device, I guess I'll need to set up a real SCSI
> > target instead.)
>
> The in-kernel lio target will work just fine. You can actually export
> that directly to VMs using the vhost_scsi driver, but I've not tried
> that option myself yet.
>
> > I think there's not any additional documentation required--this is just
> > like block layout but without the need to configure fencing.
>
> Exactly.
>
> > (Or do
> > people need to do some additional configuration of the SCSI devices? Or
> > check the specs on their hardware to make sure it has support for the
> > right features?)
>
> Yes, the device needs to support persistent reservations.
>
> I can write up a little blurb on that.
OK, thanks.
--b.
prev parent reply other threads:[~2016-03-01 20:49 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-29 13:24 pNFS SCSI layout support Christoph Hellwig
2016-02-29 13:24 ` [PATCH 1/3] nfs4.h: add SCSI layout defintions Christoph Hellwig
2016-02-29 13:24 ` [PATCH 2/3] nfs/blocklayout: add SCSI layout support Christoph Hellwig
2016-02-29 23:40 ` kbuild test robot
2016-03-01 8:11 ` kbuild test robot
2016-02-29 13:24 ` [PATCH 3/3] nfsd: " Christoph Hellwig
2016-03-01 19:24 ` pNFS " J. Bruce Fields
2016-03-01 20:30 ` Christoph Hellwig
2016-03-01 20:49 ` J. Bruce Fields [this message]
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=20160301204927.GE23792@fieldses.org \
--to=bfields@fieldses.org \
--cc=hch@lst.de \
--cc=linux-nfs@vger.kernel.org \
--cc=trond.myklebust@primarydata.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 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.