[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] util/vfio-helpers: Unify trace-events size format
From: |
Laurent Vivier |
Subject: |
Re: [PATCH] util/vfio-helpers: Unify trace-events size format |
Date: |
Tue, 1 Sep 2020 11:35:51 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
Le 20/08/2020 à 19:10, Philippe Mathieu-Daudé a écrit :
> Some 'qemu_vfio_*' trace events sizes are displayed using
> decimal notation, other using hexadecimal notation:
>
> qemu_vfio_ram_block_added s 0xaaaaf2448d90 host 0xffff5bc00000 size
> 0x4000000
> qemu_vfio_dma_map s 0xaaaaf2448d90 host 0xffff5bc00000 size 67108864
> temporary 0 iova (nil)
> qemu_vfio_find_mapping s 0xaaaaf2448d90 host 0xffff5bc00000
> qemu_vfio_new_mapping s 0xaaaaf2448d90 host 0xffff5bc00000 size 67108864
> index 4 iova 0x114000
> qemu_vfio_do_mapping s 0xaaaaf2448d90 host 0xffff5bc00000 size 67108864
> iova 0x114000
>
> As it is hard to follow, unify using hexadecimal for all sizes:
>
> qemu_vfio_ram_block_added s 0xaaaaf1c60d90 host 0xffff2bc00000 size
> 0x4000000
> qemu_vfio_dma_map s 0xaaaaf1c60d90 host 0xffff2bc00000 size 0x4000000
> temporary 0 iova (nil)
> qemu_vfio_find_mapping s 0xaaaaf1c60d90 host 0xffff2bc00000
> qemu_vfio_new_mapping s 0xaaaaf1c60d90 host 0xffff2bc00000 size 0x4000000
> index 4 iova 0x114000
> qemu_vfio_do_mapping s 0xaaaaf1c60d90 host 0xffff2bc00000 size 0x4000000
> iova 0x114000
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> util/trace-events | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/util/trace-events b/util/trace-events
> index 0ce42822eba..d9a0b4f8c63 100644
> --- a/util/trace-events
> +++ b/util/trace-events
> @@ -79,7 +79,7 @@ qemu_vfio_dma_reset_temporary(void *s) "s %p"
> qemu_vfio_ram_block_added(void *s, void *p, size_t size) "s %p host %p size
> 0x%zx"
> qemu_vfio_ram_block_removed(void *s, void *p, size_t size) "s %p host %p
> size 0x%zx"
> qemu_vfio_find_mapping(void *s, void *p) "s %p host %p"
> -qemu_vfio_new_mapping(void *s, void *host, size_t size, int index, uint64_t
> iova) "s %p host %p size %zu index %d iova 0x%"PRIx64
> -qemu_vfio_do_mapping(void *s, void *host, size_t size, uint64_t iova) "s %p
> host %p size %zu iova 0x%"PRIx64
> -qemu_vfio_dma_map(void *s, void *host, size_t size, bool temporary, uint64_t
> *iova) "s %p host %p size %zu temporary %d iova %p"
> +qemu_vfio_new_mapping(void *s, void *host, size_t size, int index, uint64_t
> iova) "s %p host %p size 0x%zx index %d iova 0x%"PRIx64
> +qemu_vfio_do_mapping(void *s, void *host, size_t size, uint64_t iova) "s %p
> host %p size 0x%zx iova 0x%"PRIx64
> +qemu_vfio_dma_map(void *s, void *host, size_t size, bool temporary, uint64_t
> *iova) "s %p host %p size 0x%zx temporary %d iova %p"
> qemu_vfio_dma_unmap(void *s, void *host) "s %p host %p"
>
Applied to my trivial-patches branch.
Thanks,
Laurent
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH] util/vfio-helpers: Unify trace-events size format,
Laurent Vivier <=