Re: [PATCH 00/11] Mixed bag of ioctl and agp cleanups

From: Daniel Vetter <daniel@ffwll.ch>
To: "Christian König" <deathsimple@vodafone.de>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>,
	Intel Graphics Development <intel-gfx@lists.freedesktop.org>,
	DRI Development <dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH 00/11] Mixed bag of ioctl and agp cleanups
Date: Thu, 8 Oct 2015 19:01:45 +0200	[thread overview]
Message-ID: <20151008170145.GG3383@phenom.ffwll.local> (raw)
In-Reply-To: <55EEDB8C.3050907@vodafone.de>

On Tue, Sep 08, 2015 at 02:58:52PM +0200, Christian König wrote:
> Patches #1, #6, #7, #8 and #10 are Reviewed-by: Christian König
> <christian.koenig@amd.com>.
> 
> That should be everything touching radeon/amdgpu if I missed something leave
> me a note.

Ok with an irc r-b from Chris and David I and yours here I pulled in the
remaining patches from this series.

Thanks, Daniel

> 
> Regards,
> Christian.
> 
> On 08.09.2015 13:56, Daniel Vetter wrote:
> >Hi all,
> >
> >Big thing for sure is deprecating DRM_UNLOCKED for DRIVER_MODESET (i.e. modern
> >drivers) since all of them have their own locking. Besides that a few random
> >things that kind where in the same area/files.
> >
> >Of course kerneldoc is updated too.
> >
> >Feedback highly welcome.
> >
> >Cheers, Daniel
> >
> >Daniel Vetter (11):
> >   drm: Remove __OS_HAS_AGP
> >   drm/i915: Kill cross-module option depencies
> >   drm/i915: Mark debug mod options as _unsafe
> >   drm/i915: Remove setparam ioctl
> >   drm/i915: Mark getparam ioctl as DRM_UNLOCKED
> >   drm: Define a drm_invalid_op ioctl implementation
> >   drm/drm_ioctl.c: kerneldoc
> >   drm: Enforce unlocked ioctl operation for kms driver ioctls
> >   drm/vmwgfx: Stop checking for DRM_UNLOCKED
> >   drm/<drivers>: Drop DRM_UNLOCKED from modeset drivers
> >   drm: Remove dummy agp ioctl wrappers
> >
> >  Documentation/DocBook/drm.tmpl          |   5 +-
> >  drivers/gpu/drm/Makefile                |   3 +-
> >  drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c |  24 +++----
> >  drivers/gpu/drm/armada/armada_drv.c     |   9 +--
> >  drivers/gpu/drm/drm_agpsupport.c        |   4 +-
> >  drivers/gpu/drm/drm_bufs.c              |   6 +-
> >  drivers/gpu/drm/drm_ioc32.c             |   6 +-
> >  drivers/gpu/drm/drm_ioctl.c             |  89 +++++++++++++++++------
> >  drivers/gpu/drm/drm_memory.c            |   6 +-
> >  drivers/gpu/drm/drm_vm.c                |   8 +--
> >  drivers/gpu/drm/exynos/exynos_drm_drv.c |  20 +++---
> >  drivers/gpu/drm/i915/i915_dma.c         | 101 ++++++++++----------------
> >  drivers/gpu/drm/i915/i915_gem_gtt.c     |   2 +-
> >  drivers/gpu/drm/i915/i915_params.c      |  30 ++++----
> >  drivers/gpu/drm/i915/intel_lrc.c        |   3 +-
> >  drivers/gpu/drm/mga/mga_dma.c           |   4 +-
> >  drivers/gpu/drm/msm/msm_drv.c           |  14 ++--
> >  drivers/gpu/drm/nouveau/nouveau_bo.c    |   8 +--
> >  drivers/gpu/drm/nouveau/nouveau_drm.c   |  24 +++----
> >  drivers/gpu/drm/omapdrm/omap_drv.c      |  12 ++--
> >  drivers/gpu/drm/qxl/qxl_ioctl.c         |  14 ++--
> >  drivers/gpu/drm/r128/r128_cce.c         |  12 ++--
> >  drivers/gpu/drm/radeon/r600_cp.c        |  14 ++--
> >  drivers/gpu/drm/radeon/radeon_agp.c     |   8 +--
> >  drivers/gpu/drm/radeon/radeon_cp.c      |  16 ++---
> >  drivers/gpu/drm/radeon/radeon_kms.c     | 124 +++++++++++---------------------
> >  drivers/gpu/drm/radeon/radeon_ttm.c     |  10 +--
> >  drivers/gpu/drm/tegra/drm.c             |  28 ++++----
> >  drivers/gpu/drm/vmwgfx/vmwgfx_drv.c     |  62 +++++++---------
> >  include/drm/drmP.h                      |   2 +
> >  include/drm/drm_agpsupport.h            |  54 +-------------
> >  31 files changed, 321 insertions(+), 401 deletions(-)
> >
> 

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel