ShuriZma
/
xemu
mirror of https://github.com/xemu-project/xemu.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

vnc: stop using DisplayState 

Rework DisplayStateListener callbacks to not use the DisplayState
any more.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
This commit is contained in:
Gerd Hoffmann 2013-02-28 17:16:48 +01:00
parent 9d9801cf80
commit d39fa6d86d
4 changed files with 48 additions and 40 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
ui/vnc-enc-tight.c
View File

@ -123,7 +123,7 @@ static bool tight_can_send_png_rect(VncState *vs, int w, int h)
return false; return false;
} }
if (ds_get_bytes_per_pixel(vs->ds) == 1 || if (surface_bytes_per_pixel(vs->vd->ds) == 1 ||
vs->client_pf.bytes_per_pixel == 1) { vs->client_pf.bytes_per_pixel == 1) {
return false; return false;
} }
@ -301,7 +301,7 @@ tight_detect_smooth_image(VncState *vs, int w, int h)
return 0; return 0;
} }
if (ds_get_bytes_per_pixel(vs->ds) == 1 || if (surface_bytes_per_pixel(vs->vd->ds) == 1 ||
vs->client_pf.bytes_per_pixel == 1 || vs->client_pf.bytes_per_pixel == 1 ||
w < VNC_TIGHT_DETECT_MIN_WIDTH || h < VNC_TIGHT_DETECT_MIN_HEIGHT) { w < VNC_TIGHT_DETECT_MIN_WIDTH || h < VNC_TIGHT_DETECT_MIN_HEIGHT) {
return 0; return 0;
@ -1184,8 +1184,9 @@ static int send_jpeg_rect(VncState *vs, int x, int y, int w, int h, int quality)
uint8_t *buf; uint8_t *buf;
int dy; int dy;
if (ds_get_bytes_per_pixel(vs->ds) == 1) if (surface_bytes_per_pixel(vs->vd->ds) == 1) {
return send_full_color_rect(vs, x, y, w, h); return send_full_color_rect(vs, x, y, w, h);
}
buffer_reserve(&vs->tight.jpeg, 2048); buffer_reserve(&vs->tight.jpeg, 2048);

1
ui/vnc-jobs.c
View File

@ -183,7 +183,6 @@ static void vnc_async_encoding_start(VncState *orig, VncState *local)
{ {
local->vnc_encoding = orig->vnc_encoding; local->vnc_encoding = orig->vnc_encoding;
local->features = orig->features; local->features = orig->features;
local->ds = orig->ds;
local->vd = orig->vd; local->vd = orig->vd;
local->lossy_rect = orig->lossy_rect; local->lossy_rect = orig->lossy_rect;
local->write_pixels = orig->write_pixels; local->write_pixels = orig->write_pixels;

77
ui/vnc.c
View File

@ -436,8 +436,8 @@ static void vnc_dpy_update(DisplayChangeListener *dcl,
int i; int i;
VncDisplay *vd = container_of(dcl, VncDisplay, dcl); VncDisplay *vd = container_of(dcl, VncDisplay, dcl);
struct VncSurface *s = &vd->guest; struct VncSurface *s = &vd->guest;
int width = ds_get_width(ds); int width = surface_width(vd->ds);
int height = ds_get_height(ds); int height = surface_height(vd->ds);
h += y; h += y;
@ -519,17 +519,17 @@ void buffer_advance(Buffer *buf, size_t len)
static void vnc_desktop_resize(VncState *vs) static void vnc_desktop_resize(VncState *vs)
{ {
DisplayState *ds = vs->ds; DisplaySurface *ds = vs->vd->ds;
if (vs->csock == -1 || !vnc_has_feature(vs, VNC_FEATURE_RESIZE)) { if (vs->csock == -1 || !vnc_has_feature(vs, VNC_FEATURE_RESIZE)) {
return; return;
} }
if (vs->client_width == ds_get_width(ds) && if (vs->client_width == surface_width(ds) &&
vs->client_height == ds_get_height(ds)) { vs->client_height == surface_height(ds)) {
return; return;
} }
vs->client_width = ds_get_width(ds); vs->client_width = surface_width(ds);
vs->client_height = ds_get_height(ds); vs->client_height = surface_height(ds);
vnc_lock_output(vs); vnc_lock_output(vs);
vnc_write_u8(vs, VNC_MSG_SERVER_FRAMEBUFFER_UPDATE); vnc_write_u8(vs, VNC_MSG_SERVER_FRAMEBUFFER_UPDATE);
vnc_write_u8(vs, 0); vnc_write_u8(vs, 0);
@ -575,7 +575,7 @@ void *vnc_server_fb_ptr(VncDisplay *vd, int x, int y)
} }
static void vnc_dpy_switch(DisplayChangeListener *dcl, static void vnc_dpy_switch(DisplayChangeListener *dcl,
DisplayState *ds, DisplayState *dontuse,
DisplaySurface *surface) DisplaySurface *surface)
{ {
VncDisplay *vd = container_of(dcl, VncDisplay, dcl); VncDisplay *vd = container_of(dcl, VncDisplay, dcl);
@ -585,9 +585,10 @@ static void vnc_dpy_switch(DisplayChangeListener *dcl,
/* server surface */ /* server surface */
qemu_pixman_image_unref(vd->server); qemu_pixman_image_unref(vd->server);
vd->ds = surface;
vd->server = pixman_image_create_bits(VNC_SERVER_FB_FORMAT, vd->server = pixman_image_create_bits(VNC_SERVER_FB_FORMAT,
ds_get_width(ds), surface_width(vd->ds),
ds_get_height(ds), surface_height(vd->ds),
NULL, 0); NULL, 0);
/* guest surface */ /* guest surface */
@ -596,8 +597,8 @@ static void vnc_dpy_switch(DisplayChangeListener *dcl,
console_color_init(ds); console_color_init(ds);
#endif #endif
qemu_pixman_image_unref(vd->guest.fb); qemu_pixman_image_unref(vd->guest.fb);
vd->guest.fb = pixman_image_ref(ds->surface->image); vd->guest.fb = pixman_image_ref(surface->image);
vd->guest.format = ds->surface->format; vd->guest.format = surface->format;
memset(vd->guest.dirty, 0xFF, sizeof(vd->guest.dirty)); memset(vd->guest.dirty, 0xFF, sizeof(vd->guest.dirty));
QTAILQ_FOREACH(vs, &vd->clients, next) { QTAILQ_FOREACH(vs, &vd->clients, next) {
@ -739,7 +740,7 @@ static void vnc_copy(VncState *vs, int src_x, int src_y, int dst_x, int dst_y, i
} }
static void vnc_dpy_copy(DisplayChangeListener *dcl, static void vnc_dpy_copy(DisplayChangeListener *dcl,
DisplayState *ds, DisplayState *dontuse,
int src_x, int src_y, int src_x, int src_y,
int dst_x, int dst_y, int w, int h) int dst_x, int dst_y, int w, int h)
{ {
@ -813,7 +814,7 @@ static void vnc_dpy_copy(DisplayChangeListener *dcl,
} }
static void vnc_mouse_set(DisplayChangeListener *dcl, static void vnc_mouse_set(DisplayChangeListener *dcl,
DisplayState *ds, DisplayState *dontuse,
int x, int y, int visible) 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 ??? */
@ -841,7 +842,7 @@ static int vnc_cursor_define(VncState *vs)
} }
static void vnc_dpy_cursor_define(DisplayChangeListener *dcl, static void vnc_dpy_cursor_define(DisplayChangeListener *dcl,
DisplayState *ds, DisplayState *dontuse,
QEMUCursor *c) QEMUCursor *c)
{ {
VncDisplay *vd = vnc_display; VncDisplay *vd = vnc_display;
@ -1463,7 +1464,8 @@ static void check_pointer_type_change(Notifier *notifier, void *data)
vnc_write_u8(vs, 0); vnc_write_u8(vs, 0);
vnc_write_u16(vs, 1); vnc_write_u16(vs, 1);
vnc_framebuffer_update(vs, absolute, 0, vnc_framebuffer_update(vs, absolute, 0,
ds_get_width(vs->ds), ds_get_height(vs->ds), surface_width(vs->vd->ds),
surface_height(vs->vd->ds),
VNC_ENCODING_POINTER_TYPE_CHANGE); VNC_ENCODING_POINTER_TYPE_CHANGE);
vnc_unlock_output(vs); vnc_unlock_output(vs);
vnc_flush(vs); vnc_flush(vs);
@ -1475,6 +1477,8 @@ static void pointer_event(VncState *vs, int button_mask, int x, int y)
{ {
int buttons = 0; int buttons = 0;
int dz = 0; int dz = 0;
int width = surface_width(vs->vd->ds);
int height = surface_height(vs->vd->ds);
if (button_mask & 0x01) if (button_mask & 0x01)
buttons |= MOUSE_EVENT_LBUTTON; buttons |= MOUSE_EVENT_LBUTTON;
@ -1488,10 +1492,8 @@ static void pointer_event(VncState *vs, int button_mask, int x, int y)
dz = 1; dz = 1;
if (vs->absolute) { if (vs->absolute) {
kbd_mouse_event(ds_get_width(vs->ds) > 1 ? kbd_mouse_event(width > 1 ? x * 0x7FFF / (width - 1) : 0x4000,
x * 0x7FFF / (ds_get_width(vs->ds) - 1) : 0x4000, height > 1 ? y * 0x7FFF / (height - 1) : 0x4000,
ds_get_height(vs->ds) > 1 ?
y * 0x7FFF / (ds_get_height(vs->ds) - 1) : 0x4000,
dz, buttons); dz, buttons);
} else if (vnc_has_feature(vs, VNC_FEATURE_POINTER_TYPE_CHANGE)) { } else if (vnc_has_feature(vs, VNC_FEATURE_POINTER_TYPE_CHANGE)) {
x -= 0x7FFF; x -= 0x7FFF;
@ -1781,12 +1783,15 @@ static void framebuffer_update_request(VncState *vs, int incremental,
int w, int h) int w, int h)
{ {
int i; int i;
const size_t width = ds_get_width(vs->ds) / 16; const size_t width = surface_width(vs->vd->ds) / 16;
const size_t height = surface_height(vs->vd->ds);
if (y_position > ds_get_height(vs->ds)) if (y_position > height) {
y_position = ds_get_height(vs->ds); y_position = height;
if (y_position + h >= ds_get_height(vs->ds)) }
h = ds_get_height(vs->ds) - y_position; if (y_position + h >= height) {
h = height - y_position;
}
vs->need_update = 1; vs->need_update = 1;
if (!incremental) { if (!incremental) {
@ -1805,7 +1810,9 @@ static void send_ext_key_event_ack(VncState *vs)
vnc_write_u8(vs, VNC_MSG_SERVER_FRAMEBUFFER_UPDATE); vnc_write_u8(vs, VNC_MSG_SERVER_FRAMEBUFFER_UPDATE);
vnc_write_u8(vs, 0); vnc_write_u8(vs, 0);
vnc_write_u16(vs, 1); vnc_write_u16(vs, 1);
vnc_framebuffer_update(vs, 0, 0, ds_get_width(vs->ds), ds_get_height(vs->ds), vnc_framebuffer_update(vs, 0, 0,
surface_width(vs->vd->ds),
surface_height(vs->vd->ds),
VNC_ENCODING_EXT_KEY_EVENT); VNC_ENCODING_EXT_KEY_EVENT);
vnc_unlock_output(vs); vnc_unlock_output(vs);
vnc_flush(vs); vnc_flush(vs);
@ -1817,7 +1824,9 @@ static void send_ext_audio_ack(VncState *vs)
vnc_write_u8(vs, VNC_MSG_SERVER_FRAMEBUFFER_UPDATE); vnc_write_u8(vs, VNC_MSG_SERVER_FRAMEBUFFER_UPDATE);
vnc_write_u8(vs, 0); vnc_write_u8(vs, 0);
vnc_write_u16(vs, 1); vnc_write_u16(vs, 1);
vnc_framebuffer_update(vs, 0, 0, ds_get_width(vs->ds), ds_get_height(vs->ds), vnc_framebuffer_update(vs, 0, 0,
surface_width(vs->vd->ds),
surface_height(vs->vd->ds),
VNC_ENCODING_AUDIO); VNC_ENCODING_AUDIO);
vnc_unlock_output(vs); vnc_unlock_output(vs);
vnc_flush(vs); vnc_flush(vs);
@ -1990,8 +1999,10 @@ static void vnc_colordepth(VncState *vs)
vnc_write_u8(vs, VNC_MSG_SERVER_FRAMEBUFFER_UPDATE); vnc_write_u8(vs, VNC_MSG_SERVER_FRAMEBUFFER_UPDATE);
vnc_write_u8(vs, 0); vnc_write_u8(vs, 0);
vnc_write_u16(vs, 1); /* number of rects */ vnc_write_u16(vs, 1); /* number of rects */
vnc_framebuffer_update(vs, 0, 0, ds_get_width(vs->ds), vnc_framebuffer_update(vs, 0, 0,
ds_get_height(vs->ds), VNC_ENCODING_WMVi); surface_width(vs->vd->ds),
surface_height(vs->vd->ds),
VNC_ENCODING_WMVi);
pixel_format_message(vs); pixel_format_message(vs);
vnc_unlock_output(vs); vnc_unlock_output(vs);
vnc_flush(vs); vnc_flush(vs);
@ -2207,8 +2218,8 @@ static int protocol_client_init(VncState *vs, uint8_t *data, size_t len)
} }
vnc_set_share_mode(vs, mode); vnc_set_share_mode(vs, mode);
vs->client_width = ds_get_width(vs->ds); vs->client_width = surface_width(vs->vd->ds);
vs->client_height = ds_get_height(vs->ds); vs->client_height = surface_height(vs->vd->ds);
vnc_write_u16(vs, vs->client_width); vnc_write_u16(vs, vs->client_width);
vnc_write_u16(vs, vs->client_height); vnc_write_u16(vs, vs->client_height);
@ -2686,7 +2697,7 @@ static void vnc_init_timer(VncDisplay *vd)
vd->timer_interval = VNC_REFRESH_INTERVAL_BASE; vd->timer_interval = VNC_REFRESH_INTERVAL_BASE;
if (vd->timer == NULL && !QTAILQ_EMPTY(&vd->clients)) { if (vd->timer == NULL && !QTAILQ_EMPTY(&vd->clients)) {
vd->timer = qemu_new_timer_ms(rt_clock, vnc_refresh, vd); vd->timer = qemu_new_timer_ms(rt_clock, vnc_refresh, vd);
vnc_dpy_switch(&vd->dcl, vd->ds, vd->ds->surface); vga_hw_update();
vnc_refresh(vd); vnc_refresh(vd);
} }
} }
@ -2756,7 +2767,6 @@ void vnc_init_state(VncState *vs)
vs->initialized = true; vs->initialized = true;
VncDisplay *vd = vs->vd; VncDisplay *vd = vs->vd;
vs->ds = vd->ds;
vs->last_x = -1; vs->last_x = -1;
vs->last_y = -1; vs->last_y = -1;
@ -2843,7 +2853,6 @@ void vnc_display_init(DisplayState *ds)
vs->lwebsock = -1; vs->lwebsock = -1;
#endif #endif
vs->ds = ds;
QTAILQ_INIT(&vs->clients); QTAILQ_INIT(&vs->clients);
vs->expires = TIME_MAX; vs->expires = TIME_MAX;

3
ui/vnc.h
View File

@ -150,7 +150,7 @@ struct VncDisplay
bool websocket; bool websocket;
char *ws_display; char *ws_display;
#endif #endif
DisplayState *ds; DisplaySurface *ds;
DisplayChangeListener dcl; DisplayChangeListener dcl;
kbd_layout_t *kbd_layout; kbd_layout_t *kbd_layout;
int lock_key_sync; int lock_key_sync;
@ -248,7 +248,6 @@ struct VncState
{ {
int csock; int csock;
DisplayState *ds;
DECLARE_BITMAP(dirty[VNC_MAX_HEIGHT], VNC_DIRTY_BITS); DECLARE_BITMAP(dirty[VNC_MAX_HEIGHT], VNC_DIRTY_BITS);
uint8_t **lossy_rect; /* Not an Array to avoid costly memcpy in uint8_t **lossy_rect; /* Not an Array to avoid costly memcpy in
* vnc-jobs-async.c */ * vnc-jobs-async.c */