[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/7] hw/char/serial: Rename I/O read/write trace events
From: |
Richard Henderson |
Subject: |
Re: [PATCH 4/7] hw/char/serial: Rename I/O read/write trace events |
Date: |
Fri, 11 Sep 2020 16:08:03 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 |
On 9/6/20 6:55 PM, Philippe Mathieu-Daudé wrote:
> The serial_mm_read/write() handlers from the TYPE_SERIAL_MM device
> call the serial_ioport_read/write() handlers with shifted offset.
>
> When looking at the trace events from this MMIO device, it is
> confusing to read the accesses as I/O. Simplify using generic
> trace event names which make sense the various uses.
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> hw/char/serial.c | 4 ++--
> hw/char/trace-events | 4 ++--
> 2 files changed, 4 insertions(+), 4 deletions(-)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
- [PATCH 0/7] hw/char/serial: Housekeeping, Philippe Mathieu-Daudé, 2020/09/06
- [PATCH 1/7] hw/char/serial: Assert serial_ioport_read/write offset fits 8 bytes, Philippe Mathieu-Daudé, 2020/09/06
- [PATCH 2/7] hw/char/serial: Replace commented DPRINTF() by trace event, Philippe Mathieu-Daudé, 2020/09/06
- [PATCH 4/7] hw/char/serial: Rename I/O read/write trace events, Philippe Mathieu-Daudé, 2020/09/06
- Re: [PATCH 4/7] hw/char/serial: Rename I/O read/write trace events,
Richard Henderson <=
- [PATCH 5/7] hw/char/serial: Make 'wakeup' property boolean, Philippe Mathieu-Daudé, 2020/09/06
- [PATCH 3/7] hw/char/serial: Remove old DEBUG_SERIAL commented code, Philippe Mathieu-Daudé, 2020/09/06
- [PATCH 6/7] hw/char/serial-isa: Alias QDEV properties from generic serial object, Philippe Mathieu-Daudé, 2020/09/06
- [PATCH 7/7] hw/char/serial: Let SerialState have an 'id' field, Philippe Mathieu-Daudé, 2020/09/06