linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Hannes Reinecke <hare@suse.de>
To: David Disseldorp <ddiss@suse.de>, Christoph Hellwig <hch@lst.de>
Cc: linux-scsi@vger.kernel.org
Subject: Re: Persistent Reservation API
Date: Wed, 26 Aug 2015 14:45:59 +0200	[thread overview]
Message-ID: <55DDB507.8000403@suse.de> (raw)
In-Reply-To: <20150811113041.68d9f122@g21.suse.de>

On 08/11/2015 11:30 AM, David Disseldorp wrote:
> Hi Christoph,
> 
> On Tue,  4 Aug 2015 09:11:05 +0200, Christoph Hellwig wrote:
> 
>> This series adds support for a simplified persistent reservation API
>> to the block layer.  The intent is that both in-kernel and userspace
>> consumers can use the API instead of having to hand craft SCSI or NVMe
>> command through the various pass through interfaces.  It also adds
>> DM support as getting reservations through dm-multipath is a major
>> pain with the current scheme.
>>
>> NVMe support currently isn't included as I don't have a multihost
>> NVMe setup to test on, but if I can find a volunteer to test it I'm
>> happy to write the code for it.
>>
>> The ioctl API is documented in Documentation/block/pr.txt, but to
>> fully understand the concept you'll have to read up the SPC spec,
>> PRs are too complicated that trying to rephrase them into different
>> terminology is just going to create confusion.
> 
> Do you have any thoughts on where SCSI-2 RESERVE/RELEASE should fit into
> this API, if at all? I.e. as a new enum pr_type members for
> pr_reserve()/pr_release(), as separate pr_ops hooks, etc?
> Similarly for PR_IN - IIUC, if LIO is to handle cluster wide PRs
> for Ceph rbd via the block layer, these will all need to be supported
> by block layer (and LIO backend) APIs.
> 
Just don't. Ignore SCSI-2 RESERVE/RELEASE.

To my knowledge there are _no_ 'real' SCSI-2 systems in the market
anymore; those which you might come across are 'fake' devices,
trying to emulate SCSI-2 for backwards compability despite being in
fact SCSI-3 devices.

I would vote for just ignoring them. Simply not worth the pain.

Cheers,

Hannes
-- 
Dr. Hannes Reinecke		               zSeries & Storage
hare@suse.de			               +49 911 74053 688
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton
HRB 21284 (AG Nürnberg)
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

      parent reply	other threads:[~2015-08-26 12:46 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-04  7:11 Persistent Reservation API Christoph Hellwig
2015-08-04  7:11 ` [PATCH 1/6] block: cleanup blkdev_ioctl Christoph Hellwig
     [not found]   ` <1438672271-11309-2-git-send-email-hch-jcswGhMUV9g@public.gmane.org>
2015-08-12  0:29     ` Martin K. Petersen
2015-08-04  7:11 ` [PATCH 2/6] block: add a API for persistent reservations Christoph Hellwig
     [not found]   ` <1438672271-11309-3-git-send-email-hch-jcswGhMUV9g@public.gmane.org>
2015-08-12  0:45     ` Martin K. Petersen
2015-08-04  7:11 ` [PATCH 3/6] sd: implement the persisten reservation API Christoph Hellwig
     [not found]   ` <1438672271-11309-4-git-send-email-hch-jcswGhMUV9g@public.gmane.org>
2015-08-12  0:57     ` Martin K. Petersen
2015-08-04  7:11 ` [PATCH 4/6] dm: refactor ioctl handling Christoph Hellwig
     [not found]   ` <1438672271-11309-5-git-send-email-hch-jcswGhMUV9g@public.gmane.org>
2015-08-04 17:56     ` Mike Snitzer
2015-08-06 13:45       ` Christoph Hellwig
2015-08-04  7:11 ` [PATCH 5/6] dm: split out a helper to find the ioctl target Christoph Hellwig
     [not found]   ` <1438672271-11309-6-git-send-email-hch-jcswGhMUV9g@public.gmane.org>
2015-08-04 18:05     ` Mike Snitzer
     [not found]       ` <20150804180558.GB28971-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-08-06 13:45         ` Christoph Hellwig
2015-08-04  7:11 ` [PATCH 6/6] dm: add support for passing through persistent reservations Christoph Hellwig
2015-08-04 17:53 ` Persistent Reservation API Mike Snitzer
2015-08-06 13:44   ` Christoph Hellwig
2015-08-04 18:04 ` Keith Busch
2015-08-06 13:45   ` Christoph Hellwig
2015-08-11  9:30 ` David Disseldorp
2015-08-11 12:26   ` Christoph Hellwig
2015-08-11 12:52     ` David Disseldorp
2015-08-26 12:45   ` Hannes Reinecke [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=55DDB507.8000403@suse.de \
    --to=hare@suse.de \
    --cc=ddiss@suse.de \
    --cc=hch@lst.de \
    --cc=linux-scsi@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).