dri-devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
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 --]

      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