From: Thomas Zimmermann <tzimmermann@suse.de>
To: airlied@gmail.com, daniel@ffwll.ch,
maarten.lankhorst@linux.intel.com, mripard@kernel.org,
cai.huoqing@linux.dev
Cc: dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 00/14] drm: Remove infrastructure for user-space mode setting
Date: Wed, 6 Dec 2023 10:20:29 +0100 [thread overview]
Message-ID: <e5d76f8e-98ab-4d49-aecf-14d0b581fa46@suse.de> (raw)
In-Reply-To: <20231122122449.11588-1-tzimmermann@suse.de>
[-- Attachment #1.1: Type: text/plain, Size: 5619 bytes --]
Merged into drm-misc-next, except for patch 1. It's fairly independent
from the other patches and I'll resend it directly to the the PPC list.
Best regards
Thomas
Am 22.11.23 um 13:09 schrieb Thomas Zimmermann:
> The old drivers for user-space mode setting have been removed in Linux
> v6.3. No one has complained or requested their return. It is time to
> remove these drivers' infrastructure from the DRM core.
>
> The recent Linux v6.6 has been designated as long-term release, [1] so
> any remaining users have a few more years to get a new graphics card. The
> legacy devices continue to be supported via simpledrm. Merging proper
> drivers with kernel mode setting is also an option.
>
> Patches 1 to 7 fix a few trivial issues that have been forgotten during
> the removal of the drivers.
>
> Patches 8 and 9 remove the leagcy ioctl interfaces. One op is different
> from the others, so it gets its own patch.
>
> Patches 10 to 12 remove the legacy source code from DRM. With that gone
> patch 13, the AGP code can be simplified as well. There used to be
> a device file for user-space mode setting, /dev/agpgart, that is now
> obsolete.
>
> Patch 14 removes the option from Kconfig.
>
> [1] https://kernel.org/category/releases.html
>
> Thomas Zimmermann (14):
> arch/powerpc: Remove legacy DRM drivers from default configs
> drm: Fix TODO list mentioning non-KMS drivers
> drm: Include <drm/drm_auth.h>
> drm/i915: Include <drm/drm_auth.h>
> accel: Include <drm/drm_auth.h>
> drm: Include <drm/drm_device.h>
> drm/radeon: Do not include <drm/drm_legacy.h>
> drm: Remove entry points for legacy ioctls
> drm: Remove the legacy DRM_IOCTL_MODESET_CTL ioctl
> drm: Remove support for legacy drivers
> drm: Remove locking for legacy ioctls and DRM_UNLOCKED
> drm: Remove source code for non-KMS drivers
> char/agp: Remove frontend code
> drm: Remove Kconfig option for legacy support (CONFIG_DRM_LEGACY)
>
> Documentation/gpu/todo.rst | 7 +-
> arch/powerpc/configs/pmac32_defconfig | 2 -
> arch/powerpc/configs/ppc6xx_defconfig | 7 -
> drivers/accel/drm_accel.c | 1 +
> drivers/char/agp/Makefile | 6 -
> drivers/char/agp/agp.h | 9 -
> drivers/char/agp/backend.c | 11 -
> drivers/char/agp/compat_ioctl.c | 291 ---
> drivers/char/agp/compat_ioctl.h | 106 --
> drivers/char/agp/frontend.c | 1068 -----------
> drivers/gpu/drm/Kconfig | 21 -
> drivers/gpu/drm/Makefile | 12 -
> drivers/gpu/drm/drm_agpsupport.c | 451 -----
> drivers/gpu/drm/drm_auth.c | 8 +-
> drivers/gpu/drm/drm_bufs.c | 1627 -----------------
> drivers/gpu/drm/drm_context.c | 513 ------
> drivers/gpu/drm/drm_dma.c | 178 --
> drivers/gpu/drm/drm_drv.c | 17 -
> drivers/gpu/drm/drm_file.c | 64 +-
> drivers/gpu/drm/drm_hashtab.c | 203 --
> drivers/gpu/drm/drm_internal.h | 7 -
> drivers/gpu/drm/drm_ioc32.c | 613 +------
> drivers/gpu/drm/drm_ioctl.c | 82 +-
> drivers/gpu/drm/drm_irq.c | 204 ---
> drivers/gpu/drm/drm_legacy.h | 290 ---
> drivers/gpu/drm/drm_legacy_misc.c | 105 --
> drivers/gpu/drm/drm_lock.c | 373 ----
> drivers/gpu/drm/drm_memory.c | 138 --
> drivers/gpu/drm/drm_pci.c | 204 +--
> drivers/gpu/drm/drm_scatter.c | 220 ---
> drivers/gpu/drm/drm_vblank.c | 101 -
> drivers/gpu/drm/drm_vm.c | 665 -------
> .../gpu/drm/i915/gem/i915_gem_execbuffer.c | 1 +
> drivers/gpu/drm/radeon/radeon_drv.h | 1 -
> include/drm/drm_auth.h | 22 -
> include/drm/drm_device.h | 71 +-
> include/drm/drm_drv.h | 19 -
> include/drm/drm_file.h | 5 -
> include/drm/drm_ioctl.h | 11 -
> include/drm/drm_legacy.h | 331 ----
> 40 files changed, 20 insertions(+), 8045 deletions(-)
> delete mode 100644 drivers/char/agp/compat_ioctl.c
> delete mode 100644 drivers/char/agp/compat_ioctl.h
> delete mode 100644 drivers/char/agp/frontend.c
> delete mode 100644 drivers/gpu/drm/drm_agpsupport.c
> delete mode 100644 drivers/gpu/drm/drm_bufs.c
> delete mode 100644 drivers/gpu/drm/drm_context.c
> delete mode 100644 drivers/gpu/drm/drm_dma.c
> delete mode 100644 drivers/gpu/drm/drm_hashtab.c
> delete mode 100644 drivers/gpu/drm/drm_irq.c
> delete mode 100644 drivers/gpu/drm/drm_legacy.h
> delete mode 100644 drivers/gpu/drm/drm_legacy_misc.c
> delete mode 100644 drivers/gpu/drm/drm_lock.c
> delete mode 100644 drivers/gpu/drm/drm_memory.c
> delete mode 100644 drivers/gpu/drm/drm_scatter.c
> delete mode 100644 drivers/gpu/drm/drm_vm.c
> delete mode 100644 include/drm/drm_legacy.h
>
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]
prev parent reply other threads:[~2023-12-06 9:20 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-22 12:09 [PATCH 00/14] drm: Remove infrastructure for user-space mode setting Thomas Zimmermann
2023-11-22 12:09 ` [PATCH 01/14] arch/powerpc: Remove legacy DRM drivers from default configs Thomas Zimmermann
2023-11-27 21:03 ` Alex Deucher
2023-11-28 7:19 ` Cai Huoqing
2023-11-22 12:09 ` [PATCH 02/14] drm: Fix TODO list mentioning non-KMS drivers Thomas Zimmermann
2023-11-27 21:02 ` Alex Deucher
2023-11-22 12:09 ` [PATCH 03/14] drm: Include <drm/drm_auth.h> Thomas Zimmermann
2023-11-27 21:03 ` Alex Deucher
2023-11-22 12:09 ` [PATCH 04/14] drm/i915: " Thomas Zimmermann
2023-11-22 18:14 ` Jani Nikula
2023-11-22 12:09 ` [PATCH 05/14] accel: " Thomas Zimmermann
2023-11-27 21:03 ` Alex Deucher
2023-11-22 12:09 ` [PATCH 06/14] drm: Include <drm/drm_device.h> Thomas Zimmermann
2023-11-22 12:09 ` [PATCH 07/14] drm/radeon: Do not include <drm/drm_legacy.h> Thomas Zimmermann
2023-11-27 21:00 ` Alex Deucher
2023-11-22 12:09 ` [PATCH 08/14] drm: Remove entry points for legacy ioctls Thomas Zimmermann
2023-11-27 21:04 ` Alex Deucher
2023-11-22 12:09 ` [PATCH 09/14] drm: Remove the legacy DRM_IOCTL_MODESET_CTL ioctl Thomas Zimmermann
2023-11-27 21:05 ` Alex Deucher
2023-11-22 12:09 ` [PATCH 10/14] drm: Remove support for legacy drivers Thomas Zimmermann
2023-11-27 21:05 ` Alex Deucher
2023-11-22 12:09 ` [PATCH 11/14] drm: Remove locking for legacy ioctls and DRM_UNLOCKED Thomas Zimmermann
2023-11-27 21:05 ` Alex Deucher
2023-11-22 12:09 ` [PATCH 12/14] drm: Remove source code for non-KMS drivers Thomas Zimmermann
2023-11-27 21:06 ` Alex Deucher
2023-11-22 12:09 ` [PATCH 13/14] char/agp: Remove frontend code Thomas Zimmermann
2023-11-27 21:07 ` Alex Deucher
2023-11-22 12:09 ` [PATCH 14/14] drm: Remove Kconfig option for legacy support (CONFIG_DRM_LEGACY) Thomas Zimmermann
2023-11-27 21:07 ` Alex Deucher
2023-12-01 8:39 ` [PATCH 00/14] drm: Remove infrastructure for user-space mode setting Thomas Zimmermann
2023-12-06 9:20 ` Thomas Zimmermann [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e5d76f8e-98ab-4d49-aecf-14d0b581fa46@suse.de \
--to=tzimmermann@suse.de \
--cc=airlied@gmail.com \
--cc=cai.huoqing@linux.dev \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mripard@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox