On 7/6/26 12:36 PM, Shameer Kolothum wrote:
> The trace event receives dev_id before type, but its format string prints
> them in the wrong order. Correct the order.
>
> Fixes: f2d31df0d925 ("backends/iommufd: Introduce
> iommufd_backend_alloc_viommu")
> Reported-by: Nicolin Chen <[email protected]>
> Signed-off-by: Shameer Kolothum <[email protected]>
Reviewed-by: Eric Auger <[email protected]>
Thanks
Eric
> ---
> backends/trace-events | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/backends/trace-events b/backends/trace-events
> index b63420b73e..009a25b0be 100644
> --- a/backends/trace-events
> +++ b/backends/trace-events
> @@ -21,7 +21,7 @@ iommufd_backend_free_id(int iommufd, uint32_t id, int ret)
> " iommufd=%d id=%d (%
> iommufd_backend_set_dirty(int iommufd, uint32_t hwpt_id, bool start, int
> ret) " iommufd=%d hwpt=%u enable=%d (%d)"
> iommufd_backend_get_dirty_bitmap(int iommufd, uint32_t hwpt_id, uint64_t
> iova, uint64_t size, uint64_t flags, uint64_t page_size, int ret) "
> iommufd=%d hwpt=%u iova=0x%"PRIx64" size=0x%"PRIx64" flags=0x%"PRIx64"
> page_size=0x%"PRIx64" (%d)"
> iommufd_backend_invalidate_cache(int iommufd, uint32_t id, uint32_t
> data_type, uint32_t entry_len, uint32_t entry_num, uint32_t done_num,
> uint64_t data_ptr, int ret) " iommufd=%d id=%u data_type=%u entry_len=%u
> entry_num=%u done_num=%u data_ptr=0x%"PRIx64" (%d)"
> -iommufd_backend_alloc_viommu(int iommufd, uint32_t dev_id, uint32_t type,
> uint32_t hwpt_id, uint64_t data_ptr, uint32_t data_len, uint32_t viommu_id,
> int ret) " iommufd=%d type=%u dev_id=%u hwpt_id=%u data_ptr=0x%"PRIx64"
> data_len=0x%x viommu_id=%u (%d)"
> +iommufd_backend_alloc_viommu(int iommufd, uint32_t dev_id, uint32_t type,
> uint32_t hwpt_id, uint64_t data_ptr, uint32_t data_len, uint32_t viommu_id,
> int ret) " iommufd=%d dev_id=%u type=%u hwpt_id=%u data_ptr=0x%"PRIx64"
> data_len=0x%x viommu_id=%u (%d)"
> iommufd_backend_alloc_vdev(int iommufd, uint32_t dev_id, uint32_t viommu_id,
> uint64_t virt_id, uint32_t vdev_id, int ret) " iommufd=%d dev_id=%u
> viommu_id=%u virt_id=0x%"PRIx64" vdev_id=%u (%d)"
> iommufd_viommu_alloc_eventq(int iommufd, uint32_t viommu_id, uint32_t type,
> uint32_t veventq_id, uint32_t veventq_fd, int ret) " iommufd=%d viommu_id=%u
> type=%u veventq_id=%u veventq_fd=%u (%d)"
> iommufd_backend_alloc_hw_queue(int iommufd, uint32_t viommu_id, uint32_t
> queue_type, uint32_t index, uint64_t addr, uint64_t size, uint32_t queue_id,
> int ret) " iommufd=%d viommu_id=%u queue_type=%u index=%u addr=0x%"PRIx64"
> size=0x%"PRIx64" queue_id=%u (%d)"
