diff options
author | Julien Cristau <jcristau@debian.org> | 2010-06-20 00:05:40 +0100 |
---|---|---|
committer | Julien Cristau <jcristau@debian.org> | 2010-07-16 20:47:28 +0100 |
commit | e336dc2a8bf08d5774a51220c917a2440778bc6f (patch) | |
tree | 02f86c473a0523a5a09b341029e9361ef5419b21 | |
parent | a2c13f0d6548310e3cd115cf486d3e43edf23dcc (diff) |
Xephyr: start converting hostx.c over to xcb
Reviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>
Signed-off-by: Julien Cristau <jcristau@debian.org>
-rw-r--r-- | configure.ac | 4 | ||||
-rw-r--r-- | hw/kdrive/ephyr/hostx.c | 654 | ||||
-rw-r--r-- | hw/kdrive/ephyr/hostx.h | 7 |
3 files changed, 366 insertions, 299 deletions
diff --git a/configure.ac b/configure.ac index baa0b5c47..c91b7166b 100644 --- a/configure.ac +++ b/configure.ac @@ -2061,12 +2061,12 @@ if test "$KDRIVE" = yes; then AC_DEFINE(KDRIVE_MOUSE, 1, [Enable KDrive mouse driver]) fi - XEPHYR_REQUIRED_LIBS="x11 $LIBXEXT xau xdmcp" + XEPHYR_REQUIRED_LIBS="x11 $LIBXEXT xau xdmcp xcb x11-xcb xcb-shape xcb-aux xcb-image xcb-icccm xcb-shm" if test "x$XV" = xyes; then XEPHYR_REQUIRED_LIBS="$XEPHYR_REQUIRED_LIBS xv" fi if test "x$DRI" = xyes && test "x$GLX" = xyes; then - XEPHYR_REQUIRED_LIBS="$XEPHYR_REQUIRED_LIBS $LIBGL libdrm" + XEPHYR_REQUIRED_LIBS="$XEPHYR_REQUIRED_LIBS $LIBGL libdrm xcb-glx xcb-xf86dri" fi PKG_CHECK_MODULES(XEPHYR, $XEPHYR_REQUIRED_LIBS, [xephyr="yes"], [xephyr="no"]) diff --git a/hw/kdrive/ephyr/hostx.c b/hw/kdrive/ephyr/hostx.c index 2ebeca958..62e67e407 100644 --- a/hw/kdrive/ephyr/hostx.c +++ b/hw/kdrive/ephyr/hostx.c @@ -53,20 +53,22 @@ #include <X11/Xlib.h> #include <X11/Xutil.h> -#include <X11/Xatom.h> #include <X11/keysym.h> -#include <X11/extensions/XShm.h> #include <X11/extensions/shape.h> +#include <xcb/xcb.h> +#include <xcb/xproto.h> +#include <X11/Xlib-xcb.h> +#include <xcb/xcb_icccm.h> +#include <xcb/xcb_aux.h> +#include <xcb/shm.h> +#include <xcb/xcb_image.h> +#include <xcb/shape.h> #ifdef XF86DRI -#include <GL/glx.h> +#include <xcb/xf86dri.h> +#include <xcb/glx.h> #endif /* XF86DRI */ #include "ephyrlog.h" -#ifdef XF86DRI -extern Bool XF86DRIQueryExtension (Display *dpy, - int *event_basep, - int *error_basep); -#endif /* * All xlib calls go here, which gets built as its own .a . @@ -79,11 +81,11 @@ struct EphyrHostScreen Window win; Window win_pre_existing; /* Set via -parent option like xnest */ Window peer_win; /* Used for GL; should be at most one */ - XImage *ximg; + xcb_image_t *ximg; int win_width, win_height; int server_depth; unsigned char *fb_data; /* only used when host bpp != server bpp */ - XShmSegmentInfo shminfo; + xcb_shm_segment_info_t shminfo; void *info; /* Pointer to the screen this is associated with */ int mynum; /* Screen number */ @@ -93,10 +95,11 @@ struct EphyrHostXVars { char *server_dpy_name; Display *dpy; + xcb_connection_t *conn; int screen; Visual *visual; Window winroot; - GC gc; + xcb_gcontext_t gc; int depth; Bool use_host_cursor; Bool use_fullscreen; @@ -127,11 +130,6 @@ char *ephyrTitle = NULL; static void hostx_set_fullscreen_hint(void); -/* X Error traps */ - -static int trapped_error_code = 0; -static int (*old_error_handler) (Display *d, XErrorEvent *e); - #define host_depth_matches_server(_vars) (HostX.depth == (_vars)->server_depth) static struct EphyrHostScreen * @@ -149,28 +147,6 @@ host_screen_from_screen_info (EphyrScreenInfo *screen) return NULL; } -static int -error_handler(Display *display, - XErrorEvent *error) -{ - trapped_error_code = error->error_code; - return 0; -} - -static void -hostx_errors_trap(void) -{ - trapped_error_code = 0; - old_error_handler = XSetErrorHandler(error_handler); -} - -static int -hostx_errors_untrap(void) -{ - XSetErrorHandler(old_error_handler); - return trapped_error_code; -} - int hostx_want_screen_size (EphyrScreenInfo screen, int *width, int *height ) { @@ -229,7 +205,12 @@ hostx_set_win_title (EphyrScreenInfo screen, char *extra_text) return; if (ephyrTitle) { - XStoreName(HostX.dpy, host_screen->win, ephyrTitle); + xcb_set_wm_name(HostX.conn, + host_screen->win, + XCB_ATOM_STRING, + strlen(ephyrTitle), + ephyrTitle); + } else { #define BUF_LEN 256 char buf[BUF_LEN+1]; @@ -240,7 +221,11 @@ hostx_set_win_title (EphyrScreenInfo screen, char *extra_text) host_screen->mynum, (extra_text != NULL) ? extra_text : ""); - XStoreName (HostX.dpy, host_screen->win, buf); + xcb_set_wm_name(HostX.conn, + host_screen->win, + XCB_ATOM_STRING, + strlen(buf), + buf); } } @@ -283,24 +268,33 @@ hostx_want_fullscreen (void) return HostX.use_fullscreen; } +static xcb_intern_atom_cookie_t cookie_WINDOW_STATE, + cookie_WINDOW_STATE_FULLSCREEN; + static void hostx_set_fullscreen_hint (void) { - Atom atom_WINDOW_STATE, atom_WINDOW_STATE_FULLSCREEN; + xcb_atom_t atom_WINDOW_STATE, atom_WINDOW_STATE_FULLSCREEN; int index; + xcb_intern_atom_reply_t *reply; - atom_WINDOW_STATE - = XInternAtom(HostX.dpy, "_NET_WM_STATE", False); - atom_WINDOW_STATE_FULLSCREEN - = XInternAtom(HostX.dpy, "_NET_WM_STATE_FULLSCREEN",False); + reply = xcb_intern_atom_reply(HostX.conn, cookie_WINDOW_STATE, NULL); + atom_WINDOW_STATE = reply->atom; + free(reply); + reply = xcb_intern_atom_reply(HostX.conn, cookie_WINDOW_STATE_FULLSCREEN, + NULL); + atom_WINDOW_STATE_FULLSCREEN = reply->atom; + free(reply); for (index = 0 ; index < HostX.n_screens ; index++) - { - XChangeProperty (HostX.dpy, HostX.screens[index].win, - atom_WINDOW_STATE, XA_ATOM, 32, - PropModeReplace, - (unsigned char *)&atom_WINDOW_STATE_FULLSCREEN, 1); - } + xcb_change_property(HostX.conn, + PropModeReplace, + HostX.screens[index].win, + atom_WINDOW_STATE, + XCB_ATOM_ATOM, + 32, + 1, + &atom_WINDOW_STATE_FULLSCREEN); } @@ -334,21 +328,25 @@ hostx_set_title (char *title) int hostx_init (void) { - XSetWindowAttributes attr; - Cursor empty_cursor; - Pixmap cursor_pxm; - XColor col; + uint32_t attr; + xcb_cursor_t empty_cursor; + xcb_pixmap_t cursor_pxm; + uint16_t red, green, blue; + uint32_t pixel; int index; - char *tmpstr; - XClassHint *class_hint; - - attr.event_mask = - ButtonPressMask - |ButtonReleaseMask - |PointerMotionMask - |KeyPressMask - |KeyReleaseMask - |ExposureMask; + char *tmpstr; + char *class_hint; + size_t class_len; + const xcb_query_extension_reply_t *shm_rep; + xcb_screen_t *screen; + + attr = + XCB_EVENT_MASK_BUTTON_PRESS + | XCB_EVENT_MASK_BUTTON_RELEASE + | XCB_EVENT_MASK_POINTER_MOTION + | XCB_EVENT_MASK_KEY_PRESS + | XCB_EVENT_MASK_KEY_RELEASE + | XCB_EVENT_MASK_EXPOSURE; EPHYR_DBG("mark"); @@ -358,65 +356,72 @@ hostx_init (void) exit(1); } + HostX.conn = XGetXCBConnection(HostX.dpy); HostX.screen = DefaultScreen(HostX.dpy); + screen = xcb_aux_get_screen(HostX.conn, HostX.screen); HostX.winroot = RootWindow(HostX.dpy, HostX.screen); - HostX.gc = XCreateGC(HostX.dpy, HostX.winroot, 0, NULL); + HostX.gc = xcb_generate_id(HostX.conn); HostX.depth = DefaultDepth(HostX.dpy, HostX.screen); HostX.visual = DefaultVisual(HostX.dpy, HostX.screen); - class_hint = XAllocClassHint(); + xcb_create_gc(HostX.conn, HostX.gc, HostX.winroot, 0, NULL); + cookie_WINDOW_STATE = xcb_intern_atom(HostX.conn, False, strlen("_NET_WM_STATE"), "_NET_WM_STATE"); + cookie_WINDOW_STATE_FULLSCREEN = xcb_intern_atom(HostX.conn, False, strlen("_NET_WM_STATE_FULLSCREEN"), "_NET_WM_STATE_FULLSCREEN"); for (index = 0 ; index < HostX.n_screens ; index++) { struct EphyrHostScreen *host_screen = &HostX.screens[index]; + host_screen->win = xcb_generate_id(HostX.conn); host_screen->server_depth = HostX.depth; - if (host_screen->win_pre_existing != None) + if (host_screen->win_pre_existing != XCB_WINDOW_NONE) { - Status result; - XWindowAttributes prewin_attr; + xcb_get_geometry_reply_t *prewin_geom; + xcb_get_geometry_cookie_t cookie; + xcb_generic_error_t *e = NULL; /* Get screen size from existing window */ + cookie = xcb_get_geometry(HostX.conn, host_screen->win_pre_existing); + prewin_geom = xcb_get_geometry_reply(HostX.conn, cookie, &e); - hostx_errors_trap(); - - result = XGetWindowAttributes (HostX.dpy, - host_screen->win_pre_existing, - &prewin_attr); - - - if (hostx_errors_untrap() || !result) + if (e) { + free(e); + free(prewin_geom); fprintf (stderr, "\nXephyr -parent window' does not exist!\n"); exit (1); } - host_screen->win_width = prewin_attr.width; - host_screen->win_height = prewin_attr.height; - - host_screen->win = XCreateWindow (HostX.dpy, - host_screen->win_pre_existing, - 0,0, - host_screen->win_width, - host_screen->win_height, - 0, - CopyFromParent, - CopyFromParent, - CopyFromParent, - CWEventMask, - &attr); + host_screen->win_width = prewin_geom->width; + host_screen->win_height = prewin_geom->height; + + free(prewin_geom); + + xcb_create_window(HostX.conn, + XCB_COPY_FROM_PARENT, + host_screen->win, + host_screen->win_pre_existing, + 0,0, + host_screen->win_width, + host_screen->win_height, + 0, + XCB_WINDOW_CLASS_COPY_FROM_PARENT, + XCB_COPY_FROM_PARENT, + XCB_CW_EVENT_MASK, + &attr); } else { - host_screen->win = XCreateWindow (HostX.dpy, - HostX.winroot, - 0,0,100,100, /* will resize */ - 0, - CopyFromParent, - CopyFromParent, - CopyFromParent, - CWEventMask, - &attr); + xcb_create_window(HostX.conn, + XCB_COPY_FROM_PARENT, + host_screen->win, + HostX.winroot, + 0,0,100,100, /* will resize */ + 0, + XCB_WINDOW_CLASS_COPY_FROM_PARENT, + XCB_COPY_FROM_PARENT, + XCB_CW_EVENT_MASK, + &attr); hostx_set_win_title (host_screen->info, "(ctrl+shift grabs mouse and keyboard)"); @@ -429,53 +434,82 @@ hostx_init (void) hostx_set_fullscreen_hint(); } - if (class_hint) - { - tmpstr = getenv("RESOURCE_NAME"); - if (tmpstr && (!ephyrResNameFromCmd)) - ephyrResName = tmpstr; - class_hint->res_name = ephyrResName; - class_hint->res_class = "Xephyr"; - XSetClassHint(hostx_get_display(), host_screen->win, class_hint); - - } + tmpstr = getenv("RESOURCE_NAME"); + if (tmpstr && (!ephyrResNameFromCmd)) + ephyrResName = tmpstr; + class_len = strlen(ephyrResName) + 1 + strlen("Xephyr") + 1; + class_hint = malloc(class_len); + if (class_hint) { + strcpy(class_hint, ephyrResName); + strcpy(class_hint + strlen(ephyrResName) + 1, "Xephyr"); + xcb_change_property(HostX.conn, + XCB_PROP_MODE_REPLACE, + host_screen->win, + XCB_ATOM_WM_CLASS, + XCB_ATOM_STRING, + 8, + class_len, + class_hint); + free(class_hint); + } - } + } } - if (class_hint) - XFree(class_hint); - - XParseColor (HostX.dpy, DefaultColormap (HostX.dpy,HostX.screen), - "red", &col); - XAllocColor (HostX.dpy, DefaultColormap (HostX.dpy, HostX.screen), - &col); - XSetForeground (HostX.dpy, HostX.gc, col.pixel); + if (!xcb_aux_parse_color("red", &red, &green, &blue)) { + xcb_lookup_color_cookie_t c = + xcb_lookup_color(HostX.conn, screen->default_colormap, 3, "red"); + xcb_lookup_color_reply_t *reply = + xcb_lookup_color_reply(HostX.conn, c, NULL); + red = reply->exact_red; + green = reply->exact_green; + blue = reply->exact_blue; + free(reply); + } + { + xcb_alloc_color_cookie_t c = xcb_alloc_color(HostX.conn, + screen->default_colormap, + red, green, blue); + xcb_alloc_color_reply_t *r = xcb_alloc_color_reply(HostX.conn, c, NULL); + red = r->red; + green = r->green; + blue = r->blue; + pixel = r->pixel; + free(r); + } + + xcb_change_gc(HostX.conn, HostX.gc, XCB_GC_FOREGROUND, &pixel); if (!hostx_want_host_cursor ()) { /* Ditch the cursor, we provide our 'own' */ - cursor_pxm = XCreatePixmap (HostX.dpy, HostX.winroot, 1, 1, 1); - memset (&col, 0, sizeof (col)); - empty_cursor = XCreatePixmapCursor (HostX.dpy, - cursor_pxm, cursor_pxm, - &col, &col, 1, 1); + cursor_pxm = xcb_generate_id(HostX.conn); + xcb_create_pixmap(HostX.conn, 1, cursor_pxm, HostX.winroot, 1, 1); + empty_cursor = xcb_generate_id(HostX.conn); + xcb_create_cursor(HostX.conn, + empty_cursor, + cursor_pxm, cursor_pxm, + 0,0,0, + 0,0,0, + 1,1); for ( index = 0 ; index < HostX.n_screens ; index++ ) { - XDefineCursor (HostX.dpy, - HostX.screens[index].win, - empty_cursor); + xcb_change_window_attributes(HostX.conn, + HostX.screens[index].win, + XCB_CW_CURSOR, + &empty_cursor); } - XFreePixmap (HostX.dpy, cursor_pxm); + xcb_free_pixmap(HostX.conn, cursor_pxm); } for (index = 0 ; index < HostX.n_screens ; index++) { HostX.screens[index].ximg = NULL; } - /* Try to get share memory ximages for a little bit more speed */ - if (!XShmQueryExtension(HostX.dpy) || getenv("XEPHYR_NO_SHM")) + /* Try to get share memory ximages for a little bit more speed */ + shm_rep = xcb_get_extension_data(HostX.conn, &xcb_shm_id); + if (!shm_rep || !shm_rep->present || getenv("XEPHYR_NO_SHM")) { fprintf(stderr, "\nXephyr unable to use SHM XImages\n"); HostX.have_shm = False; @@ -483,30 +517,31 @@ hostx_init (void) else { /* Really really check we have shm - better way ?*/ - XShmSegmentInfo shminfo; + xcb_shm_segment_info_t shminfo; + xcb_generic_error_t *e; + xcb_void_cookie_t cookie; + xcb_shm_seg_t shmseg; - HostX.have_shm = True; + HostX.have_shm = True; - shminfo.shmid=shmget(IPC_PRIVATE, 1, IPC_CREAT|0777); - shminfo.shmaddr=shmat(shminfo.shmid,0,0); - shminfo.readOnly=True; + shminfo.shmid=shmget(IPC_PRIVATE, 1, IPC_CREAT|0777); + shminfo.shmaddr=shmat(shminfo.shmid,0,0); - hostx_errors_trap(); + shmseg = xcb_generate_id(HostX.conn); + cookie = xcb_shm_attach_checked(HostX.conn, shmseg, shminfo.shmid, True); + e = xcb_request_check(HostX.conn, cookie); - XShmAttach(HostX.dpy, &shminfo); - XSync(HostX.dpy, False); + if (e) { + fprintf(stderr, "\nXephyr unable to use SHM XImages\n"); + HostX.have_shm = False; + free(e); + } - if (hostx_errors_untrap()) - { - fprintf(stderr, "\nXephyr unable to use SHM XImages\n"); - HostX.have_shm = False; - } - - shmdt(shminfo.shmaddr); - shmctl(shminfo.shmid, IPC_RMID, 0); -} + shmdt(shminfo.shmaddr); + shmctl(shminfo.shmid, IPC_RMID, 0); + } - XFlush(HostX.dpy); + xcb_flush(HostX.conn); /* Setup the pause time between paints when debugging updates */ @@ -642,7 +677,6 @@ hostx_screen_init (EphyrScreenInfo screen, { int bitmap_pad; Bool shm_success = False; - XSizeHints *size_hints; struct EphyrHostScreen *host_screen = host_screen_from_screen_info (screen); if (!host_screen) @@ -662,8 +696,8 @@ hostx_screen_init (EphyrScreenInfo screen, if (HostX.have_shm) { - XShmDetach(HostX.dpy, &host_screen->shminfo); - XDestroyImage (host_screen->ximg); + xcb_shm_detach(HostX.conn, host_screen->shminfo.shmseg); + xcb_image_destroy (host_screen->ximg); shmdt(host_screen->shminfo.shmaddr); shmctl(host_screen->shminfo.shmid, IPC_RMID, 0); } @@ -672,35 +706,43 @@ hostx_screen_init (EphyrScreenInfo screen, free(host_screen->ximg->data); host_screen->ximg->data = NULL; - XDestroyImage(host_screen->ximg); + xcb_image_destroy(host_screen->ximg); } } if (HostX.have_shm) { - host_screen->ximg = XShmCreateImage (HostX.dpy, HostX.visual, HostX.depth, - ZPixmap, NULL, &host_screen->shminfo, - width, buffer_height ); + host_screen->ximg = xcb_image_create_native(HostX.conn, + width, + buffer_height, + XCB_IMAGE_FORMAT_Z_PIXMAP, + HostX.depth, + NULL, + ~0, + NULL); host_screen->shminfo.shmid = shmget(IPC_PRIVATE, - host_screen->ximg->bytes_per_line * buffer_height, + host_screen->ximg->stride * buffer_height, IPC_CREAT|0777); host_screen->ximg->data = shmat(host_screen->shminfo.shmid, 0, 0); host_screen->shminfo.shmaddr = host_screen->ximg->data; - if (host_screen->ximg->data == (char *)-1) + if (host_screen->ximg->data == (uint8_t *)-1) { EPHYR_DBG("Can't attach SHM Segment, falling back to plain XImages"); HostX.have_shm = False; - XDestroyImage(host_screen->ximg); + xcb_image_destroy(host_screen->ximg); shmctl(host_screen->shminfo.shmid, IPC_RMID, 0); } else { EPHYR_DBG("SHM segment attached %p", host_screen->shminfo.shmaddr); - host_screen->shminfo.readOnly = False; - XShmAttach(HostX.dpy, &host_screen->shminfo); + host_screen->shminfo.shmseg = xcb_generate_id(HostX.conn); + xcb_shm_attach(HostX.conn, + host_screen->shminfo.shmseg, + host_screen->shminfo.shmid, + False); shm_success = True; } } @@ -711,32 +753,37 @@ hostx_screen_init (EphyrScreenInfo screen, EPHYR_DBG("Creating image %dx%d for screen host_screen=%p\n", width, buffer_height, host_screen ); - host_screen->ximg = XCreateImage (HostX.dpy, - HostX.visual, - HostX.depth, - ZPixmap, 0, 0, - width, - buffer_height, - bitmap_pad, - 0); + host_screen->ximg = xcb_image_create_native(HostX.conn, + width, + buffer_height, + XCB_IMAGE_FORMAT_Z_PIXMAP, + HostX.depth, + NULL, + ~0, + NULL); host_screen->ximg->data = - malloc (host_screen->ximg->bytes_per_line * buffer_height); + malloc (host_screen->ximg->stride * buffer_height); } - XResizeWindow (HostX.dpy, host_screen->win, width, height); - - /* Ask the WM to keep our size static */ - size_hints = XAllocSizeHints(); - size_hints->max_width = size_hints->min_width = width; - size_hints->max_height = size_hints->min_height = height; - size_hints->flags = PMinSize|PMaxSize; - XSetWMNormalHints(HostX.dpy, host_screen->win, size_hints); - XFree(size_hints); - - XMapWindow(HostX.dpy, host_screen->win); - - XSync(HostX.dpy, False); + { + uint32_t cw[2] = { width, height }; + xcb_configure_window(HostX.conn, host_screen->win, + XCB_CONFIG_WINDOW_WIDTH | XCB_CONFIG_WINDOW_HEIGHT, + cw); + } + { + /* Ask the WM to keep our size static */ + xcb_size_hints_t size_hints = {0}; + size_hints.max_width = size_hints.min_width = width; + size_hints.max_height = size_hints.min_height = height; + size_hints.flags = XCB_SIZE_HINT_P_MIN_SIZE | XCB_SIZE_HINT_P_MAX_SIZE; + xcb_set_wm_normal_hints(HostX.conn, host_screen->win, &size_hints); + } + + xcb_map_window(HostX.conn, host_screen->win); + + xcb_aux_sync(HostX.conn); host_screen->win_width = width; host_screen->win_height = height; @@ -814,13 +861,13 @@ hostx_paint_rect (EphyrScreenInfo screen, host_pixel = (r << 16) | (g << 8) | (b); - XPutPixel(host_screen->ximg, x, y, host_pixel); + xcb_image_put_pixel(host_screen->ximg, x, y, host_pixel); break; } case 8: { unsigned char pixel = *(unsigned char*)(host_screen->fb_data+idx); - XPutPixel(host_screen->ximg, x, y, HostX.cmap[pixel]); + xcb_image_put_pixel(host_screen->ximg, x, y, HostX.cmap[pixel]); break; } default: @@ -831,17 +878,18 @@ hostx_paint_rect (EphyrScreenInfo screen, if (HostX.have_shm) { - XShmPutImage (HostX.dpy, host_screen->win, - HostX.gc, host_screen->ximg, - sx, sy, dx, dy, width, height, False); + xcb_image_shm_put(HostX.conn, host_screen->win, + HostX.gc, host_screen->ximg, + host_screen->shminfo, + sx, sy, dx, dy, width, height, False); } else { - XPutImage (HostX.dpy, host_screen->win, HostX.gc, host_screen->ximg, - sx, sy, dx, dy, width, height); + xcb_image_put(HostX.conn, host_screen->win, HostX.gc, host_screen->ximg, + dx, dy, 0); } - XSync (HostX.dpy, False); + xcb_aux_sync(HostX.conn); } static void @@ -850,6 +898,9 @@ hostx_paint_debug_rect (struct EphyrHostScreen *host_screen, int width, int height) { struct timespec tspec; + xcb_rectangle_t rect = { .x = x, .y = y, .width = width, .height = height }; + xcb_void_cookie_t cookie; + xcb_generic_error_t *e; tspec.tv_sec = HostX.damage_debug_msec / (1000000); tspec.tv_nsec = (HostX.damage_debug_msec % 1000000) * 1000; @@ -859,8 +910,10 @@ hostx_paint_debug_rect (struct EphyrHostScreen *host_screen, /* fprintf(stderr, "Xephyr updating: %i+%i %ix%i\n", x, y, width, height); */ - XFillRectangle (HostX.dpy, host_screen->win, HostX.gc, x, y, width,height); - XSync (HostX.dpy, False); + cookie = xcb_poly_fill_rectangle_checked(HostX.conn, host_screen->win, + HostX.gc, 1, &rect); + e = xcb_request_check(HostX.conn, cookie); + free(e); /* nanosleep seems to work better than usleep for me... */ nanosleep(&tspec, NULL); @@ -869,18 +922,22 @@ hostx_paint_debug_rect (struct EphyrHostScreen *host_screen, void hostx_load_keymap(void) { - XID *keymap; + xcb_keysym_t *keymap; int host_width, min_keycode, max_keycode, width; int i,j; + xcb_get_keyboard_mapping_cookie_t cookie; + xcb_get_keyboard_mapping_reply_t *reply; XDisplayKeycodes (HostX.dpy, &min_keycode, &max_keycode); EPHYR_DBG ("min: %d, max: %d", min_keycode, max_keycode); - keymap = XGetKeyboardMapping (HostX.dpy, - min_keycode, - max_keycode - min_keycode + 1, - &host_width); + cookie = xcb_get_keyboard_mapping(HostX.conn, + min_keycode, + max_keycode - min_keycode + 1); + reply = xcb_get_keyboard_mapping_reply(HostX.conn, cookie, NULL); + host_width = reply->keysyms_per_keycode; + keymap = xcb_get_keyboard_mapping_keysyms(reply); /* Try and copy the hosts keymap into our keymap to avoid loads * of messing around. @@ -906,7 +963,7 @@ hostx_load_keymap(void) ephyrKeySyms.maxKeyCode = max_keycode; ephyrKeySyms.mapWidth = width; - XFree(keymap); + free(reply); } static struct EphyrHostScreen * @@ -1065,6 +1122,18 @@ hostx_get_display(void) return HostX.dpy ; } +xcb_connection_t * +hostx_get_xcbconn(void) +{ + return HostX.conn; +} + +int +hostx_get_screen(void) +{ + return DefaultScreen(HostX.dpy); +} + int hostx_get_window (int a_screen_number) { @@ -1078,21 +1147,24 @@ hostx_get_window (int a_screen_number) int hostx_get_window_attributes (int a_window, EphyrHostWindowAttributes *a_attrs) { - XWindowAttributes attrs ; - - memset (&attrs, 0, sizeof (attrs)) ; - - if (!XGetWindowAttributes (hostx_get_display (), - a_window, - &attrs)) { - return FALSE ; - } - a_attrs->x = attrs.x ; - a_attrs->y = attrs.y ; - a_attrs->width = attrs.width ; - a_attrs->height = attrs.height ; - if (attrs.visual) - a_attrs->visualid = attrs.visual->visualid ; + xcb_get_geometry_cookie_t geom_cookie; + xcb_get_window_attributes_cookie_t attr_cookie; + xcb_get_geometry_reply_t *geom_reply; + xcb_get_window_attributes_reply_t *attr_reply; + + geom_cookie = xcb_get_geometry(HostX.conn, a_window); + attr_cookie = xcb_get_window_attributes(HostX.conn, a_window); + geom_reply = xcb_get_geometry_reply(HostX.conn, geom_cookie, NULL); + attr_reply = xcb_get_window_attributes_reply(HostX.conn, attr_cookie, NULL); + + a_attrs->x = geom_reply->x; + a_attrs->y = geom_reply->y; + a_attrs->width = geom_reply->width; + a_attrs->height = geom_reply->height; + a_attrs->visualid = attr_reply->visual; + + free(geom_reply); + free(attr_reply); return TRUE ; } @@ -1171,60 +1243,57 @@ hostx_create_window (int a_screen_number, int *a_host_peer /*out parameter*/) { Bool is_ok=FALSE ; - Display *dpy=hostx_get_display () ; - XVisualInfo *visual_info=NULL, visual_info_templ; - int visual_mask=VisualIDMask ; - Window win=None ; - int nb_visuals=0, winmask=0; - XSetWindowAttributes attrs; - - EPHYR_RETURN_VAL_IF_FAIL (dpy && a_geometry, FALSE) ; + xcb_window_t win; + int winmask=0; + uint32_t attrs[2]; + xcb_screen_t *screen = xcb_aux_get_screen(HostX.conn, hostx_get_screen()); + xcb_visualtype_t *visual; + int depth = 0; + EPHYR_RETURN_VAL_IF_FAIL (screen && a_geometry, FALSE) ; EPHYR_LOG ("enter\n") ; - /*get visual*/ - memset (&visual_info, 0, sizeof (visual_info)) ; - visual_info_templ.visualid = a_visual_id ; - visual_info = XGetVisualInfo (dpy, visual_mask, - &visual_info_templ, - &nb_visuals) ; - if (!visual_info) { + visual = xcb_aux_find_visual_by_id(screen, a_visual_id); + if (!visual) { EPHYR_LOG_ERROR ("argh, could not find a remote visual with id:%d\n", - a_visual_id) ; - goto out ; - } - memset (&attrs, 0, sizeof (attrs)) ; - attrs.colormap = XCreateColormap (dpy, - RootWindow (dpy, - visual_info->screen), - visual_info->visual, - AllocNone) ; - attrs.event_mask = ButtonPressMask - |ButtonReleaseMask - |PointerMotionMask - |KeyPressMask - |KeyReleaseMask - |ExposureMask; - winmask = CWColormap|CWEventMask; - - win = XCreateWindow (dpy, hostx_get_window (a_screen_number), - a_geometry->x, a_geometry->y, - a_geometry->width, a_geometry->height, 0, - visual_info->depth, CopyFromParent, - visual_info->visual, winmask, &attrs) ; - if (win == None) { - EPHYR_LOG_ERROR ("failed to create peer window\n") ; - goto out ; + a_visual_id); + goto out; } - if (HostX.screens[a_screen_number].peer_win == None) { + depth = xcb_aux_get_depth_of_visual(screen, a_visual_id); + + winmask = XCB_CW_EVENT_MASK | XCB_CW_COLORMAP; + attrs[0] = XCB_EVENT_MASK_BUTTON_PRESS + |XCB_EVENT_MASK_BUTTON_RELEASE + |XCB_EVENT_MASK_POINTER_MOTION + |XCB_EVENT_MASK_KEY_PRESS + |XCB_EVENT_MASK_KEY_RELEASE + |XCB_EVENT_MASK_EXPOSURE; + attrs[1] = xcb_generate_id(HostX.conn); + xcb_create_colormap(HostX.conn, + XCB_COLORMAP_ALLOC_NONE, + attrs[1], + hostx_get_window(a_screen_number), + a_visual_id); + + win = xcb_generate_id(HostX.conn); + xcb_create_window(HostX.conn, + depth, + win, + hostx_get_window (a_screen_number), + a_geometry->x, a_geometry->y, + a_geometry->width, a_geometry->height, 0, + XCB_WINDOW_CLASS_COPY_FROM_PARENT, + a_visual_id, winmask, attrs); + + if (HostX.screens[a_screen_number].peer_win == XCB_NONE) { HostX.screens[a_screen_number].peer_win = win; } else { EPHYR_LOG_ERROR ("multiple peer windows created for same screen\n") ; } - XFlush (dpy) ; - XMapWindow (dpy, win) ; - *a_host_peer = win ; - is_ok = TRUE ; + xcb_flush(HostX.conn); + xcb_map_window(HostX.conn, win); + *a_host_peer = win; + is_ok = TRUE; out: EPHYR_LOG ("leave\n") ; return is_ok ; @@ -1233,27 +1302,32 @@ out: int hostx_destroy_window (int a_win) { - Display *dpy=hostx_get_display () ; - - EPHYR_RETURN_VAL_IF_FAIL (dpy, FALSE) ; - XDestroyWindow (dpy, a_win) ; - XFlush (dpy) ; + xcb_destroy_window(HostX.conn, a_win) ; + xcb_flush(HostX.conn); return TRUE ; } int hostx_set_window_geometry (int a_win, EphyrBox *a_geo) { - Display *dpy=hostx_get_display (); - - EPHYR_RETURN_VAL_IF_FAIL (dpy && a_geo, FALSE) ; + uint32_t mask = XCB_CONFIG_WINDOW_X + | XCB_CONFIG_WINDOW_Y + | XCB_CONFIG_WINDOW_WIDTH + | XCB_CONFIG_WINDOW_HEIGHT; + uint32_t values[4]; + + EPHYR_RETURN_VAL_IF_FAIL (a_geo, FALSE) ; EPHYR_LOG ("enter. x,y,w,h:(%d,%d,%d,%d)\n", a_geo->x, a_geo->y, a_geo->width, a_geo->height) ; - XMoveWindow (dpy, a_win, a_geo->x, a_geo->y) ; - XResizeWindow (dpy, a_win, a_geo->width, a_geo->height) ; + values[0] = a_geo->x; + values[1] = a_geo->y; + values[2] = a_geo->width; + values[3] = a_geo->height; + xcb_configure_window(HostX.conn, a_win, mask, values); + EPHYR_LOG ("leave\n") ; return TRUE; } @@ -1331,14 +1405,10 @@ hostx_set_window_clipping_rectangles (int a_window, int hostx_has_xshape (void) { - int event_base=0, error_base=0 ; - Display *dpy=hostx_get_display () ; - if (!XShapeQueryExtension (dpy, - &event_base, - &error_base)) { - return FALSE ; - } - return TRUE; + const xcb_query_extension_reply_t *rep; + + rep = xcb_get_extension_data(HostX.conn, &xcb_shape_id); + return rep && rep->present; } #ifdef XF86DRI @@ -1358,7 +1428,6 @@ hostx_allocate_resource_id_peer (int a_local_resource_id, { int i=0 ; ResourcePair *peer=NULL ; - Display *dpy=hostx_get_display (); /* * first make sure a resource peer @@ -1383,7 +1452,7 @@ hostx_allocate_resource_id_peer (int a_local_resource_id, } } if (peer) { - peer->remote_id = XAllocID (dpy); + peer->remote_id = xcb_generate_id(HostX.conn); peer->local_id = a_local_resource_id ; peer->is_valid = TRUE ; } @@ -1418,30 +1487,21 @@ hostx_get_resource_id_peer (int a_local_resource_id, int hostx_has_dri (void) { - int event_base=0, error_base=0 ; - Display *dpy=hostx_get_display () ; - - if (!dpy) - return FALSE ; - - if (!XF86DRIQueryExtension (dpy, - &event_base, - &error_base)) { - return FALSE ; - } - return TRUE ; + const xcb_query_extension_reply_t *dri; + + dri = xcb_get_extension_data(HostX.conn, &xcb_xf86dri_id); + + return dri && dri->present; } int hostx_has_glx (void) { - Display *dpy=hostx_get_display () ; - int event_base=0, error_base=0 ; + const xcb_query_extension_reply_t *glx; - if (!glXQueryExtension (dpy, &event_base, &error_base)) { - return FALSE ; - } - return TRUE ; + glx = xcb_get_extension_data(HostX.conn, &xcb_glx_id); + + return glx && glx->present; } #endif /* XF86DRI */ diff --git a/hw/kdrive/ephyr/hostx.h b/hw/kdrive/ephyr/hostx.h index e65e0c9bc..8694f9a0d 100644 --- a/hw/kdrive/ephyr/hostx.h +++ b/hw/kdrive/ephyr/hostx.h @@ -28,6 +28,7 @@ #include <X11/X.h> #include <X11/Xmd.h> +#include <xcb/xcb.h> #define EPHYR_WANT_DEBUG 0 @@ -213,6 +214,12 @@ hostx_get_event (EphyrHostXEvent *ev); void* hostx_get_display (void) ; +xcb_connection_t * +hostx_get_xcbconn(void); + +int +hostx_get_screen(void); + int hostx_get_window (int a_screen_number) ; |