Generate a hotplug event after registering a client to allow the client to configure its display. Remove the hotplug calls from the existing clients for fbdev emulation. This change fixes a concurrency bug between registering a client and receiving events from the DRM core. The bug is present in the fbdev emulation of all drivers. The fbdev emulation currently generates a hotplug event before registering the client to the device. For each new output, the DRM core sends an additional hotplug event to each registered client. If the DRM core detects first output between sending the artificial hotplug and registering the device, the output's hotplug event gets lost. If this is the first output, the fbdev console display remains dark. This has been observed with amdgpu and fbdev-generic. Fix this by adding hotplug generation directly to the client's register helper drm_client_register(). Registering the client and receiving events are serialized by struct drm_device.clientlist_mutex. So an output is either configured by the initial hotplug event, or the client has already been registered. The bug was originally added in commit6e3f17ee73
("drm/fb-helper: generic: Call drm_client_add() after setup is done"), in which adding a client and receiving a hotplug event switched order. It was hidden, as most hardware and drivers have at least on static output configured. Other drivers didn't use the internal DRM client or still had struct drm_mode_config_funcs.output_poll_changed set. That callback handled hotplug events as well. After not setting the callback in amdgpu in commit0e3172bac3
("drm/amdgpu: Don't set struct drm_driver.output_poll_changed"), amdgpu did not show a framebuffer console if output events got lost. The bug got copy-pasted from fbdev-generic into the other fbdev emulation. Reported-by: Moritz Duge <MoritzDuge@kolahilft.de> Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/2649 Fixes:6e3f17ee73
("drm/fb-helper: generic: Call drm_client_add() after setup is done") Fixes:8ab59da26b
("drm/fb-helper: Move generic fbdev emulation into separate source file") Fixes:b79fe9abd5
("drm/fbdev-dma: Implement fbdev emulation for GEM DMA helpers") Fixes:63c381552f
("drm/armada: Implement fbdev emulation as in-kernel client") Fixes:49953b70e7
("drm/exynos: Implement fbdev emulation as in-kernel client") Fixes:8f1aaccb04
("drm/gma500: Implement client-based fbdev emulation") Fixes:940b869c2f
("drm/msm: Implement fbdev emulation as in-kernel client") Fixes:9e69bcd88e
("drm/omapdrm: Implement fbdev emulation as in-kernel client") Fixes:e317a69fe8
("drm/radeon: Implement client-based fbdev emulation") Fixes:71ec16f45e
("drm/tegra: Implement fbdev emulation as in-kernel client") Fixes:0e3172bac3
("drm/amdgpu: Don't set struct drm_driver.output_poll_changed") Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de> Tested-by: Moritz Duge <MoritzDuge@kolahilft.de> Tested-by: Torsten Krah <krah.tm@gmail.com> Tested-by: Paul Schyska <pschyska@gmail.com> Cc: Daniel Vetter <daniel.vetter@ffwll.ch> Cc: David Airlie <airlied@gmail.com> Cc: Noralf Trønnes <noralf@tronnes.org> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Cc: Maxime Ripard <mripard@kernel.org> Cc: Javier Martinez Canillas <javierm@redhat.com> Cc: Russell King <linux@armlinux.org.uk> Cc: Inki Dae <inki.dae@samsung.com> Cc: Seung-Woo Kim <sw0312.kim@samsung.com> Cc: Kyungmin Park <kyungmin.park@samsung.com> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Cc: Patrik Jakobsson <patrik.r.jakobsson@gmail.com> Cc: Rob Clark <robdclark@gmail.com> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Cc: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> Cc: Alex Deucher <alexander.deucher@amd.com> Cc: "Christian König" <christian.koenig@amd.com> Cc: "Pan, Xinhui" <Xinhui.Pan@amd.com> Cc: Thierry Reding <thierry.reding@gmail.com> Cc: Mikko Perttunen <mperttunen@nvidia.com> Cc: dri-devel@lists.freedesktop.org Cc: linux-kernel@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org Cc: linux-samsung-soc@vger.kernel.org Cc: linux-arm-msm@vger.kernel.org Cc: freedreno@lists.freedesktop.org Cc: amd-gfx@lists.freedesktop.org Cc: linux-tegra@vger.kernel.org Cc: dri-devel@lists.freedesktop.org Cc: <stable@vger.kernel.org> # v5.2+ Reviewed-by: Javier Martinez Canillas <javierm@redhat.com> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> # msm Link: https://patchwork.freedesktop.org/patch/msgid/20230710091029.27503-1-tzimmermann@suse.de
225 lines
5.6 KiB
C
225 lines
5.6 KiB
C
// SPDX-License-Identifier: GPL-2.0-or-later
|
|
/* exynos_drm_fbdev.c
|
|
*
|
|
* Copyright (c) 2011 Samsung Electronics Co., Ltd.
|
|
* Authors:
|
|
* Inki Dae <inki.dae@samsung.com>
|
|
* Joonyoung Shim <jy0922.shim@samsung.com>
|
|
* Seung-Woo Kim <sw0312.kim@samsung.com>
|
|
*/
|
|
|
|
#include <linux/fb.h>
|
|
|
|
#include <drm/drm_crtc_helper.h>
|
|
#include <drm/drm_drv.h>
|
|
#include <drm/drm_fb_helper.h>
|
|
#include <drm/drm_framebuffer.h>
|
|
#include <drm/drm_gem_framebuffer_helper.h>
|
|
#include <drm/drm_prime.h>
|
|
#include <drm/exynos_drm.h>
|
|
|
|
#include "exynos_drm_drv.h"
|
|
#include "exynos_drm_fb.h"
|
|
#include "exynos_drm_fbdev.h"
|
|
|
|
#define MAX_CONNECTOR 4
|
|
#define PREFERRED_BPP 32
|
|
|
|
static int exynos_drm_fb_mmap(struct fb_info *info, struct vm_area_struct *vma)
|
|
{
|
|
struct drm_fb_helper *helper = info->par;
|
|
struct drm_gem_object *obj = drm_gem_fb_get_obj(helper->fb, 0);
|
|
|
|
return drm_gem_prime_mmap(obj, vma);
|
|
}
|
|
|
|
static void exynos_drm_fb_destroy(struct fb_info *info)
|
|
{
|
|
struct drm_fb_helper *fb_helper = info->par;
|
|
struct drm_framebuffer *fb = fb_helper->fb;
|
|
|
|
drm_fb_helper_fini(fb_helper);
|
|
|
|
drm_framebuffer_remove(fb);
|
|
|
|
drm_client_release(&fb_helper->client);
|
|
drm_fb_helper_unprepare(fb_helper);
|
|
kfree(fb_helper);
|
|
}
|
|
|
|
static const struct fb_ops exynos_drm_fb_ops = {
|
|
.owner = THIS_MODULE,
|
|
__FB_DEFAULT_IO_OPS_RDWR,
|
|
DRM_FB_HELPER_DEFAULT_OPS,
|
|
__FB_DEFAULT_IO_OPS_DRAW,
|
|
.fb_mmap = exynos_drm_fb_mmap,
|
|
.fb_destroy = exynos_drm_fb_destroy,
|
|
};
|
|
|
|
static int exynos_drm_fbdev_update(struct drm_fb_helper *helper,
|
|
struct drm_fb_helper_surface_size *sizes,
|
|
struct exynos_drm_gem *exynos_gem)
|
|
{
|
|
struct fb_info *fbi;
|
|
struct drm_framebuffer *fb = helper->fb;
|
|
unsigned int size = fb->width * fb->height * fb->format->cpp[0];
|
|
unsigned long offset;
|
|
|
|
fbi = drm_fb_helper_alloc_info(helper);
|
|
if (IS_ERR(fbi)) {
|
|
DRM_DEV_ERROR(to_dma_dev(helper->dev),
|
|
"failed to allocate fb info.\n");
|
|
return PTR_ERR(fbi);
|
|
}
|
|
|
|
fbi->fbops = &exynos_drm_fb_ops;
|
|
|
|
drm_fb_helper_fill_info(fbi, helper, sizes);
|
|
|
|
offset = fbi->var.xoffset * fb->format->cpp[0];
|
|
offset += fbi->var.yoffset * fb->pitches[0];
|
|
|
|
fbi->screen_buffer = exynos_gem->kvaddr + offset;
|
|
fbi->screen_size = size;
|
|
fbi->fix.smem_len = size;
|
|
|
|
return 0;
|
|
}
|
|
|
|
static int exynos_drm_fbdev_create(struct drm_fb_helper *helper,
|
|
struct drm_fb_helper_surface_size *sizes)
|
|
{
|
|
struct exynos_drm_gem *exynos_gem;
|
|
struct drm_device *dev = helper->dev;
|
|
struct drm_mode_fb_cmd2 mode_cmd = { 0 };
|
|
unsigned long size;
|
|
int ret;
|
|
|
|
DRM_DEV_DEBUG_KMS(dev->dev,
|
|
"surface width(%d), height(%d) and bpp(%d\n",
|
|
sizes->surface_width, sizes->surface_height,
|
|
sizes->surface_bpp);
|
|
|
|
mode_cmd.width = sizes->surface_width;
|
|
mode_cmd.height = sizes->surface_height;
|
|
mode_cmd.pitches[0] = sizes->surface_width * (sizes->surface_bpp >> 3);
|
|
mode_cmd.pixel_format = drm_mode_legacy_fb_format(sizes->surface_bpp,
|
|
sizes->surface_depth);
|
|
|
|
size = mode_cmd.pitches[0] * mode_cmd.height;
|
|
|
|
exynos_gem = exynos_drm_gem_create(dev, EXYNOS_BO_WC, size, true);
|
|
if (IS_ERR(exynos_gem))
|
|
return PTR_ERR(exynos_gem);
|
|
|
|
helper->fb =
|
|
exynos_drm_framebuffer_init(dev, &mode_cmd, &exynos_gem, 1);
|
|
if (IS_ERR(helper->fb)) {
|
|
DRM_DEV_ERROR(dev->dev, "failed to create drm framebuffer.\n");
|
|
ret = PTR_ERR(helper->fb);
|
|
goto err_destroy_gem;
|
|
}
|
|
|
|
ret = exynos_drm_fbdev_update(helper, sizes, exynos_gem);
|
|
if (ret < 0)
|
|
goto err_destroy_framebuffer;
|
|
|
|
return 0;
|
|
|
|
err_destroy_framebuffer:
|
|
drm_framebuffer_cleanup(helper->fb);
|
|
helper->fb = NULL;
|
|
err_destroy_gem:
|
|
exynos_drm_gem_destroy(exynos_gem);
|
|
return ret;
|
|
}
|
|
|
|
static const struct drm_fb_helper_funcs exynos_drm_fb_helper_funcs = {
|
|
.fb_probe = exynos_drm_fbdev_create,
|
|
};
|
|
|
|
/*
|
|
* struct drm_client
|
|
*/
|
|
|
|
static void exynos_drm_fbdev_client_unregister(struct drm_client_dev *client)
|
|
{
|
|
struct drm_fb_helper *fb_helper = drm_fb_helper_from_client(client);
|
|
|
|
if (fb_helper->info) {
|
|
drm_fb_helper_unregister_info(fb_helper);
|
|
} else {
|
|
drm_client_release(&fb_helper->client);
|
|
drm_fb_helper_unprepare(fb_helper);
|
|
kfree(fb_helper);
|
|
}
|
|
}
|
|
|
|
static int exynos_drm_fbdev_client_restore(struct drm_client_dev *client)
|
|
{
|
|
drm_fb_helper_lastclose(client->dev);
|
|
|
|
return 0;
|
|
}
|
|
|
|
static int exynos_drm_fbdev_client_hotplug(struct drm_client_dev *client)
|
|
{
|
|
struct drm_fb_helper *fb_helper = drm_fb_helper_from_client(client);
|
|
struct drm_device *dev = client->dev;
|
|
int ret;
|
|
|
|
if (dev->fb_helper)
|
|
return drm_fb_helper_hotplug_event(dev->fb_helper);
|
|
|
|
ret = drm_fb_helper_init(dev, fb_helper);
|
|
if (ret)
|
|
goto err_drm_err;
|
|
|
|
if (!drm_drv_uses_atomic_modeset(dev))
|
|
drm_helper_disable_unused_functions(dev);
|
|
|
|
ret = drm_fb_helper_initial_config(fb_helper);
|
|
if (ret)
|
|
goto err_drm_fb_helper_fini;
|
|
|
|
return 0;
|
|
|
|
err_drm_fb_helper_fini:
|
|
drm_fb_helper_fini(fb_helper);
|
|
err_drm_err:
|
|
drm_err(dev, "Failed to setup fbdev emulation (ret=%d)\n", ret);
|
|
return ret;
|
|
}
|
|
|
|
static const struct drm_client_funcs exynos_drm_fbdev_client_funcs = {
|
|
.owner = THIS_MODULE,
|
|
.unregister = exynos_drm_fbdev_client_unregister,
|
|
.restore = exynos_drm_fbdev_client_restore,
|
|
.hotplug = exynos_drm_fbdev_client_hotplug,
|
|
};
|
|
|
|
void exynos_drm_fbdev_setup(struct drm_device *dev)
|
|
{
|
|
struct drm_fb_helper *fb_helper;
|
|
int ret;
|
|
|
|
drm_WARN(dev, !dev->registered, "Device has not been registered.\n");
|
|
drm_WARN(dev, dev->fb_helper, "fb_helper is already set!\n");
|
|
|
|
fb_helper = kzalloc(sizeof(*fb_helper), GFP_KERNEL);
|
|
if (!fb_helper)
|
|
return;
|
|
drm_fb_helper_prepare(dev, fb_helper, PREFERRED_BPP, &exynos_drm_fb_helper_funcs);
|
|
|
|
ret = drm_client_init(dev, &fb_helper->client, "fbdev", &exynos_drm_fbdev_client_funcs);
|
|
if (ret)
|
|
goto err_drm_client_init;
|
|
|
|
drm_client_register(&fb_helper->client);
|
|
|
|
return;
|
|
|
|
err_drm_client_init:
|
|
drm_fb_helper_unprepare(fb_helper);
|
|
kfree(fb_helper);
|
|
}
|