]> git.sesse.net Git - vlc/blobdiff - modules/video_output/xcb/glx.c
XCB: reset the X11 screen saver when displaying a picture
[vlc] / modules / video_output / xcb / glx.c
index aada03cb5aab20d8e3b664d634eac4cff1a1f5af..42ff5cf111a5de4fc69343c73b7d5b14f186c820 100644 (file)
@@ -34,6 +34,7 @@
 
 #include <vlc_common.h>
 #include <vlc_plugin.h>
+#include <vlc_xlib.h>
 #include <vlc_vout_display.h>
 #include <vlc_vout_opengl.h>
 #include "../opengl.h"
@@ -51,11 +52,10 @@ vlc_module_begin ()
     set_description (N_("GLX video output (XCB)"))
     set_category (CAT_VIDEO)
     set_subcategory (SUBCAT_VIDEO_VOUT)
-    set_capability ("vout display", 20)
+    set_capability ("vout display", 50)
     set_callbacks (Open, Close)
 
-    add_shortcut ("xcb-glx")
-    add_shortcut ("glx")
+    add_shortcut ("xcb-glx", "glx", "opengl", "xid")
 vlc_module_end ()
 
 struct vout_display_sys_t
@@ -65,7 +65,9 @@ struct vout_display_sys_t
 
     xcb_cursor_t cursor; /* blank cursor */
     xcb_window_t window; /* drawable X window */
+    xcb_window_t glwin; /* GLX window */
     bool visible; /* whether to draw */
+    bool v1_3; /* whether GLX >= 1.3 is available */
 
     GLXContext ctx;
     vout_opengl_t gl;
@@ -73,7 +75,7 @@ struct vout_display_sys_t
     picture_pool_t *pool; /* picture pool */
 };
 
-static picture_t *Get (vout_display_t *);
+static picture_pool_t *Pool (vout_display_t *, unsigned);
 static void PictureRender (vout_display_t *, picture_t *);
 static void PictureDisplay (vout_display_t *, picture_t *);
 static int Control (vout_display_t *, int, va_list);
@@ -87,6 +89,8 @@ static vout_window_t *MakeWindow (vout_display_t *vd)
 
     memset (&wnd_cfg, 0, sizeof (wnd_cfg));
     wnd_cfg.type = VOUT_WINDOW_TYPE_XID;
+    wnd_cfg.x = var_InheritInteger (vd, "video-x");
+    wnd_cfg.y = var_InheritInteger (vd, "video-y");
     wnd_cfg.width  = vd->cfg->display.width;
     wnd_cfg.height = vd->cfg->display.height;
 
@@ -98,13 +102,14 @@ static vout_window_t *MakeWindow (vout_display_t *vd)
 
 static const xcb_screen_t *
 FindWindow (vout_display_t *vd, xcb_connection_t *conn,
-            unsigned *restrict pnum, uint8_t *restrict pdepth)
+            unsigned *restrict pnum, uint8_t *restrict pdepth,
+            uint16_t *restrict pwidth, uint16_t *restrict pheight)
 {
     vout_display_sys_t *sys = vd->sys;
 
     xcb_get_geometry_reply_t *geo =
         xcb_get_geometry_reply (conn,
-            xcb_get_geometry (conn, sys->embed->xid), NULL);
+            xcb_get_geometry (conn, sys->embed->handle.xid), NULL);
     if (geo == NULL)
     {
         msg_Err (vd, "parent window not valid");
@@ -113,6 +118,8 @@ FindWindow (vout_display_t *vd, xcb_connection_t *conn,
 
     xcb_window_t root = geo->root;
     *pdepth = geo->depth;
+    *pwidth = geo->width;
+    *pheight = geo->height;
     free (geo);
 
     /* Find the selected screen */
@@ -142,7 +149,7 @@ FindWindow (vout_display_t *vd, xcb_connection_t *conn,
     return screen;
 }
 
-static bool CheckGLX (vout_display_t *vd, Display *dpy)
+static bool CheckGLX (vout_display_t *vd, Display *dpy, bool *restrict pv13)
 {
     int major, minor;
     bool ok = false;
@@ -159,15 +166,45 @@ static bool CheckGLX (vout_display_t *vd, Display *dpy)
     {
         msg_Dbg (vd, "using GLX extension version %d.%d", major, minor);
         ok = true;
+        *pv13 = minor >= 3;
     }
     return ok;
 }
 
+static int CreateWindow (vout_display_t *vd, xcb_connection_t *conn,
+                         uint_fast8_t depth, xcb_visualid_t vid,
+                         uint_fast16_t width, uint_fast16_t height)
+{
+    vout_display_sys_t *sys = vd->sys;
+    const uint32_t mask = XCB_CW_EVENT_MASK;
+    const uint32_t values[] = {
+        /* XCB_CW_EVENT_MASK */
+        XCB_EVENT_MASK_VISIBILITY_CHANGE,
+    };
+    xcb_void_cookie_t cc, cm;
+
+    cc = xcb_create_window_checked (conn, depth, sys->window,
+                                    sys->embed->handle.xid, 0, 0,
+                                    width, height, 0,
+                                    XCB_WINDOW_CLASS_INPUT_OUTPUT,
+                                    vid, mask, values);
+    cm = xcb_map_window_checked (conn, sys->window);
+    if (CheckError (vd, conn, "cannot create X11 window", cc)
+     || CheckError (vd, conn, "cannot map X11 window", cm))
+        return VLC_EGENERIC;
+
+    msg_Dbg (vd, "using X11 window %08"PRIx32, sys->window);
+    return VLC_SUCCESS;
+}
+
 /**
  * Probe the X server.
  */
 static int Open (vlc_object_t *obj)
 {
+    if (!vlc_xlib_init (obj))
+        return VLC_EGENERIC;
+
     vout_display_t *vd = (vout_display_t *)obj;
     vout_display_sys_t *sys = malloc (sizeof (*sys));
 
@@ -187,7 +224,7 @@ static int Open (vlc_object_t *obj)
     }
 
     /* Connect to X server */
-    Display *dpy = XOpenDisplay (sys->embed->x11_display);
+    Display *dpy = XOpenDisplay (sys->embed->display.x11);
     if (dpy == NULL)
     {
         vout_display_DeleteWindow (vd, sys->embed);
@@ -198,29 +235,106 @@ static int Open (vlc_object_t *obj)
     sys->ctx = NULL;
     XSetEventQueueOwner (dpy, XCBOwnsEventQueue);
 
-    if (!CheckGLX (vd, dpy))
+    if (!CheckGLX (vd, dpy, &sys->v1_3))
         goto error;
 
     xcb_connection_t *conn = XGetXCBConnection (dpy);
     assert (conn);
-    RegisterMouseEvents (obj, conn, sys->embed->xid);
+    RegisterMouseEvents (obj, conn, sys->embed->handle.xid);
 
     /* Find window parameters */
     unsigned snum;
     uint8_t depth;
-    const xcb_screen_t *scr = FindWindow (vd, conn, &snum, &depth);
+    uint16_t width, height;
+    const xcb_screen_t *scr = FindWindow (vd, conn, &snum, &depth,
+                                          &width, &height);
     if (scr == NULL)
         goto error;
 
+    sys->window = xcb_generate_id (conn);
+
     /* Determine our pixel format */
-    {
+    if (sys->v1_3)
+    {   /* GLX 1.3 */
+        static const int attr[] = {
+            GLX_RED_SIZE, 5,
+            GLX_GREEN_SIZE, 5,
+            GLX_BLUE_SIZE, 5,
+            GLX_DOUBLEBUFFER, True,
+            GLX_X_RENDERABLE, True,
+            GLX_DRAWABLE_TYPE, GLX_WINDOW_BIT,
+            None };
+
+        xcb_get_window_attributes_reply_t *wa =
+            xcb_get_window_attributes_reply (conn,
+                xcb_get_window_attributes (conn, sys->embed->handle.xid),
+                NULL);
+        if (wa == NULL)
+            goto error;
+        xcb_visualid_t visual = wa->visual;
+        free (wa);
+
+        int nelem;
+        GLXFBConfig *confs = glXChooseFBConfig (dpy, snum, attr, &nelem);
+        if (confs == NULL)
+        {
+            msg_Err (vd, "no GLX frame bufer configurations");
+            goto error;
+        }
+
+        GLXFBConfig conf;
+        bool found = false;
+
+        for (int i = 0; i < nelem && !found; i++)
+        {
+            conf = confs[i];
+
+            XVisualInfo *vi = glXGetVisualFromFBConfig (dpy, conf);
+            if (vi == NULL)
+                continue;
+
+            if (vi->visualid == visual)
+                found = true;
+            XFree (vi);
+        }
+        XFree (confs);
+
+        if (!found)
+        {
+            msg_Err (vd, "no matching GLX frame buffer configuration");
+            goto error;
+        }
+
+        sys->glwin = None;
+        if (!CreateWindow (vd, conn, depth, 0 /* ??? */, width, height))
+            sys->glwin = glXCreateWindow (dpy, conf, sys->window, NULL );
+        if (sys->glwin == None)
+        {
+            msg_Err (vd, "cannot create GLX window");
+            goto error;
+        }
+
+        /* Create an OpenGL context */
+        sys->ctx = glXCreateNewContext (dpy, conf, GLX_RGBA_TYPE, NULL,
+                                        True);
+        if (sys->ctx == NULL)
+        {
+            msg_Err (vd, "cannot create GLX context");
+            goto error;
+        }
+
+        if (!glXMakeContextCurrent (dpy, sys->glwin, sys->glwin, sys->ctx))
+            goto error;
+    }
+    else
+    {   /* GLX 1.2 */
         int attr[] = {
             GLX_RGBA,
             GLX_RED_SIZE, 5,
             GLX_GREEN_SIZE, 5,
             GLX_BLUE_SIZE, 5,
             GLX_DOUBLEBUFFER,
-            0 };
+            None };
 
         XVisualInfo *vi = glXChooseVisual (dpy, snum, attr);
         if (vi == NULL)
@@ -228,43 +342,21 @@ static int Open (vlc_object_t *obj)
             msg_Err (vd, "cannot find GLX 1.2 visual" );
             goto error;
         }
+        msg_Dbg (vd, "using GLX visual ID 0x%"PRIx32, (uint32_t)vi->visualid);
 
-        sys->ctx = glXCreateContext (dpy, vi, 0, True);
+        if (CreateWindow (vd, conn, depth, 0 /* ??? */, width, height) == 0)
+            sys->ctx = glXCreateContext (dpy, vi, 0, True);
         XFree (vi);
         if (sys->ctx == NULL)
         {
             msg_Err (vd, "cannot create GLX context");
             goto error;
         }
-    }
 
-    /* Create window */
-    unsigned width, height;
-    if (GetWindowSize (sys->embed, conn, &width, &height))
-        goto error;
-
-    sys->window = xcb_generate_id (conn);
-    {
-        const uint32_t mask = XCB_CW_EVENT_MASK;
-        const uint32_t values[] = {
-            /* XCB_CW_EVENT_MASK */
-            XCB_EVENT_MASK_VISIBILITY_CHANGE,
-        };
-        xcb_void_cookie_t c;
-
-        c = xcb_create_window_checked (conn, depth, sys->window,
-                                       sys->embed->xid, 0, 0, width, height, 0,
-                                       XCB_WINDOW_CLASS_INPUT_OUTPUT,
-                                       0, mask, values);
-        xcb_map_window (conn, sys->window);
-
-        if (CheckError (vd, conn, "cannot create X11 window", c))
+        if (glXMakeCurrent (dpy, sys->window, sys->ctx) == False)
             goto error;
+        sys->glwin = sys->window;
     }
-    msg_Dbg (vd, "using X11 window %08"PRIx32, sys->window);
-
-    if (glXMakeCurrent (dpy, sys->window, sys->ctx) == False)
-        goto error;
 
     /* Initialize common OpenGL video display */
     sys->gl.lock = NULL;
@@ -284,19 +376,23 @@ static int Open (vlc_object_t *obj)
     /* */
     vout_display_info_t info = vd->info;
     info.has_pictures_invalid = false;
+    info.has_event_thread = true;
 
     /* Setup vout_display_t once everything is fine */
     vd->info = info;
 
-    vd->get = Get;
+    vd->pool = Pool;
     vd->prepare = PictureRender;
     vd->display = PictureDisplay;
     vd->control = Control;
     vd->manage = Manage;
 
     /* */
-    vout_display_SendEventFullscreen (vd, false);
-    vout_display_SendEventDisplaySize (vd, width, height, false);
+    bool is_fullscreen = vd->cfg->is_fullscreen;
+    if (is_fullscreen && vout_window_SetFullScreen (sys->embed, true))
+        is_fullscreen = false;
+    vout_display_SendEventFullscreen (vd, is_fullscreen);
+    vout_display_SendEventDisplaySize (vd, width, height, is_fullscreen);
 
     return VLC_SUCCESS;
 
@@ -320,9 +416,21 @@ static void Close (vlc_object_t *obj)
 
     if (sys->ctx != NULL)
     {
-        glXMakeCurrent (dpy, 0, NULL);
+        if (sys->v1_3)
+            glXMakeContextCurrent (dpy, None, None, NULL);
+        else
+            glXMakeCurrent (dpy, None, NULL);
         glXDestroyContext (dpy, sys->ctx);
+        if (sys->v1_3)
+            glXDestroyWindow (dpy, sys->glwin);
     }
+
+    /* show the default cursor */
+    xcb_change_window_attributes (XGetXCBConnection (sys->display),
+                                  sys->embed->handle.xid, XCB_CW_CURSOR,
+                                  &(uint32_t) { XCB_CURSOR_NONE });
+    xcb_flush (XGetXCBConnection (sys->display));
+
     XCloseDisplay (dpy);
     vout_display_DeleteWindow (vd, sys->embed);
     free (sys);
@@ -332,28 +440,25 @@ static void SwapBuffers (vout_opengl_t *gl)
 {
     vout_display_sys_t *sys = gl->sys;
 
-    glXSwapBuffers (sys->display, sys->window);
+    glXSwapBuffers (sys->display, sys->glwin);
 }
 
 /**
  * Return a direct buffer
  */
-static picture_t *Get (vout_display_t *vd)
+static picture_pool_t *Pool (vout_display_t *vd, unsigned requested_count)
 {
     vout_display_sys_t *sys = vd->sys;
+    (void)requested_count;
 
     if (!sys->pool)
-    {
         sys->pool = vout_display_opengl_GetPool (&sys->vgl);
-        if (!sys->pool)
-            return NULL;
-    }
-    return picture_pool_Get (sys->pool);
+    return sys->pool;
 }
 
 static void PictureRender (vout_display_t *vd, picture_t *pic)
 {
-   vout_display_sys_t *sys = vd->sys;
+    vout_display_sys_t *sys = vd->sys;
 
     vout_display_opengl_Prepare (&sys->vgl, pic);
 }
@@ -361,8 +466,11 @@ static void PictureRender (vout_display_t *vd, picture_t *pic)
 static void PictureDisplay (vout_display_t *vd, picture_t *pic)
 {
     vout_display_sys_t *sys = vd->sys;
+    xcb_connection_t *conn = XGetXCBConnection (sys->display);
 
+    xcb_force_screen_saver (conn, XCB_SCREEN_SAVER_RESET);
     vout_display_opengl_Display (&sys->vgl, &vd->source);
+    xcb_flush (conn);
     picture_Release (pic);
 }
 
@@ -378,10 +486,10 @@ static int Control (vout_display_t *vd, int query, va_list ap)
         return vout_window_SetFullScreen (sys->embed, c->is_fullscreen);
     }
 
-    case VOUT_DISPLAY_CHANGE_ON_TOP:
+    case VOUT_DISPLAY_CHANGE_WINDOW_STATE:
     {
-        int b_on_top = (int)va_arg (ap, int);
-        return vout_window_SetOnTop (sys->embed, b_on_top);
+        unsigned state = va_arg (ap, unsigned);
+        return vout_window_SetState (sys->embed, state);
     }
 
     case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
@@ -389,16 +497,68 @@ static int Control (vout_display_t *vd, int query, va_list ap)
     case VOUT_DISPLAY_CHANGE_ZOOM:
     case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
     case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
-        msg_Err (vd, "unimplemented control request");
-        return VLC_EGENERIC;
+    {
+        xcb_connection_t *conn = XGetXCBConnection (sys->display);
+        const vout_display_cfg_t *cfg;
+        const video_format_t *source;
+        bool is_forced = false;
+
+        if (query == VOUT_DISPLAY_CHANGE_SOURCE_ASPECT
+         || query == VOUT_DISPLAY_CHANGE_SOURCE_CROP)
+        {
+            source = (const video_format_t *)va_arg (ap, const video_format_t *);
+            cfg = vd->cfg;
+        }
+        else
+        {
+            source = &vd->source;
+            cfg = (const vout_display_cfg_t*)va_arg (ap, const vout_display_cfg_t *);
+            if (query == VOUT_DISPLAY_CHANGE_DISPLAY_SIZE)
+                is_forced = (bool)va_arg (ap, int);
+        }
+
+        /* */
+        if (query == VOUT_DISPLAY_CHANGE_DISPLAY_SIZE
+         && is_forced
+         && (cfg->display.width  != vd->cfg->display.width
+           ||cfg->display.height != vd->cfg->display.height)
+         && vout_window_SetSize (sys->embed,
+                                 cfg->display.width, cfg->display.height))
+            return VLC_EGENERIC;
+
+        vout_display_place_t place;
+        vout_display_PlacePicture (&place, source, cfg, false);
+
+        /* Move the picture within the window */
+        const uint32_t values[] = { place.x, place.y,
+                                    place.width, place.height, };
+        xcb_void_cookie_t ck =
+            xcb_configure_window_checked (conn, sys->window,
+                            XCB_CONFIG_WINDOW_X | XCB_CONFIG_WINDOW_Y
+                          | XCB_CONFIG_WINDOW_WIDTH | XCB_CONFIG_WINDOW_HEIGHT,
+                              values);
+        if (CheckError (vd, conn, "cannot resize X11 window", ck))
+            return VLC_EGENERIC;
+
+        glViewport (0, 0, place.width, place.height);
+        return VLC_SUCCESS;
+    }
 
     /* Hide the mouse. It will be send when
      * vout_display_t::info.b_hide_mouse is false */
     case VOUT_DISPLAY_HIDE_MOUSE:
         xcb_change_window_attributes (XGetXCBConnection (sys->display),
-                                      sys->embed->xid,
+                                      sys->embed->handle.xid,
                                     XCB_CW_CURSOR, &(uint32_t){ sys->cursor });
         return VLC_SUCCESS;
+
+    case VOUT_DISPLAY_GET_OPENGL:
+    {
+        vout_opengl_t **gl = va_arg (ap, vout_opengl_t **);
+        *gl = &sys->gl;
+        return VLC_SUCCESS;
+    }
+
     case VOUT_DISPLAY_RESET_PICTURES:
         assert (0);
     default: