[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 06/14] hw/display/cirrus_vga: Convert debug printf() to trace
From: |
Alistair Francis |
Subject: |
Re: [PATCH 06/14] hw/display/cirrus_vga: Convert debug printf() to trace event |
Date: |
Tue, 26 May 2020 09:46:43 -0700 |
On Mon, May 25, 2020 at 11:25 PM Philippe Mathieu-Daudé <address@hidden> wrote:
>
> Convert the final bit of DEBUG_BITBLT to a tracepoint.
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Alistair Francis <address@hidden>
Alistair
> ---
> hw/display/cirrus_vga.c | 24 ++++++++++--------------
> hw/display/trace-events | 1 +
> 2 files changed, 11 insertions(+), 14 deletions(-)
>
> diff --git a/hw/display/cirrus_vga.c b/hw/display/cirrus_vga.c
> index 76e2dc5bb6..92c197cdde 100644
> --- a/hw/display/cirrus_vga.c
> +++ b/hw/display/cirrus_vga.c
> @@ -53,7 +53,6 @@
> */
>
> //#define DEBUG_CIRRUS
> -//#define DEBUG_BITBLT
>
> /***************************************
> *
> @@ -950,19 +949,16 @@ static void cirrus_bitblt_start(CirrusVGAState * s)
> s->cirrus_blt_dstaddr &= s->cirrus_addr_mask;
> s->cirrus_blt_srcaddr &= s->cirrus_addr_mask;
>
> -#ifdef DEBUG_BITBLT
> - printf("rop=0x%02x mode=0x%02x modeext=0x%02x w=%d h=%d dpitch=%d
> spitch=%d daddr=0x%08x saddr=0x%08x writemask=0x%02x\n",
> - blt_rop,
> - s->cirrus_blt_mode,
> - s->cirrus_blt_modeext,
> - s->cirrus_blt_width,
> - s->cirrus_blt_height,
> - s->cirrus_blt_dstpitch,
> - s->cirrus_blt_srcpitch,
> - s->cirrus_blt_dstaddr,
> - s->cirrus_blt_srcaddr,
> - s->vga.gr[0x2f]);
> -#endif
> + trace_vga_cirrus_bitblt_start(blt_rop,
> + s->cirrus_blt_mode,
> + s->cirrus_blt_modeext,
> + s->cirrus_blt_width,
> + s->cirrus_blt_height,
> + s->cirrus_blt_dstpitch,
> + s->cirrus_blt_srcpitch,
> + s->cirrus_blt_dstaddr,
> + s->cirrus_blt_srcaddr,
> + s->vga.gr[0x2f]);
>
> switch (s->cirrus_blt_mode & CIRRUS_BLTMODE_PIXELWIDTHMASK) {
> case CIRRUS_BLTMODE_PIXELWIDTH8:
> diff --git a/hw/display/trace-events b/hw/display/trace-events
> index c3043e4ced..bb089a5f5e 100644
> --- a/hw/display/trace-events
> +++ b/hw/display/trace-events
> @@ -134,6 +134,7 @@ vga_cirrus_read_io(uint32_t addr, uint32_t val) "addr
> 0x%x, val 0x%x"
> vga_cirrus_write_io(uint32_t addr, uint32_t val) "addr 0x%x, val 0x%x"
> vga_cirrus_write_blt(uint32_t offset, uint32_t val) "offset 0x%x, val 0x%x"
> vga_cirrus_write_gr(uint8_t index, uint8_t val) "GR addr 0x%02x, val 0x%02x"
> +vga_cirrus_bitblt_start(uint8_t blt_rop, uint8_t blt_mode, uint8_t
> blt_modeext, int blt_width, int blt_height, int blt_dstpitch, int
> blt_srcpitch, uint32_t blt_dstaddr, uint32_t blt_srcaddr, uint8_t gr_val)
> "rop=0x%02x mode=0x%02x modeext=0x%02x w=%d h=%d dpitch=%d spitch=%d
> daddr=0x%08"PRIx32" saddr=0x%08"PRIx32" writemask=0x%02x"
>
> # sii9022.c
> sii9022_read_reg(uint8_t addr, uint8_t val) "addr 0x%02x, val 0x%02x"
> --
> 2.21.3
>
>
- [PATCH 00/14] hw/display: Omnibus cleanups, Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 01/14] hw/display/edid: Add missing 'qdev-properties.h' header, Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 02/14] hw/display/cg3: Convert debug printf()s to trace events, Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 03/14] hw/display/cirrus_vga: Convert debug printf() to trace event, Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 04/14] hw/display/cirrus_vga: Use qemu_log_mask(UNIMP) instead of debug printf, Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 06/14] hw/display/cirrus_vga: Convert debug printf() to trace event, Philippe Mathieu-Daudé, 2020/05/26
- Re: [PATCH 06/14] hw/display/cirrus_vga: Convert debug printf() to trace event,
Alistair Francis <=
- [PATCH 07/14] hw/display/dpcd: Fix memory region size, Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 05/14] hw/display/cirrus_vga: Use qemu_log_mask(ERROR) instead of debug printf, Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 08/14] hw/display/dpcd: Convert debug printf()s to trace events, Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 10/14] hw/display/vmware_vga: Replace printf() calls by qemu_log_mask(ERROR), Philippe Mathieu-Daudé, 2020/05/26
- [PATCH 09/14] hw/display/xlnx_dp: Replace disabled DPRINTF() by error_report(), Philippe Mathieu-Daudé, 2020/05/26