On Thu, Dec 7, 2023 at 7:51 PM Si-Wei Liu <si-wei....@oracle.com> wrote:
>
> For better debuggability and observability.
>
> Signed-off-by: Si-Wei Liu <si-wei....@oracle.com>
Reviewed-by: Eugenio Pérez <epere...@redhat.com>
> ---
> hw/virtio/trace-events | 2 +-
> hw/virtio/vhost-vdpa.c | 3 ++-
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
> index 196f32f..a8d3321 100644
> --- a/hw/virtio/trace-events
> +++ b/hw/virtio/trace-events
> @@ -58,7 +58,7 @@ vhost_vdpa_set_log_base(void *dev, uint64_t base, unsigned
> long long size, int r
> vhost_vdpa_set_vring_addr(void *dev, unsigned int index, unsigned int flags,
> uint64_t desc_user_addr, uint64_t used_user_addr, uint64_t avail_user_addr,
> uint64_t log_guest_addr) "dev: %p index: %u flags: 0x%x desc_user_addr:
> 0x%"PRIx64" used_user_addr: 0x%"PRIx64" avail_user_addr: 0x%"PRIx64"
> log_guest_addr: 0x%"PRIx64
> vhost_vdpa_set_vring_num(void *dev, unsigned int index, unsigned int num)
> "dev: %p index: %u num: %u"
> vhost_vdpa_set_vring_base(void *dev, unsigned int index, unsigned int num)
> "dev: %p index: %u num: %u"
> -vhost_vdpa_get_vring_base(void *dev, unsigned int index, unsigned int num)
> "dev: %p index: %u num: %u"
> +vhost_vdpa_get_vring_base(void *dev, unsigned int index, unsigned int num,
> bool svq) "dev: %p index: %u num: %u svq: %d"
> vhost_vdpa_set_vring_kick(void *dev, unsigned int index, int fd) "dev: %p
> index: %u fd: %d"
> vhost_vdpa_set_vring_call(void *dev, unsigned int index, int fd) "dev: %p
> index: %u fd: %d"
> vhost_vdpa_get_features(void *dev, uint64_t features) "dev: %p features:
> 0x%"PRIx64
> diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
> index 8ba390d..d66936f 100644
> --- a/hw/virtio/vhost-vdpa.c
> +++ b/hw/virtio/vhost-vdpa.c
> @@ -1607,6 +1607,7 @@ static int vhost_vdpa_get_vring_base(struct vhost_dev
> *dev,
>
> if (v->shadow_vqs_enabled) {
> ring->num = virtio_queue_get_last_avail_idx(dev->vdev, ring->index);
> + trace_vhost_vdpa_get_vring_base(dev, ring->index, ring->num, true);
> return 0;
> }
>
> @@ -1619,7 +1620,7 @@ static int vhost_vdpa_get_vring_base(struct vhost_dev
> *dev,
> }
>
> ret = vhost_vdpa_call(dev, VHOST_GET_VRING_BASE, ring);
> - trace_vhost_vdpa_get_vring_base(dev, ring->index, ring->num);
> + trace_vhost_vdpa_get_vring_base(dev, ring->index, ring->num, false);
> return ret;
> }
>
> --
> 1.8.3.1
>
