backend/drm: clarify error message on drmModeAddFB fallback

The previous code would always print "falling back to legacy method",
even if the format wasn't ARGB8888.

Drop get_fb_for_bo_legacy, since the code can just be inlined without
hurting readability.

Ideally we should only fallback to drmModeAddFB if the error code
indicates the BE failure, but the original PR [1] doesn't say what
error code is returned by the kernel.

[1]: https://github.com/swaywm/wlroots/pull/2569
This commit is contained in:
Simon Ser 2021-04-23 16:31:00 +02:00 committed by Kenny Levinsen
parent f9f90b4173
commit e804de923d

View file

@ -172,35 +172,6 @@ const char *conn_get_name(uint32_t type_id) {
}
}
static uint32_t get_fb_for_bo_legacy(struct gbm_bo *bo) {
struct gbm_device *gbm = gbm_bo_get_device(bo);
/* We only support this as a fallback of last resort for ARGB8888 visuals,
* like xf86-video-modesetting does. This is necessary on BE machines. */
if (gbm_bo_get_format(bo) != GBM_FORMAT_ARGB8888 ||
gbm_bo_get_plane_count(bo) != 1) {
wlr_log(WLR_DEBUG,
"Invalid visual %x (%d planes) requested for legacy DRM framebuffer",
gbm_bo_get_format(bo), gbm_bo_get_plane_count(bo));
return 0;
}
int fd = gbm_device_get_fd(gbm);
uint32_t width = gbm_bo_get_width(bo);
uint32_t height = gbm_bo_get_height(bo);
uint32_t depth = 32;
uint32_t bpp = gbm_bo_get_bpp(bo);
uint32_t pitch = gbm_bo_get_stride(bo);
uint32_t handle = gbm_bo_get_handle(bo).u32;
uint32_t id = 0;
if (drmModeAddFB(fd, width, height, depth, bpp, pitch, handle, &id)) {
wlr_log_errno(WLR_ERROR, "Unable to add DRM framebuffer");
}
return id;
}
uint32_t get_fb_for_bo(struct gbm_bo *bo, bool with_modifiers) {
struct gbm_device *gbm = gbm_bo_get_device(bo);
@ -236,14 +207,23 @@ uint32_t get_fb_for_bo(struct gbm_bo *bo, bool with_modifiers) {
return 0;
}
if (drmModeAddFB2(fd, width, height, format, handles, strides,
offsets, &id, 0)) {
wlr_log_errno(WLR_DEBUG,
"Unable to add DRM framebuffer, trying legacy method");
id = get_fb_for_bo_legacy(bo);
if (id == 0) {
wlr_log(WLR_ERROR, "Unable to add DRM framebuffer");
}
int ret = drmModeAddFB2(fd, width, height, format, handles, strides,
offsets, &id, 0);
if (ret != 0 && gbm_bo_get_format(bo) == GBM_FORMAT_ARGB8888 &&
gbm_bo_get_plane_count(bo) == 1) {
// Some big-endian machines don't support drmModeAddFB2. Try a
// last-resort fallback for ARGB8888 buffers, like Xorg's
// modesetting driver does.
wlr_log(WLR_DEBUG, "drmModeAddFB2 failed (%s), falling back to "
"legacy drmModeAddFB", strerror(-ret));
uint32_t depth = 32;
uint32_t bpp = gbm_bo_get_bpp(bo);
ret = drmModeAddFB(fd, width, height, depth, bpp, strides[0],
handles[0], &id);
}
if (ret != 0) {
wlr_log(WLR_ERROR, "Unable to add DRM framebuffer: %s", strerror(-ret));
}
}