From: Christoph Hellwig <hch@infradead.org>
To: Mike Christie <mchristi@redhat.com>
Cc: Christoph Hellwig <hch@infradead.org>,
ceph-devel@vger.kernel.org, target-devel@vger.kernel.org
Subject: Re: [PATCH 0/5] target/rbd: distributed lun reset support
Date: Sat, 9 May 2015 07:29:35 -0700 [thread overview]
Message-ID: <20150509142935.GA31964@infradead.org> (raw)
In-Reply-To: <554C51D5.30202@redhat.com>
On Fri, May 08, 2015 at 01:04:05AM -0500, Mike Christie wrote:
> > All of them. It's just another consumer of the in-kernel block devices
> > interface.
>
> Will users use lio with what you are working on?
It's the other way around. The pNFS block device uses a block device
it wants to use persistent reservations on. If we do have a proper
in-kernel API for that it could also work on say RBD and NVMe as long as
they provide a mapping for SCSI3 PRs. For NVMe that exists as a spec,
and for RBD it seems like you're about to deliver one.
> I ask because I am thinking it might be better to implement my own
> se_subsystem_api and sbc_ops structs and in my backend just make libceph
> calls directly for callouts like sbc_ops->execute_rw. So I would not have
> a block_device or request_queue if I went that route.
>
> If I do my own backend, for reset support (PR support will be similar
> with callouts added to the se_subsystem_api) I would just need the patch
> below.
It's defintively simpler, so for prototyping it might be the way to go.
I think in the long run going for a general block device API will be
more useful for everyone involved.
next prev parent reply other threads:[~2015-05-09 14:29 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-05 9:56 [PATCH 0/5] target/rbd: distributed lun reset support mchristi
2015-05-05 9:56 ` [PATCH 1/5] rbd: rename rbd.c to rbd_main.c mchristi
2015-05-05 9:56 ` [PATCH 2/5] target: add cluster device init hooks mchristi
2015-05-05 9:56 ` [PATCH 3/5] target: lun reset cluster hook into support mchristi
2015-05-05 9:56 ` [PATCH 4/5] target: add helper to go from iblock to request queue mchristi
2015-05-05 9:56 ` [PATCH 5/5] rbd: implment lio cluster api module mchristi
2015-05-05 17:49 ` [PATCH 0/5] target/rbd: distributed lun reset support Christoph Hellwig
2015-05-06 4:49 ` Mike Christie
2015-05-06 6:48 ` Christoph Hellwig
2015-05-08 6:04 ` Mike Christie
2015-05-08 7:02 ` Nicholas A. Bellinger
2015-05-09 14:29 ` Christoph Hellwig [this message]
2015-05-06 4:56 ` Mike Christie
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=20150509142935.GA31964@infradead.org \
--to=hch@infradead.org \
--cc=ceph-devel@vger.kernel.org \
--cc=mchristi@redhat.com \
--cc=target-devel@vger.kernel.org \
/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.