mbox series

[0/6] Various tracing patches

Message ID 20231028122415.14869-1-shentey@gmail.com
Headers show
Series Various tracing patches | expand

Message

Bernhard Beschow Oct. 28, 2023, 12:24 p.m. UTC
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(-)

Comments

Peter Maydell Oct. 31, 2023, 4:17 p.m. UTC | #1
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
Bernhard Beschow Nov. 1, 2023, 5:37 p.m. UTC | #2
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