[PATCH 0/2] tracetool: fix log-stap format

Laurent Vivier posted 2 patches 3 years, 4 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210105191721.120463-1-lvivier@redhat.com
Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Stefan Hajnoczi <stefanha@redhat.com>
hw/virtio/trace-events               | 2 +-
hw/virtio/vhost-vdpa.c               | 4 ++--
scripts/tracetool/format/log_stap.py | 1 +
3 files changed, 4 insertions(+), 3 deletions(-)
[PATCH 0/2] tracetool: fix log-stap format
Posted by Laurent Vivier 3 years, 4 months ago
The first patch fixes a problem I have introduced in vhost-vdpa
traces by adding an unsupported format ("%llu").

The second patch fixes a problem I've seen while I was checking
the result of the first patch: %PRI formats are not decoded
correctly and we can end with things like "0x%u" because
the used format is always the one of the first format of the string.

Laurent Vivier (2):
  vhost-vdpa: fix "unsigned long long" error with stap
  tracetool: fix "PRI" macro decoding

 hw/virtio/trace-events               | 2 +-
 hw/virtio/vhost-vdpa.c               | 4 ++--
 scripts/tracetool/format/log_stap.py | 1 +
 3 files changed, 4 insertions(+), 3 deletions(-)

-- 
2.29.2



Re: [PATCH 0/2] tracetool: fix log-stap format
Posted by Philippe Mathieu-Daudé 3 years, 4 months ago
On 1/5/21 8:17 PM, Laurent Vivier wrote:
> The first patch fixes a problem I have introduced in vhost-vdpa
> traces by adding an unsupported format ("%llu").
> 
> The second patch fixes a problem I've seen while I was checking
> the result of the first patch: %PRI formats are not decoded
> correctly and we can end with things like "0x%u" because
> the used format is always the one of the first format of the string.
> 
> Laurent Vivier (2):
>   vhost-vdpa: fix "unsigned long long" error with stap
>   tracetool: fix "PRI" macro decoding
> 
>  hw/virtio/trace-events               | 2 +-
>  hw/virtio/vhost-vdpa.c               | 4 ++--
>  scripts/tracetool/format/log_stap.py | 1 +
>  3 files changed, 4 insertions(+), 3 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>