From 6ec04a75deb1695820e337abbefe1929d5928f15 Mon Sep 17 00:00:00 2001 From: Keith Packard Date: Fri, 4 Apr 2014 16:28:43 -0700 Subject: dri3: Fix dri3_open API change by adding new dri3_open_client Xwayland will eventually need the current client in dri3_open. Simply changing that API is not an option though as other drivers that implement DRI3 will not have a matching function signature and will crash when called. Add a new dri3_open_client function pointer and bump DRI3_SCREEN_INFO_VERSION so that drivers can be aware of the new function which will be used in preference to the old function when available. Signed-off-by: Keith Packard Reviewed-by: Eric Anhole --- dri3/dri3.h | 14 +++++++++++--- dri3/dri3_screen.c | 16 ++++++++++++++-- 2 files changed, 25 insertions(+), 5 deletions(-) (limited to 'dri3') diff --git a/dri3/dri3.h b/dri3/dri3.h index 1c04cbd2d..edc7fa267 100644 --- a/dri3/dri3.h +++ b/dri3/dri3.h @@ -30,13 +30,17 @@ #include #include -#define DRI3_SCREEN_INFO_VERSION 0 +#define DRI3_SCREEN_INFO_VERSION 1 -typedef int (*dri3_open_proc)(ClientPtr client, - ScreenPtr screen, +typedef int (*dri3_open_proc)(ScreenPtr screen, RRProviderPtr provider, int *fd); +typedef int (*dri3_open_client_proc)(ClientPtr client, + ScreenPtr screen, + RRProviderPtr provider, + int *fd); + typedef PixmapPtr (*dri3_pixmap_from_fd_proc) (ScreenPtr screen, int fd, CARD16 width, @@ -56,6 +60,10 @@ typedef struct dri3_screen_info { dri3_open_proc open; dri3_pixmap_from_fd_proc pixmap_from_fd; dri3_fd_from_pixmap_proc fd_from_pixmap; + + /* Version 1 */ + dri3_open_client_proc open_client; + } dri3_screen_info_rec, *dri3_screen_info_ptr; extern _X_EXPORT Bool diff --git a/dri3/dri3_screen.c b/dri3/dri3_screen.c index bbf1d05a9..6c0c60cbf 100644 --- a/dri3/dri3_screen.c +++ b/dri3/dri3_screen.c @@ -30,6 +30,14 @@ #include #include +static inline Bool has_open(dri3_screen_info_ptr info) { + if (info == NULL) + return FALSE; + + return info->open != NULL || + (info->version >= 1 && info->open_client != NULL); +} + int dri3_open(ClientPtr client, ScreenPtr screen, RRProviderPtr provider, int *fd) { @@ -37,10 +45,14 @@ dri3_open(ClientPtr client, ScreenPtr screen, RRProviderPtr provider, int *fd) dri3_screen_info_ptr info = ds->info; int rc; - if (!info || !info->open) + if (!has_open(info)) return BadMatch; - rc = (*info->open) (client, screen, provider, fd); + if (info->version >= 1 && info->open_client != NULL) + rc = (*info->open_client) (client, screen, provider, fd); + else + rc = (*info->open) (screen, provider, fd); + if (rc != Success) return rc; -- cgit v1.2.3