[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/2] vga: replace debug printf with trace points
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-devel] [PATCH 1/2] vga: replace debug printf with trace points |
Date: |
Wed, 8 Feb 2017 16:28:56 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 |
On 08/02/2017 14:51, Gerd Hoffmann wrote:
> Signed-off-by: Gerd Hoffmann <address@hidden>
Reviewed-by: Laurent Vivier <address@hidden>
> ---
> hw/display/trace-events | 6 ++++++
> hw/display/vga.c | 27 ++++-----------------------
> 2 files changed, 10 insertions(+), 23 deletions(-)
>
> diff --git a/hw/display/trace-events b/hw/display/trace-events
> index aadb612..26910e2 100644
> --- a/hw/display/trace-events
> +++ b/hw/display/trace-events
> @@ -119,3 +119,9 @@ qxl_set_client_capabilities_unsupported_by_revision(int
> qid, int revision) "%d r
> qxl_render_blit(int32_t stride, int32_t left, int32_t right, int32_t top,
> int32_t bottom) "stride=%d [%d, %d, %d, %d]"
> qxl_render_guest_primary_resized(int32_t width, int32_t height, int32_t
> stride, int32_t bytes_pp, int32_t bits_pp) "%dx%d, stride %d, bpp %d, depth
> %d"
> qxl_render_update_area_done(void *cookie) "%p"
> +
> +# hw/display/vga.c
> +vga_std_read_io(uint32_t addr, uint32_t val) "addr 0x%x, val 0x%x"
> +vga_std_write_io(uint32_t addr, uint32_t val) "addr 0x%x, val 0x%x"
> +vga_vbe_read(uint32_t index, uint32_t val) "index 0x%x, val 0x%x"
> +vga_vbe_write(uint32_t index, uint32_t val) "index 0x%x, val 0x%x"
> diff --git a/hw/display/vga.c b/hw/display/vga.c
> index 2a88b3c..69c3e1d 100644
> --- a/hw/display/vga.c
> +++ b/hw/display/vga.c
> @@ -34,12 +34,9 @@
> #include "hw/xen/xen.h"
> #include "trace.h"
>
> -//#define DEBUG_VGA
> //#define DEBUG_VGA_MEM
> //#define DEBUG_VGA_REG
>
> -//#define DEBUG_BOCHS_VBE
> -
> /* 16 state changes per vertical frame @60 Hz */
> #define VGA_TEXT_CURSOR_PERIOD_MS (1000 * 2 * 16 / 60)
>
> @@ -428,9 +425,7 @@ uint32_t vga_ioport_read(void *opaque, uint32_t addr)
> break;
> }
> }
> -#if defined(DEBUG_VGA)
> - printf("VGA: read addr=0x%04x data=0x%02x\n", addr, val);
> -#endif
> + trace_vga_std_read_io(addr, val);
> return val;
> }
>
> @@ -443,9 +438,7 @@ void vga_ioport_write(void *opaque, uint32_t addr,
> uint32_t val)
> if (vga_ioport_invalid(s, addr)) {
> return;
> }
> -#ifdef DEBUG_VGA
> - printf("VGA: write addr=0x%04x data=0x%02x\n", addr, val);
> -#endif
> + trace_vga_std_write_io(addr, val);
>
> switch(addr) {
> case VGA_ATT_W:
> @@ -733,9 +726,7 @@ uint32_t vbe_ioport_read_data(void *opaque, uint32_t addr)
> } else {
> val = 0;
> }
> -#ifdef DEBUG_BOCHS_VBE
> - printf("VBE: read index=0x%x val=0x%x\n", s->vbe_index, val);
> -#endif
> + trace_vga_vbe_read(s->vbe_index, val);
> return val;
> }
>
> @@ -750,9 +741,7 @@ void vbe_ioport_write_data(void *opaque, uint32_t addr,
> uint32_t val)
> VGACommonState *s = opaque;
>
> if (s->vbe_index <= VBE_DISPI_INDEX_NB) {
> -#ifdef DEBUG_BOCHS_VBE
> - printf("VBE: write index=0x%x val=0x%x\n", s->vbe_index, val);
> -#endif
> + trace_vga_vbe_write(s->vbe_index, val);
> switch(s->vbe_index) {
> case VBE_DISPI_INDEX_ID:
> if (val == VBE_DISPI_ID0 ||
> @@ -1543,17 +1532,9 @@ static void vga_draw_graphic(VGACommonState *s, int
> full_update)
> height, format, s->line_offset,
> s->vram_ptr + (s->start_addr * 4));
> dpy_gfx_replace_surface(s->con, surface);
> -#ifdef DEBUG_VGA
> - printf("VGA: Using shared surface for depth=%d swap=%d\n",
> - depth, byteswap);
> -#endif
> } else {
> qemu_console_resize(s->con, disp_width, height);
> surface = qemu_console_surface(s->con);
> -#ifdef DEBUG_VGA
> - printf("VGA: Using shadow surface for depth=%d swap=%d\n",
> - depth, byteswap);
> -#endif
> }
> s->last_scr_width = disp_width;
> s->last_scr_height = height;
>