Re: [PATCH v2 1/1] scsi: core: Fix error handler encryption support

[Hannes Reinecke]

On 12/3/25 16:55, Bart Van Assche wrote:
> On 12/2/25 10:42 PM, Hannes Reinecke wrote:
> > There had been an intersection with the reserved command stuff, but
> > now that Bart has dusted things off there I guess I should give it
> > another go.
>
> Does that patch series perhaps involve allocating a reserved command
> from inside the SCSI error handler? Won't that break SCSI LLDs that
> restrict the queue depth to one? I think that the following SCSI LLDs
> only support one command (.can_queue = 1):

reserved commands are only implemented for adapters which require an
LLD specific 'tag' to send TMFs (eg fnic, aacraid, or hpsa).
Most HBAs (especially the older ones) are not that elaborate, and
there TMFs are not commands per se but rather operations on the HBA.

EG fdomain host_reset is just settings some bits in some registers,
no command allocation needed at all.

So for those we don't need to allocate _any_ commands for TMFs,
consequently we don't need to implement reserved commands and
hence TMFs are not guarded by .can_queue at all.

> * drivers/scsi/fdomain.c
> * drivers/scsi/mac53c94.c
> * drivers/scsi/ppa.c
> * drivers/scsi/imm.c
> * drivers/scsi/aha152x.c
Cheers,
Hannes
--
Dr. Hannes Reinecke                  Kernel Storage Architect
hare@xxxxxxxx                               +49 911 74053 688
SUSE Software Solutions GmbH, Frankenstr. 146, 90461 Nürnberg
HRB 36809 (AG Nürnberg), GF: I. Totev, A. McDonald, W. Knoblich