On 2/9/21 9:59 AM, Bodo Stroesser wrote: > On 09.02.21 13:38, Mike Christie wrote: >> @@ -179,6 +184,11 @@ static void tcm_loop_submission_work(struct work_struct *work) >> static int tcm_loop_queuecommand(struct Scsi_Host *sh, struct scsi_cmnd *sc) >> { >> struct tcm_loop_cmd *tl_cmd = scsi_cmd_priv(sc); >> + struct tcm_loop_hba *tl_hba; >> + struct tcm_loop_tpg *tl_tpg; >> + >> + tl_hba = *(struct tcm_loop_hba **)shost_priv(sc->device->host); >> + tl_tpg = &tl_hba->tl_hba_tpgs[sc->device->id]; >> pr_debug("%s() %d:%d:%d:%llu got CDB: 0x%02x scsi_buf_len: %u\n", >> __func__, sc->device->host->host_no, sc->device->id, > > AFAICS these new lines are not needed. Or am I missing something? You're right. I'll fix it.