Re: [PATCH v2 2/3] virtio-iommu: Trace domain range limits as unsigned int

2024-02-05 Thread Cédric Le Goater 

On 2/1/24 17:32, Eric Auger wrote:

Use %u format to trace domain_range limits.

Signed-off-by: Eric Auger 



Reviewed-by: Cédric Le Goater 

Thanks,

C.



---
  hw/virtio/trace-events | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
index 77905d1994..2350849fbd 100644
--- a/hw/virtio/trace-events
+++ b/hw/virtio/trace-events
@@ -111,7 +111,7 @@ virtio_iommu_device_reset(void) "reset!"
  virtio_iommu_system_reset(void) "system reset!"
  virtio_iommu_get_features(uint64_t features) "device supports 
features=0x%"PRIx64
  virtio_iommu_device_status(uint8_t status) "driver status = %d"
-virtio_iommu_get_config(uint64_t page_size_mask, uint64_t start, uint64_t end, uint32_t domain_start, uint32_t 
domain_end, uint32_t probe_size, uint8_t bypass) "page_size_mask=0x%"PRIx64" input range 
start=0x%"PRIx64" input range end=0x%"PRIx64" domain range start=%d domain range end=%d 
probe_size=0x%x bypass=0x%x"
+virtio_iommu_get_config(uint64_t page_size_mask, uint64_t start, uint64_t end, uint32_t domain_start, uint32_t 
domain_end, uint32_t probe_size, uint8_t bypass) "page_size_mask=0x%"PRIx64" input range 
start=0x%"PRIx64" input range end=0x%"PRIx64" domain range start=%u domain range end=%u 
probe_size=0x%x bypass=0x%x"
  virtio_iommu_set_config(uint8_t bypass) "bypass=0x%x"
  virtio_iommu_attach(uint32_t domain_id, uint32_t ep_id) "domain=%d 
endpoint=%d"
  virtio_iommu_detach(uint32_t domain_id, uint32_t ep_id) "domain=%d 
endpoint=%d"

RE: [PATCH v2 2/3] virtio-iommu: Trace domain range limits as unsigned int

2024-02-01 Thread Duan, Zhenzhong 



>-Original Message-
>From: Eric Auger 
>Subject: [PATCH v2 2/3] virtio-iommu: Trace domain range limits as
>unsigned int
>
>Use %u format to trace domain_range limits.
>
>Signed-off-by: Eric Auger 

Reviewed-by: Zhenzhong Duan 

Thanks
Zhenzhong

>---
> hw/virtio/trace-events | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
>index 77905d1994..2350849fbd 100644
>--- a/hw/virtio/trace-events
>+++ b/hw/virtio/trace-events
>@@ -111,7 +111,7 @@ virtio_iommu_device_reset(void) "reset!"
> virtio_iommu_system_reset(void) "system reset!"
> virtio_iommu_get_features(uint64_t features) "device supports
>features=0x%"PRIx64
> virtio_iommu_device_status(uint8_t status) "driver status = %d"
>-virtio_iommu_get_config(uint64_t page_size_mask, uint64_t start,
>uint64_t end, uint32_t domain_start, uint32_t domain_end, uint32_t
>probe_size, uint8_t bypass) "page_size_mask=0x%"PRIx64" input range
>start=0x%"PRIx64" input range end=0x%"PRIx64" domain range start=%d
>domain range end=%d probe_size=0x%x bypass=0x%x"
>+virtio_iommu_get_config(uint64_t page_size_mask, uint64_t start,
>uint64_t end, uint32_t domain_start, uint32_t domain_end, uint32_t
>probe_size, uint8_t bypass) "page_size_mask=0x%"PRIx64" input range
>start=0x%"PRIx64" input range end=0x%"PRIx64" domain range start=%u
>domain range end=%u probe_size=0x%x bypass=0x%x"
> virtio_iommu_set_config(uint8_t bypass) "bypass=0x%x"
> virtio_iommu_attach(uint32_t domain_id, uint32_t ep_id) "domain=%d
>endpoint=%d"
> virtio_iommu_detach(uint32_t domain_id, uint32_t ep_id) "domain=%d
>endpoint=%d"
>--
>2.41.0

[PATCH v2 2/3] virtio-iommu: Trace domain range limits as unsigned int

2024-02-01 Thread Eric Auger 
Use %u format to trace domain_range limits.

Signed-off-by: Eric Auger 
---
 hw/virtio/trace-events | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
index 77905d1994..2350849fbd 100644
--- a/hw/virtio/trace-events
+++ b/hw/virtio/trace-events
@@ -111,7 +111,7 @@ virtio_iommu_device_reset(void) "reset!"
 virtio_iommu_system_reset(void) "system reset!"
 virtio_iommu_get_features(uint64_t features) "device supports 
features=0x%"PRIx64
 virtio_iommu_device_status(uint8_t status) "driver status = %d"
-virtio_iommu_get_config(uint64_t page_size_mask, uint64_t start, uint64_t end, 
uint32_t domain_start, uint32_t domain_end, uint32_t probe_size, uint8_t 
bypass) "page_size_mask=0x%"PRIx64" input range start=0x%"PRIx64" input range 
end=0x%"PRIx64" domain range start=%d domain range end=%d probe_size=0x%x 
bypass=0x%x"
+virtio_iommu_get_config(uint64_t page_size_mask, uint64_t start, uint64_t end, 
uint32_t domain_start, uint32_t domain_end, uint32_t probe_size, uint8_t 
bypass) "page_size_mask=0x%"PRIx64" input range start=0x%"PRIx64" input range 
end=0x%"PRIx64" domain range start=%u domain range end=%u probe_size=0x%x 
bypass=0x%x"
 virtio_iommu_set_config(uint8_t bypass) "bypass=0x%x"
 virtio_iommu_attach(uint32_t domain_id, uint32_t ep_id) "domain=%d endpoint=%d"
 virtio_iommu_detach(uint32_t domain_id, uint32_t ep_id) "domain=%d endpoint=%d"
-- 
2.41.0