On Mon, Oct 16, 2023 at 11:24:25AM +0200, Hannes Reinecke wrote: > + req = scsi_alloc_request(sdev->request_queue, REQ_OP_DRV_IN, > + BLK_MQ_REQ_NOWAIT); > + if (!req) { > + SNIC_HOST_ERR(snic->shost, > + "Devrst: TMF busy\n"); > + goto dev_rst_end; So if we fail this allocation, which can easily happen, we just fail the reset and escalate. If that's fine we probably want a big fat comment about that here.