lkml.org 
[lkml]   [2021]   [Oct]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[RFC PATCH] drm/aperture: Add param to disable conflicting framebuffers removal
    Date
    The simpledrm driver allows to use the frame buffer that was set-up by the
    firmware. This gives early video output before the platform DRM driver is
    probed and takes over.

    But it would be useful to have a way to disable this take over by the real
    DRM drivers. For example, there may be bugs in the DRM drivers that could
    cause the display output to not work correctly.

    For those cases, it would be good to keep the simpledrm driver instead and
    at least get a working display as set-up by the firmware.

    Let's add a drm.remove_fb boolean kernel command line parameter, that when
    set to false will prevent the conflicting framebuffers to being removed.

    Since the drivers call drm_aperture_remove_conflicting_framebuffers() very
    early in their probe callback, this will cause the drivers' probe to fail.

    Thanks to Neal Gompa for the suggestion and Thomas Zimmermann for the idea
    on how this could be implemented.

    Suggested-by: Neal Gompa <ngompa13@gmail.com>
    Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
    ---
    Hello,

    I'm sending this as an RFC because I wasn't sure about the correct name for
    this module parameter, and also if 'remove_fb=0' is intitutive or instead a
    parameter that's enabled is preferred (i.e: 'disable_fb_removal=1').

    Best regards,
    Javier

    drivers/gpu/drm/drm_aperture.c | 19 +++++++++++++++++++
    1 file changed, 19 insertions(+)

    diff --git a/drivers/gpu/drm/drm_aperture.c b/drivers/gpu/drm/drm_aperture.c
    index 74bd4a76b253..0b454c8f7465 100644
    --- a/drivers/gpu/drm/drm_aperture.c
    +++ b/drivers/gpu/drm/drm_aperture.c
    @@ -14,6 +14,11 @@
    #include <drm/drm_drv.h>
    #include <drm/drm_print.h>

    +static bool drm_aperture_remove_fb = true;
    +module_param_named(remove_fb, drm_aperture_remove_fb, bool, 0600);
    +MODULE_PARM_DESC(remove_fb,
    + "Allow conflicting framebuffers removal [default=true]");
    +
    /**
    * DOC: overview
    *
    @@ -283,6 +288,9 @@ static void drm_aperture_detach_drivers(resource_size_t base, resource_size_t si
    * This function removes graphics device drivers which use memory range described by
    * @base and @size.
    *
    + * The conflicting framebuffers removal can be disabled by setting the drm.remove_fb=0 kernel
    + * command line option. When this is disabled, the function will return an -EINVAL errno code.
    + *
    * Returns:
    * 0 on success, or a negative errno code otherwise
    */
    @@ -292,7 +300,12 @@ int drm_aperture_remove_conflicting_framebuffers(resource_size_t base, resource_
    #if IS_REACHABLE(CONFIG_FB)
    struct apertures_struct *a;
    int ret;
    +#endif
    +
    + if (!drm_aperture_remove_fb)
    + return -EINVAL;

    +#if IS_REACHABLE(CONFIG_FB)
    a = alloc_apertures(1);
    if (!a)
    return -ENOMEM;
    @@ -322,6 +335,9 @@ EXPORT_SYMBOL(drm_aperture_remove_conflicting_framebuffers);
    * for any of @pdev's memory bars. The function assumes that PCI device with
    * shadowed ROM drives a primary display and so kicks out vga16fb.
    *
    + * The conflicting framebuffers removal can be disabled by setting the drm.remove_fb=0 kernel
    + * command line option. When this is disabled, the function will return an -EINVAL errno code.
    + *
    * Returns:
    * 0 on success, or a negative errno code otherwise
    */
    @@ -331,6 +347,9 @@ int drm_aperture_remove_conflicting_pci_framebuffers(struct pci_dev *pdev,
    resource_size_t base, size;
    int bar, ret = 0;

    + if (!drm_aperture_remove_fb)
    + return -EINVAL;
    +
    for (bar = 0; bar < PCI_STD_NUM_BARS; ++bar) {
    if (!(pci_resource_flags(pdev, bar) & IORESOURCE_MEM))
    continue;
    --
    2.31.1
    \
     
     \ /
      Last update: 2021-10-22 16:41    [W:3.946 / U:0.644 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site