d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Simplify legacy reservation handling. Drop it on I_T nexus loss.

Browse Source
This commit is contained in:
mav 2014-09-22 07:59:25 +00:00
parent 3118b08dd6
commit 724af6b5c3
2 changed files with 19 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
sys/cam/ctl/ctl.c
View File

@ -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);
}

2
sys/cam/ctl/ctl_private.h
View File

@ -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;