diff options
| author | Nicholas Bellinger <nab@linux-iscsi.org> | 2012-03-14 08:29:06 +0400 |
|---|---|---|
| committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2012-03-14 08:43:43 +0400 |
| commit | 087a03b3ea1c8d6e2d5743a8d1c6d571058caa04 (patch) | |
| tree | 52cb057d9c2f01f8772de66f1cce7086c9e8b7c5 /drivers/target/iscsi/iscsi_target.c | |
| parent | 67236c44741e250199ccd77f1115568e68cf8848 (diff) | |
| download | linux-087a03b3ea1c8d6e2d5743a8d1c6d571058caa04.tar.xz | |
target: Fix compatible reservation handling (CRH=1) with legacy RESERVE/RELEASE
This patch addresses a bug with target_check_scsi2_reservation_conflict()
return checking in target_scsi2_reservation_[reserve,release]() that was
preventing CRH=1 operation from silently succeeding in the two special
cases defined by SPC-3, and not failing with reservation conflict status
when dealing with legacy RESERVE/RELEASE + active SPC-3 PR logic.
Also explictly set cmd->scsi_status = SAM_STAT_RESERVATION_CONFLICT during
the early non reservation holder failure from pr_ops->t10_seq_non_holder()
check in transport_generic_cmd_sequencer() for fabrics that already expect
it to be set.
This bug was originally introduced in mainline commit:
commit eacac00ce5bfde8086cd0615fb53c986f7f970fe
Author: Christoph Hellwig <hch@infradead.org>
Date: Thu Nov 3 17:50:40 2011 -0400
target: split core_scsi2_emulate_crh
Reported-by: Martin Svec <martin.svec@zoner.cz>
Cc: Martin Svec <martin.svec@zoner.cz>
Cc: Christoph Hellwig <hch@lst.de>
Cc: stable@vger.kernel.org
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'drivers/target/iscsi/iscsi_target.c')
0 files changed, 0 insertions, 0 deletions