From 66e100ffbffbb70b64b973a4606d7c7b857c19ce Mon Sep 17 00:00:00 2001 From: Simon Ser Date: Tue, 6 Apr 2021 11:36:15 +0200 Subject: [PATCH] render/gbm_allocator: add support for gbm_bo_get_fd_for_plane See [1]. This allows us to remove the workaround for GBM API limitations. [1]: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5442 --- render/gbm_allocator.c | 8 ++++++-- render/meson.build | 3 +++ 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/render/gbm_allocator.c b/render/gbm_allocator.c index 4006b66d..cbdd2640 100644 --- a/render/gbm_allocator.c +++ b/render/gbm_allocator.c @@ -35,12 +35,14 @@ static bool export_gbm_bo(struct gbm_bo *bo, int i; int32_t handle = -1; for (i = 0; i < attribs.n_planes; ++i) { +#if HAS_GBM_BO_GET_FD_FOR_PLANE + attribs.fd[i] = gbm_bo_get_fd_for_plane(bo, i); + (void)handle; +#else // GBM is lacking a function to get a FD for a given plane. Instead, // check all planes have the same handle. We can't use // drmPrimeHandleToFD because that messes up handle ref'counting in // the user-space driver. - // TODO: use gbm_bo_get_plane_fd when it lands, see - // https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5442 union gbm_bo_handle plane_handle = gbm_bo_get_handle_for_plane(bo, i); if (plane_handle.s32 < 0) { wlr_log(WLR_ERROR, "gbm_bo_get_handle_for_plane failed"); @@ -55,6 +57,8 @@ static bool export_gbm_bo(struct gbm_bo *bo, } attribs.fd[i] = gbm_bo_get_fd(bo); +#endif + if (attribs.fd[i] < 0) { wlr_log(WLR_ERROR, "gbm_bo_get_fd failed"); goto error_fd; diff --git a/render/meson.build b/render/meson.build index 6d185113..8c1793f1 100644 --- a/render/meson.build +++ b/render/meson.build @@ -18,6 +18,9 @@ wlr_files += files( 'drm_dumb_allocator.c', ) +has = cc.has_function('gbm_bo_get_fd_for_plane', dependencies: [gbm]) +add_project_arguments('-DHAS_GBM_BO_GET_FD_FOR_PLANE=@0@'.format(has.to_int()), language: 'c') + egl = dependency('egl', required: 'gles2' in renderers) if egl.found() wlr_deps += egl