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

Eric Auger posted 3 patches 9 months, 4 weeks ago
There is a newer version of this series
[PATCH v2 2/3] virtio-iommu: Trace domain range limits as unsigned int
Posted by Eric Auger 9 months, 4 weeks ago
Use %u format to trace domain_range limits.

Signed-off-by: Eric Auger <eric.auger@redhat.com>
---
 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
Re: [PATCH v2 2/3] virtio-iommu: Trace domain range limits as unsigned int
Posted by Cédric Le Goater 9 months, 3 weeks ago
On 2/1/24 17:32, Eric Auger wrote:
> Use %u format to trace domain_range limits.
> 
> Signed-off-by: Eric Auger <eric.auger@redhat.com>


Reviewed-by: Cédric Le Goater <clg@redhat.com>

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
Posted by Duan, Zhenzhong 9 months, 4 weeks ago

>-----Original Message-----
>From: Eric Auger <eric.auger@redhat.com>
>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 <eric.auger@redhat.com>

Reviewed-by: Zhenzhong Duan <zhenzhong.duan@intel.com>

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