Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

scsi: storvsc: Don't report the host packet status as the hv status

The log statement reports the packet status code as the hv status code
which causes confusion when debugging as "hv" might refer to a hypervisor,
and sometimes to the host part of the Hyper-V virtualization stack.

Fix the name of the datum being logged to clearly indicate the component
reporting the error. Also log it in hexadecimal everywhere for consistency.

Signed-off-by: Roman Kisel <romank@linux.microsoft.com>
Link: https://lore.kernel.org/r/20250304000940.9557-2-romank@linux.microsoft.com
Reviewed-by: Easwar Hariharan <eahariha@linux.microsoft.com>
Reviewed-by: Michael Kelley <mhklinux@outlook.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

authored by

Roman Kisel and committed by
Martin K. Petersen
7dcbda8a 11c79df9

+2 -2
+2 -2
drivers/scsi/storvsc_drv.c
··· 776 776 777 777 if (vstor_packet->operation != VSTOR_OPERATION_COMPLETE_IO || 778 778 vstor_packet->status != 0) { 779 - dev_err(dev, "Failed to create sub-channel: op=%d, sts=%d\n", 779 + dev_err(dev, "Failed to create sub-channel: op=%d, host=0x%x\n", 780 780 vstor_packet->operation, vstor_packet->status); 781 781 return; 782 782 } ··· 1183 1183 STORVSC_LOGGING_WARN : STORVSC_LOGGING_ERROR; 1184 1184 1185 1185 storvsc_log_ratelimited(device, loglevel, 1186 - "tag#%d cmd 0x%x status: scsi 0x%x srb 0x%x hv 0x%x\n", 1186 + "tag#%d cmd 0x%x status: scsi 0x%x srb 0x%x host 0x%x\n", 1187 1187 scsi_cmd_to_rq(request->cmd)->tag, 1188 1188 stor_pkt->vm_srb.cdb[0], 1189 1189 vstor_packet->vm_srb.scsi_status,