Various tracing patches

[PATCH 0/6] Various tracing patches

Bernhard Beschow posted 6 patches 1 year, 1 month ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20231028122415.14869-1-shentey@gmail.com

Maintainers: Stefan Hajnoczi <stefanha@redhat.com>, Mads Ynddal <mads@ynddal.dk>, "Michael S. Tsirkin" <mst@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Peter Maydell <peter.maydell@linaro.org>, Jean-Christophe Dubois <jcd@tribudubois.net>, Andrey Smirnov <andrew.smirnov@gmail.com>, Peter Xu <peterx@redhat.com>, David Hildenbrand <david@redhat.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>

docs/devel/tracing.rst   |  4 ++--
hw/i2c/pm_smbus.c        | 18 ++++--------------
hw/misc/imx6_ccm.c       | 41 ++++++++++++++--------------------------
hw/misc/imx7_snvs.c      |  5 +++++
hw/watchdog/wdt_imx2.c   | 28 +++++++++++++++++++++------
system/memory.c          | 26 +++++++++++++++----------
hw/i2c/trace-events      |  6 ++++++
hw/misc/trace-events     | 19 +++++++++++++++++++
hw/watchdog/trace-events |  6 ++++++
system/trace-events      | 12 ++++++------
10 files changed, 100 insertions(+), 65 deletions(-)

Expand all Fold all

[PATCH 0/6] Various tracing patches

Posted by Bernhard Beschow 1 year, 1 month ago

This series enhances the tracing experience of some i.MX devices by adding new
trace events and by converting from DPRINTF. SMBus gets also converted from
DPRINTF to trace events. Finally, when tracing memory region operations, host
pointers aren't traced any longer and are substituted by their memory region
names.

Bernhard Beschow (6):
  hw/watchdog/wdt_imx2: Trace MMIO access
  hw/watchdog/wdt_imx2: Trace timer activity
  hw/misc/imx7_snvs: Trace MMIO access
  hw/misc/imx6_ccm: Convert DPRINTF to trace events
  hw/i2c/pm_smbus: Convert DPRINTF to trace events
  system/memory: Trace names of MemoryRegions rather than host pointers

 docs/devel/tracing.rst   |  4 ++--
 hw/i2c/pm_smbus.c        | 18 ++++--------------
 hw/misc/imx6_ccm.c       | 41 ++++++++++++++--------------------------
 hw/misc/imx7_snvs.c      |  5 +++++
 hw/watchdog/wdt_imx2.c   | 28 +++++++++++++++++++++------
 system/memory.c          | 26 +++++++++++++++----------
 hw/i2c/trace-events      |  6 ++++++
 hw/misc/trace-events     | 19 +++++++++++++++++++
 hw/watchdog/trace-events |  6 ++++++
 system/trace-events      | 12 ++++++------
 10 files changed, 100 insertions(+), 65 deletions(-)

-- 
2.42.0

Re: [PATCH 0/6] Various tracing patches

Posted by Peter Maydell 1 year ago

On Sat, 28 Oct 2023 at 13:24, Bernhard Beschow <shentey@gmail.com> wrote:
>
> This series enhances the tracing experience of some i.MX devices by adding new
> trace events and by converting from DPRINTF. SMBus gets also converted from
> DPRINTF to trace events. Finally, when tracing memory region operations, host
> pointers aren't traced any longer and are substituted by their memory region
> names.
>
> Bernhard Beschow (6):
>   hw/watchdog/wdt_imx2: Trace MMIO access
>   hw/watchdog/wdt_imx2: Trace timer activity
>   hw/misc/imx7_snvs: Trace MMIO access
>   hw/misc/imx6_ccm: Convert DPRINTF to trace events
>   hw/i2c/pm_smbus: Convert DPRINTF to trace events
>   system/memory: Trace names of MemoryRegions rather than host pointers

Since these are mostly arm devices I've taken patches 1-5
into target-arm.next (with the addition of "Hz" to the
frequency traces in patch 4). Patch 6 looks like it needs
further discussion.

thanks
-- PMM

Re: [PATCH 0/6] Various tracing patches

Posted by Bernhard Beschow 1 year ago


Am 31. Oktober 2023 16:17:32 UTC schrieb Peter Maydell <peter.maydell@linaro.org>:
>On Sat, 28 Oct 2023 at 13:24, Bernhard Beschow <shentey@gmail.com> wrote:
>>
>> This series enhances the tracing experience of some i.MX devices by adding new
>> trace events and by converting from DPRINTF. SMBus gets also converted from
>> DPRINTF to trace events. Finally, when tracing memory region operations, host
>> pointers aren't traced any longer and are substituted by their memory region
>> names.
>>
>> Bernhard Beschow (6):
>>   hw/watchdog/wdt_imx2: Trace MMIO access
>>   hw/watchdog/wdt_imx2: Trace timer activity
>>   hw/misc/imx7_snvs: Trace MMIO access
>>   hw/misc/imx6_ccm: Convert DPRINTF to trace events
>>   hw/i2c/pm_smbus: Convert DPRINTF to trace events
>>   system/memory: Trace names of MemoryRegions rather than host pointers
>
>Since these are mostly arm devices I've taken patches 1-5
>into target-arm.next (with the addition of "Hz" to the
>frequency traces in patch 4).

Excellent. Thanks!

Best regards,
Bernhard

> Patch 6 looks like it needs
>further discussion.
>
>thanks
>-- PMM