[Spice-devel] [PATCH v7 10/20] fixup! usb-redir: add files for SCSI and USB MSC implementation
Victor Toso
victortoso at redhat.com
Mon Sep 16 10:20:53 UTC 2019
From: Frediano Ziglio <fziglio at redhat.com>
Do not use 0x%u as format, misleasding, %u is decimal.
Also most of these are formatting lun number which in multiple cases are
expressed as decimal.
---
src/cd-scsi.c | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/src/cd-scsi.c b/src/cd-scsi.c
index 25842b3..ea682ef 100644
--- a/src/cd-scsi.c
+++ b/src/cd-scsi.c
@@ -2079,7 +2079,7 @@ static void cd_scsi_cmd_send_event(CdScsiLU *dev, CdScsiRequest *req)
pers_prevent = event[1] & 0x80;
ext_req_code = (event[2] << 8) | event[3];
- SPICE_DEBUG("send_event, lun:0x%u immed:%u param_len:%u"
+ SPICE_DEBUG("send_event, lun:%u immed:%u param_len:%u"
" ext_req_event:0x%x ext_req_status:0x%x"
" pers_prevent:0x%x ext_req_code:0x%x",
req->lun, immed, event_param_len, ext_req_event,
@@ -2177,7 +2177,7 @@ static void cd_scsi_cmd_start_stop_unit(CdScsiLU *dev, CdScsiRequest *req)
load_eject = (req->cdb[4] & CD_START_STOP_FLAG_LOEJ) ? TRUE : FALSE;
power_cond = req->cdb[4] >> 4;
- SPICE_DEBUG("start_stop_unit, lun:0x%u"
+ SPICE_DEBUG("start_stop_unit, lun:%u"
" immed:%d start:%d load_eject:%d power_cond:0x%x(%s)",
req->lun, immed, start, load_eject, power_cond,
cd_scsi_start_stop_power_cond_name(power_cond));
@@ -2187,23 +2187,23 @@ static void cd_scsi_cmd_start_stop_unit(CdScsiLU *dev, CdScsiRequest *req)
if (!start) { /* stop the unit */
if (load_eject) { /* eject medium */
if (dev->prevent_media_removal) {
- SPICE_DEBUG("start_stop_unit, lun:0x%u"
+ SPICE_DEBUG("start_stop_unit, lun:%u"
" prevent_media_removal set, eject failed", req->lun);
cd_scsi_cmd_complete_check_cond(dev, req, &sense_code_MEDIUM_REMOVAL_PREVENTED);
return;
}
- SPICE_DEBUG("start_stop_unit, lun:0x%u eject", req->lun);
+ SPICE_DEBUG("start_stop_unit, lun:%u eject", req->lun);
cd_scsi_lu_unload(dev);
cd_scsi_dev_changed(dev->tgt->user_data, req->lun);
}
dev->power_cond = CD_SCSI_POWER_STOPPED;
- SPICE_DEBUG("start_stop_unit, lun:0x%u stopped", req->lun);
+ SPICE_DEBUG("start_stop_unit, lun:%u stopped", req->lun);
} else { /* start the unit */
dev->power_cond = CD_SCSI_POWER_ACTIVE;
- SPICE_DEBUG("start_stop_unit, lun:0x%u started", req->lun);
+ SPICE_DEBUG("start_stop_unit, lun:%u started", req->lun);
if (load_eject) { /* load medium */
- SPICE_DEBUG("start_stop_unit, lun:0x%u load with no media",
+ SPICE_DEBUG("start_stop_unit, lun:%u load with no media",
req->lun);
cd_scsi_lu_load(dev, NULL);
cd_scsi_dev_changed(dev->tgt->user_data, req->lun);
@@ -2213,17 +2213,17 @@ static void cd_scsi_cmd_start_stop_unit(CdScsiLU *dev, CdScsiRequest *req)
case CD_START_STOP_POWER_COND_ACTIVE:
/* not error to specify transition to the current power condition */
dev->power_cond = CD_SCSI_POWER_ACTIVE;
- SPICE_DEBUG("start_stop_unit, lun:0x%u active", req->lun);
+ SPICE_DEBUG("start_stop_unit, lun:%u active", req->lun);
break;
case CD_START_STOP_POWER_COND_IDLE:
case CD_START_STOP_POWER_COND_FORCE_IDLE_0:
dev->power_cond = CD_SCSI_POWER_IDLE;
- SPICE_DEBUG("start_stop_unit, lun:0x%u idle", req->lun);
+ SPICE_DEBUG("start_stop_unit, lun:%u idle", req->lun);
break;
case CD_START_STOP_POWER_COND_STANDBY:
case CD_START_STOP_POWER_COND_FORCE_STANDBY_0:
dev->power_cond = CD_SCSI_POWER_STANDBY;
- SPICE_DEBUG("start_stop_unit, lun:0x%u standby", req->lun);
+ SPICE_DEBUG("start_stop_unit, lun:%u standby", req->lun);
break;
case CD_START_STOP_POWER_COND_LU_CONTROL:
break;
--
2.21.0
More information about the Spice-devel
mailing list