console: move set_mouse + cursor_define callbacks
When adding DisplayChangeListeners the set_mouse and cursor_define callbacks have been left in DisplayState for some reason. Fix it. Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
This commit is contained in:
		
							parent
							
								
									35c9e0a5c2
								
							
						
					
					
						commit
						bf2fde70fe
					
				| 
						 | 
					@ -1242,7 +1242,7 @@ static void text_console_update(void *opaque, console_ch_t *chardata)
 | 
				
			||||||
        s->text_y[1] = 0;
 | 
					        s->text_y[1] = 0;
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
    if (s->cursor_invalidate) {
 | 
					    if (s->cursor_invalidate) {
 | 
				
			||||||
        dpy_cursor(s->ds, s->x, s->y);
 | 
					        dpy_text_cursor(s->ds, s->x, s->y);
 | 
				
			||||||
        s->cursor_invalidate = 0;
 | 
					        s->cursor_invalidate = 0;
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
							
								
								
									
										39
									
								
								console.h
								
								
								
								
							
							
						
						
									
										39
									
								
								console.h
								
								
								
								
							| 
						 | 
					@ -164,6 +164,9 @@ struct DisplayChangeListener {
 | 
				
			||||||
                     int w, int h, uint32_t c);
 | 
					                     int w, int h, uint32_t c);
 | 
				
			||||||
    void (*dpy_text_cursor)(struct DisplayState *s, int x, int y);
 | 
					    void (*dpy_text_cursor)(struct DisplayState *s, int x, int y);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    void (*dpy_mouse_set)(struct DisplayState *s, int x, int y, int on);
 | 
				
			||||||
 | 
					    void (*dpy_cursor_define)(struct DisplayState *s, QEMUCursor *cursor);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    QLIST_ENTRY(DisplayChangeListener) next;
 | 
					    QLIST_ENTRY(DisplayChangeListener) next;
 | 
				
			||||||
};
 | 
					};
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					@ -181,9 +184,6 @@ struct DisplayState {
 | 
				
			||||||
    struct DisplayAllocator* allocator;
 | 
					    struct DisplayAllocator* allocator;
 | 
				
			||||||
    QLIST_HEAD(, DisplayChangeListener) listeners;
 | 
					    QLIST_HEAD(, DisplayChangeListener) listeners;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    void (*mouse_set)(int x, int y, int on);
 | 
					 | 
				
			||||||
    void (*cursor_define)(QEMUCursor *cursor);
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
    struct DisplayState *next;
 | 
					    struct DisplayState *next;
 | 
				
			||||||
};
 | 
					};
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					@ -304,7 +304,7 @@ static inline void dpy_fill(struct DisplayState *s, int x, int y,
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
static inline void dpy_cursor(struct DisplayState *s, int x, int y)
 | 
					static inline void dpy_text_cursor(struct DisplayState *s, int x, int y)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
    struct DisplayChangeListener *dcl;
 | 
					    struct DisplayChangeListener *dcl;
 | 
				
			||||||
    QLIST_FOREACH(dcl, &s->listeners, next) {
 | 
					    QLIST_FOREACH(dcl, &s->listeners, next) {
 | 
				
			||||||
| 
						 | 
					@ -314,6 +314,37 @@ static inline void dpy_cursor(struct DisplayState *s, int x, int y)
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					static inline void dpy_mouse_set(struct DisplayState *s, int x, int y, int on)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
					    struct DisplayChangeListener *dcl;
 | 
				
			||||||
 | 
					    QLIST_FOREACH(dcl, &s->listeners, next) {
 | 
				
			||||||
 | 
					        if (dcl->dpy_mouse_set) {
 | 
				
			||||||
 | 
					            dcl->dpy_mouse_set(s, x, y, on);
 | 
				
			||||||
 | 
					        }
 | 
				
			||||||
 | 
					    }
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					static inline void dpy_cursor_define(struct DisplayState *s, QEMUCursor *cursor)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
					    struct DisplayChangeListener *dcl;
 | 
				
			||||||
 | 
					    QLIST_FOREACH(dcl, &s->listeners, next) {
 | 
				
			||||||
 | 
					        if (dcl->dpy_cursor_define) {
 | 
				
			||||||
 | 
					            dcl->dpy_cursor_define(s, cursor);
 | 
				
			||||||
 | 
					        }
 | 
				
			||||||
 | 
					    }
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					static inline bool dpy_cursor_define_supported(struct DisplayState *s)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
					    struct DisplayChangeListener *dcl;
 | 
				
			||||||
 | 
					    QLIST_FOREACH(dcl, &s->listeners, next) {
 | 
				
			||||||
 | 
					        if (dcl->dpy_cursor_define) {
 | 
				
			||||||
 | 
					            return true;
 | 
				
			||||||
 | 
					        }
 | 
				
			||||||
 | 
					    }
 | 
				
			||||||
 | 
					    return false;
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
static inline int ds_get_linesize(DisplayState *ds)
 | 
					static inline int ds_get_linesize(DisplayState *ds)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
    return ds->surface->linesize;
 | 
					    return ds->surface->linesize;
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -210,7 +210,7 @@ static void jazz_led_text_update(void *opaque, console_ch_t *chardata)
 | 
				
			||||||
    LedState *s = opaque;
 | 
					    LedState *s = opaque;
 | 
				
			||||||
    char buf[2];
 | 
					    char buf[2];
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    dpy_cursor(s->ds, -1, -1);
 | 
					    dpy_text_cursor(s->ds, -1, -1);
 | 
				
			||||||
    qemu_console_resize(s->ds, 2, 1);
 | 
					    qemu_console_resize(s->ds, 2, 1);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    /* TODO: draw the segments */
 | 
					    /* TODO: draw the segments */
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -234,7 +234,7 @@ int qxl_render_cursor(PCIQXLDevice *qxl, QXLCommandExt *ext)
 | 
				
			||||||
        return 1;
 | 
					        return 1;
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    if (!qxl->ssd.ds->mouse_set || !qxl->ssd.ds->cursor_define) {
 | 
					    if (!dpy_cursor_define_supported(qxl->ssd.ds)) {
 | 
				
			||||||
        return 0;
 | 
					        return 0;
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
							
								
								
									
										10
									
								
								hw/vga.c
								
								
								
								
							
							
						
						
									
										10
									
								
								hw/vga.c
								
								
								
								
							| 
						 | 
					@ -2070,11 +2070,11 @@ static void vga_update_text(void *opaque, console_ch_t *chardata)
 | 
				
			||||||
            s->cr[VGA_CRTC_CURSOR_END] != s->cursor_end || full_update) {
 | 
					            s->cr[VGA_CRTC_CURSOR_END] != s->cursor_end || full_update) {
 | 
				
			||||||
            cursor_visible = !(s->cr[VGA_CRTC_CURSOR_START] & 0x20);
 | 
					            cursor_visible = !(s->cr[VGA_CRTC_CURSOR_START] & 0x20);
 | 
				
			||||||
            if (cursor_visible && cursor_offset < size && cursor_offset >= 0)
 | 
					            if (cursor_visible && cursor_offset < size && cursor_offset >= 0)
 | 
				
			||||||
                dpy_cursor(s->ds,
 | 
					                dpy_text_cursor(s->ds,
 | 
				
			||||||
                           TEXTMODE_X(cursor_offset),
 | 
					                                TEXTMODE_X(cursor_offset),
 | 
				
			||||||
                           TEXTMODE_Y(cursor_offset));
 | 
					                                TEXTMODE_Y(cursor_offset));
 | 
				
			||||||
            else
 | 
					            else
 | 
				
			||||||
                dpy_cursor(s->ds, -1, -1);
 | 
					                dpy_text_cursor(s->ds, -1, -1);
 | 
				
			||||||
            s->cursor_offset = cursor_offset;
 | 
					            s->cursor_offset = cursor_offset;
 | 
				
			||||||
            s->cursor_start = s->cr[VGA_CRTC_CURSOR_START];
 | 
					            s->cursor_start = s->cr[VGA_CRTC_CURSOR_START];
 | 
				
			||||||
            s->cursor_end = s->cr[VGA_CRTC_CURSOR_END];
 | 
					            s->cursor_end = s->cr[VGA_CRTC_CURSOR_END];
 | 
				
			||||||
| 
						 | 
					@ -2135,7 +2135,7 @@ static void vga_update_text(void *opaque, console_ch_t *chardata)
 | 
				
			||||||
    /* Display a message */
 | 
					    /* Display a message */
 | 
				
			||||||
    s->last_width = 60;
 | 
					    s->last_width = 60;
 | 
				
			||||||
    s->last_height = height = 3;
 | 
					    s->last_height = height = 3;
 | 
				
			||||||
    dpy_cursor(s->ds, -1, -1);
 | 
					    dpy_text_cursor(s->ds, -1, -1);
 | 
				
			||||||
    s->ds->surface->width = s->last_width;
 | 
					    s->ds->surface->width = s->last_width;
 | 
				
			||||||
    s->ds->surface->height = height;
 | 
					    s->ds->surface->height = height;
 | 
				
			||||||
    dpy_resize(s->ds);
 | 
					    dpy_resize(s->ds);
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -478,8 +478,7 @@ static inline void vmsvga_cursor_define(struct vmsvga_state_s *s,
 | 
				
			||||||
        qc = cursor_builtin_left_ptr();
 | 
					        qc = cursor_builtin_left_ptr();
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    if (s->vga.ds->cursor_define)
 | 
					    dpy_cursor_define(s->vga.ds, qc);
 | 
				
			||||||
        s->vga.ds->cursor_define(qc);
 | 
					 | 
				
			||||||
    cursor_put(qc);
 | 
					    cursor_put(qc);
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
#endif
 | 
					#endif
 | 
				
			||||||
| 
						 | 
					@ -754,9 +753,10 @@ static uint32_t vmsvga_value_read(void *opaque, uint32_t address)
 | 
				
			||||||
        caps |= SVGA_CAP_RECT_FILL;
 | 
					        caps |= SVGA_CAP_RECT_FILL;
 | 
				
			||||||
#endif
 | 
					#endif
 | 
				
			||||||
#ifdef HW_MOUSE_ACCEL
 | 
					#ifdef HW_MOUSE_ACCEL
 | 
				
			||||||
        if (s->vga.ds->mouse_set)
 | 
					        if (dpy_cursor_define_supported(s->vga.ds)) {
 | 
				
			||||||
            caps |= SVGA_CAP_CURSOR | SVGA_CAP_CURSOR_BYPASS_2 |
 | 
					            caps |= SVGA_CAP_CURSOR | SVGA_CAP_CURSOR_BYPASS_2 |
 | 
				
			||||||
                    SVGA_CAP_CURSOR_BYPASS;
 | 
					                    SVGA_CAP_CURSOR_BYPASS;
 | 
				
			||||||
 | 
					        }
 | 
				
			||||||
#endif
 | 
					#endif
 | 
				
			||||||
        return caps;
 | 
					        return caps;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					@ -903,8 +903,9 @@ static void vmsvga_value_write(void *opaque, uint32_t address, uint32_t value)
 | 
				
			||||||
        s->cursor.on |= (value == SVGA_CURSOR_ON_SHOW);
 | 
					        s->cursor.on |= (value == SVGA_CURSOR_ON_SHOW);
 | 
				
			||||||
        s->cursor.on &= (value != SVGA_CURSOR_ON_HIDE);
 | 
					        s->cursor.on &= (value != SVGA_CURSOR_ON_HIDE);
 | 
				
			||||||
#ifdef HW_MOUSE_ACCEL
 | 
					#ifdef HW_MOUSE_ACCEL
 | 
				
			||||||
        if (s->vga.ds->mouse_set && value <= SVGA_CURSOR_ON_SHOW)
 | 
					        if (value <= SVGA_CURSOR_ON_SHOW) {
 | 
				
			||||||
            s->vga.ds->mouse_set(s->cursor.x, s->cursor.y, s->cursor.on);
 | 
					            dpy_mouse_set(s->vga.ds, s->cursor.x, s->cursor.y, s->cursor.on);
 | 
				
			||||||
 | 
					        }
 | 
				
			||||||
#endif
 | 
					#endif
 | 
				
			||||||
        break;
 | 
					        break;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
							
								
								
									
										8
									
								
								ui/sdl.c
								
								
								
								
							
							
						
						
									
										8
									
								
								ui/sdl.c
								
								
								
								
							| 
						 | 
					@ -905,7 +905,7 @@ static void sdl_fill(DisplayState *ds, int x, int y, int w, int h, uint32_t c)
 | 
				
			||||||
    SDL_FillRect(real_screen, &dst, c);
 | 
					    SDL_FillRect(real_screen, &dst, c);
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
static void sdl_mouse_warp(int x, int y, int on)
 | 
					static void sdl_mouse_warp(DisplayState *ds, int x, int y, int on)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
    if (on) {
 | 
					    if (on) {
 | 
				
			||||||
        if (!guest_cursor)
 | 
					        if (!guest_cursor)
 | 
				
			||||||
| 
						 | 
					@ -921,7 +921,7 @@ static void sdl_mouse_warp(int x, int y, int on)
 | 
				
			||||||
    guest_x = x, guest_y = y;
 | 
					    guest_x = x, guest_y = y;
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
static void sdl_mouse_define(QEMUCursor *c)
 | 
					static void sdl_mouse_define(DisplayState *ds, QEMUCursor *c)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
    uint8_t *image, *mask;
 | 
					    uint8_t *image, *mask;
 | 
				
			||||||
    int bpl;
 | 
					    int bpl;
 | 
				
			||||||
| 
						 | 
					@ -1025,8 +1025,8 @@ void sdl_display_init(DisplayState *ds, int full_screen, int no_frame)
 | 
				
			||||||
    dcl->dpy_refresh = sdl_refresh;
 | 
					    dcl->dpy_refresh = sdl_refresh;
 | 
				
			||||||
    dcl->dpy_setdata = sdl_setdata;
 | 
					    dcl->dpy_setdata = sdl_setdata;
 | 
				
			||||||
    dcl->dpy_fill = sdl_fill;
 | 
					    dcl->dpy_fill = sdl_fill;
 | 
				
			||||||
    ds->mouse_set = sdl_mouse_warp;
 | 
					    dcl->dpy_mouse_set = sdl_mouse_warp;
 | 
				
			||||||
    ds->cursor_define = sdl_mouse_define;
 | 
					    dcl->dpy_cursor_define = sdl_mouse_define;
 | 
				
			||||||
    register_displaychangelistener(ds, dcl);
 | 
					    register_displaychangelistener(ds, dcl);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    da = g_malloc0(sizeof(DisplayAllocator));
 | 
					    da = g_malloc0(sizeof(DisplayAllocator));
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -404,12 +404,12 @@ void qemu_spice_display_resize(SimpleSpiceDisplay *ssd)
 | 
				
			||||||
void qemu_spice_cursor_refresh_unlocked(SimpleSpiceDisplay *ssd)
 | 
					void qemu_spice_cursor_refresh_unlocked(SimpleSpiceDisplay *ssd)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
    if (ssd->cursor) {
 | 
					    if (ssd->cursor) {
 | 
				
			||||||
        ssd->ds->cursor_define(ssd->cursor);
 | 
					        dpy_cursor_define(ssd->ds, ssd->cursor);
 | 
				
			||||||
        cursor_put(ssd->cursor);
 | 
					        cursor_put(ssd->cursor);
 | 
				
			||||||
        ssd->cursor = NULL;
 | 
					        ssd->cursor = NULL;
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
    if (ssd->mouse_x != -1 && ssd->mouse_y != -1) {
 | 
					    if (ssd->mouse_x != -1 && ssd->mouse_y != -1) {
 | 
				
			||||||
        ssd->ds->mouse_set(ssd->mouse_x, ssd->mouse_y, 1);
 | 
					        dpy_mouse_set(ssd->ds, ssd->mouse_x, ssd->mouse_y, 1);
 | 
				
			||||||
        ssd->mouse_x = -1;
 | 
					        ssd->mouse_x = -1;
 | 
				
			||||||
        ssd->mouse_y = -1;
 | 
					        ssd->mouse_y = -1;
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
							
								
								
									
										8
									
								
								ui/vnc.c
								
								
								
								
							
							
						
						
									
										8
									
								
								ui/vnc.c
								
								
								
								
							| 
						 | 
					@ -802,7 +802,7 @@ static void vnc_dpy_copy(DisplayState *ds, int src_x, int src_y, int dst_x, int
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
static void vnc_mouse_set(int x, int y, int visible)
 | 
					static void vnc_mouse_set(DisplayState *ds, int x, int y, int visible)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
    /* can we ask the client(s) to move the pointer ??? */
 | 
					    /* can we ask the client(s) to move the pointer ??? */
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
| 
						 | 
					@ -829,7 +829,7 @@ static int vnc_cursor_define(VncState *vs)
 | 
				
			||||||
    return -1;
 | 
					    return -1;
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
static void vnc_dpy_cursor_define(QEMUCursor *c)
 | 
					static void vnc_dpy_cursor_define(DisplayState *ds, QEMUCursor *c)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
    VncDisplay *vd = vnc_display;
 | 
					    VncDisplay *vd = vnc_display;
 | 
				
			||||||
    VncState *vs;
 | 
					    VncState *vs;
 | 
				
			||||||
| 
						 | 
					@ -2757,9 +2757,9 @@ void vnc_display_init(DisplayState *ds)
 | 
				
			||||||
    dcl->dpy_update = vnc_dpy_update;
 | 
					    dcl->dpy_update = vnc_dpy_update;
 | 
				
			||||||
    dcl->dpy_resize = vnc_dpy_resize;
 | 
					    dcl->dpy_resize = vnc_dpy_resize;
 | 
				
			||||||
    dcl->dpy_setdata = vnc_dpy_setdata;
 | 
					    dcl->dpy_setdata = vnc_dpy_setdata;
 | 
				
			||||||
 | 
					    dcl->dpy_mouse_set = vnc_mouse_set;
 | 
				
			||||||
 | 
					    dcl->dpy_cursor_define = vnc_dpy_cursor_define;
 | 
				
			||||||
    register_displaychangelistener(ds, dcl);
 | 
					    register_displaychangelistener(ds, dcl);
 | 
				
			||||||
    ds->mouse_set = vnc_mouse_set;
 | 
					 | 
				
			||||||
    ds->cursor_define = vnc_dpy_cursor_define;
 | 
					 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in New Issue