aboutsummaryrefslogtreecommitdiff
path: root/sys
diff options
context:
space:
mode:
authorAlexander Motin <mav@FreeBSD.org>2014-09-22 07:59:25 +0000
committerAlexander Motin <mav@FreeBSD.org>2014-09-22 07:59:25 +0000
commit2f872218e723d313dbbe8672788eae694d0d17ad (patch)
treee4ec57b1ae1e700c384c34bfcfbc26e820409873 /sys
parent7c51714e0ad7b5009c80b14e6504a6eef9f639e8 (diff)
downloadsrc-2f872218e723d313dbbe8672788eae694d0d17ad.tar.gz
src-2f872218e723d313dbbe8672788eae694d0d17ad.zip
Simplify legacy reservation handling. Drop it on I_T nexus loss.
Notes
Notes: svn path=/head/; revision=271945
Diffstat (limited to 'sys')
-rw-r--r--sys/cam/ctl/ctl.c44
-rw-r--r--sys/cam/ctl/ctl_private.h2
2 files changed, 19 insertions, 27 deletions
diff --git a/sys/cam/ctl/ctl.c b/sys/cam/ctl/ctl.c
index 160cc642bb4a..66394bcd1b73 100644
--- a/sys/cam/ctl/ctl.c
+++ b/sys/cam/ctl/ctl.c
@@ -5308,12 +5308,14 @@ ctl_scsi_release(struct ctl_scsiio *ctsio)
int length, longid, thirdparty_id, resv_id;
struct ctl_softc *ctl_softc;
struct ctl_lun *lun;
+ uint32_t residx;
length = 0;
resv_id = 0;
CTL_DEBUG_PRINT(("ctl_scsi_release\n"));
+ residx = ctl_get_resindex(&ctsio->io_hdr.nexus);
lun = (struct ctl_lun *)ctsio->io_hdr.ctl_private[CTL_PRIV_LUN].ptr;
ctl_softc = control_softc;
@@ -5371,14 +5373,8 @@ ctl_scsi_release(struct ctl_scsiio *ctsio)
* released, though, by the initiator who made it or by one of
* several reset type events.
*/
- if (lun->flags & CTL_LUN_RESERVED) {
- if ((ctsio->io_hdr.nexus.initid.id == lun->rsv_nexus.initid.id)
- && (ctsio->io_hdr.nexus.targ_port == lun->rsv_nexus.targ_port)
- && (ctsio->io_hdr.nexus.targ_target.id ==
- lun->rsv_nexus.targ_target.id)) {
+ if ((lun->flags & CTL_LUN_RESERVED) && (lun->res_idx == residx))
lun->flags &= ~CTL_LUN_RESERVED;
- }
- }
mtx_unlock(&lun->lun_lock);
@@ -5402,6 +5398,7 @@ ctl_scsi_reserve(struct ctl_scsiio *ctsio)
uint64_t thirdparty_id;
struct ctl_softc *ctl_softc;
struct ctl_lun *lun;
+ uint32_t residx;
extent = 0;
thirdparty = 0;
@@ -5412,6 +5409,7 @@ ctl_scsi_reserve(struct ctl_scsiio *ctsio)
CTL_DEBUG_PRINT(("ctl_reserve\n"));
+ residx = ctl_get_resindex(&ctsio->io_hdr.nexus);
lun = (struct ctl_lun *)ctsio->io_hdr.ctl_private[CTL_PRIV_LUN].ptr;
ctl_softc = control_softc;
@@ -5460,19 +5458,14 @@ ctl_scsi_reserve(struct ctl_scsiio *ctsio)
thirdparty_id = scsi_8btou64(ctsio->kern_data_ptr);
mtx_lock(&lun->lun_lock);
- if (lun->flags & CTL_LUN_RESERVED) {
- if ((ctsio->io_hdr.nexus.initid.id != lun->rsv_nexus.initid.id)
- || (ctsio->io_hdr.nexus.targ_port != lun->rsv_nexus.targ_port)
- || (ctsio->io_hdr.nexus.targ_target.id !=
- lun->rsv_nexus.targ_target.id)) {
- ctsio->scsi_status = SCSI_STATUS_RESERV_CONFLICT;
- ctsio->io_hdr.status = CTL_SCSI_ERROR;
- goto bailout;
- }
+ if ((lun->flags & CTL_LUN_RESERVED) && (lun->res_idx != residx)) {
+ ctsio->scsi_status = SCSI_STATUS_RESERV_CONFLICT;
+ ctsio->io_hdr.status = CTL_SCSI_ERROR;
+ goto bailout;
}
lun->flags |= CTL_LUN_RESERVED;
- lun->rsv_nexus = ctsio->io_hdr.nexus;
+ lun->res_idx = residx;
ctsio->scsi_status = SCSI_STATUS_OK;
ctsio->io_hdr.status = CTL_SUCCESS;
@@ -11260,6 +11253,7 @@ ctl_scsiio_lun_check(struct ctl_softc *ctl_softc, struct ctl_lun *lun,
const struct ctl_cmd_entry *entry, struct ctl_scsiio *ctsio)
{
int retval;
+ uint32_t residx;
retval = 0;
@@ -11284,12 +11278,10 @@ ctl_scsiio_lun_check(struct ctl_softc *ctl_softc, struct ctl_lun *lun,
* even on reserved LUNs, and if this initiator isn't the one who
* reserved us, reject the command with a reservation conflict.
*/
+ residx = ctl_get_resindex(&ctsio->io_hdr.nexus);
if ((lun->flags & CTL_LUN_RESERVED)
&& ((entry->flags & CTL_CMD_FLAG_ALLOW_ON_RESV) == 0)) {
- if ((ctsio->io_hdr.nexus.initid.id != lun->rsv_nexus.initid.id)
- || (ctsio->io_hdr.nexus.targ_port != lun->rsv_nexus.targ_port)
- || (ctsio->io_hdr.nexus.targ_target.id !=
- lun->rsv_nexus.targ_target.id)) {
+ if (lun->res_idx != residx) {
ctsio->scsi_status = SCSI_STATUS_RESERV_CONFLICT;
ctsio->io_hdr.status = CTL_SCSI_ERROR;
retval = 1;
@@ -11297,11 +11289,8 @@ ctl_scsiio_lun_check(struct ctl_softc *ctl_softc, struct ctl_lun *lun,
}
}
- if ( (lun->flags & CTL_LUN_PR_RESERVED)
+ if ((lun->flags & CTL_LUN_PR_RESERVED)
&& ((entry->flags & CTL_CMD_FLAG_ALLOW_ON_PR_RESV) == 0)) {
- uint32_t residx;
-
- residx = ctl_get_resindex(&ctsio->io_hdr.nexus);
/*
* if we aren't registered or it's a res holder type
* reservation and this isn't the res holder then set a
@@ -12167,9 +12156,10 @@ ctl_i_t_nexus_reset(union ctl_io *io)
{
struct ctl_softc *softc = control_softc;
struct ctl_lun *lun;
- uint32_t initindex;
+ uint32_t initindex, residx;
initindex = ctl_get_initindex(&io->io_hdr.nexus);
+ residx = ctl_get_resindex(&io->io_hdr.nexus);
mtx_lock(&softc->ctl_lock);
STAILQ_FOREACH(lun, &softc->lun_list, links) {
mtx_lock(&lun->lun_lock);
@@ -12179,6 +12169,8 @@ ctl_i_t_nexus_reset(union ctl_io *io)
#ifdef CTL_WITH_CA
ctl_clear_mask(lun->have_ca, initindex);
#endif
+ if ((lun->flags & CTL_LUN_RESERVED) && (lun->res_idx == residx))
+ lun->flags &= ~CTL_LUN_RESERVED;
lun->pending_ua[initindex] |= CTL_UA_I_T_NEXUS_LOSS;
mtx_unlock(&lun->lun_lock);
}
diff --git a/sys/cam/ctl/ctl_private.h b/sys/cam/ctl/ctl_private.h
index 39861d38a0c6..43ee3947434b 100644
--- a/sys/cam/ctl/ctl_private.h
+++ b/sys/cam/ctl/ctl_private.h
@@ -389,7 +389,6 @@ struct ctl_lun {
TAILQ_HEAD(ctl_blockq,ctl_io_hdr) blocked_queue;
STAILQ_ENTRY(ctl_lun) links;
STAILQ_ENTRY(ctl_lun) run_links;
- struct ctl_nexus rsv_nexus;
#ifdef CTL_WITH_CA
uint32_t have_ca[CTL_MAX_INITIATORS >> 5];
struct scsi_sense_data pending_sense[CTL_MAX_INITIATORS];
@@ -397,6 +396,7 @@ struct ctl_lun {
ctl_ua_type pending_ua[CTL_MAX_INITIATORS];
struct ctl_mode_pages mode_pages;
struct ctl_lun_io_stats stats;
+ uint32_t res_idx;
struct ctl_per_res_info per_res[2*CTL_MAX_INITIATORS];
unsigned int PRGeneration;
int pr_key_count;