From: Rob Clark <robdclark@gmail.com>
To: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: DRI Development <dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH 00/36] kms locking rework
Date: Fri, 11 Jan 2013 17:17:05 -0600 [thread overview]
Message-ID: <CAF6AEGudQXgg6cMUM8Fo==0DdAraNEgn8MHcTYw_oNLosohfTw@mail.gmail.com> (raw)
In-Reply-To: <1357850897-27102-1-git-send-email-daniel.vetter@ffwll.ch>
On Thu, Jan 10, 2013 at 2:47 PM, Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
> Hi all,
>
> Concurrent EDID reads and pageflips ftw!
>
> Changes since the rfc:
> - radeon/ttm/exynos prep patches/fixes merged.
> - arm driver patches at least compile-tested.
> - error handling fixed in a few cases, all spotted by Richard Wilbur,
> - radeon/nouveau tested on one card each, didn't blow up right away.
> - i-g-t extended to cover the corner-cases, seems to hold up.
> - DocBook updated as a new patch on top.
> - one more patch to teach the fb helpers manners and not interfere so much with
> i-g-t tests.
>
> Imo this is ready for inclusion, so (especially driver maintainers) please
> review and test.
I've read through the series.. I'll have to make a few little updates
vs the dss/kms rewrite patch in omapdrm which should hopefully be
merged soon, but that shouldn't be a big deal.
Reviewed-by: Rob Clark <rob@ti.com>
> Yours, Daniel
>
> Daniel Vetter (35):
> drm: review locking rules in drm_crtc.c
> drm/doc: integrate drm_crtc.c kerneldoc
> drm/<drivers>: reorder framebuffer init sequence
> drm/vmwgfx: reorder framebuffer init sequence
> drm/gma500: move fbcon restore to lastclose
> drm/nouveau: protect evo_wait/evo_kick sections with a channel mutex
> drm/nouveau: try to protect nbo->pin_refcount
> drm/<drivers>: Unified handling of unimplemented fb->create_handle
> drm: encapsulate crtc->set_config calls
> drm: add drm_modeset_lock|unlock_all
> drm/i915: use drm_modeset_lock_all
> drm/gma500: use drm_modeset_lock_all
> drm/ast: use drm_modeset_lock_all
> drm/shmobile: use drm_modeset_lock_all
> drm/vmgfx: use drm_modeset_lock_all
> drm: add per-crtc locks
> drm: only take the crtc lock for ->cursor_set
> drm: only take the crtc lock for ->cursor_move
> drm: revamp locking around fb creation/destruction
> drm: create drm_framebuffer_lookup
> drm: revamp framebuffer cleanup interfaces
> drm: reference framebuffers which are on the idr
> drm: nest modeset locks within fpriv->fbs_lock
> drm: push modeset_lock_all into ->fb_create driver callbacks
> drm: don't take modeset locks in getfb ioctl
> drm: fb refcounting for dirtyfb_ioctl
> drm: refcounting for sprite framebuffers
> drm: refcounting for crtc framebuffers
> drm/i915: dump refcount into framebuffer debugfs file
> drm/vmwgfx: add proper framebuffer refcounting
> drm: optimize drm_framebuffer_remove
> drm: only grab the crtc lock for pageflips
> drm: don't hold crtc mutexes for connector ->detect callbacks
> drm/doc: updates for new framebuffer lifetime rules
> drm/fb_helper: check whether fbcon is bound
>
> Documentation/DocBook/drm.tmpl | 63 ++-
> drivers/gpu/drm/ast/ast_drv.c | 4 +-
> drivers/gpu/drm/ast/ast_drv.h | 2 +
> drivers/gpu/drm/ast/ast_fb.c | 1 +
> drivers/gpu/drm/ast/ast_main.c | 12 +-
> drivers/gpu/drm/cirrus/cirrus_fbdev.c | 1 +
> drivers/gpu/drm/cirrus/cirrus_main.c | 12 +-
> drivers/gpu/drm/drm_crtc.c | 792 +++++++++++++++++------------
> drivers/gpu/drm/drm_fb_cma_helper.c | 15 +-
> drivers/gpu/drm/drm_fb_helper.c | 65 ++-
> drivers/gpu/drm/drm_fops.c | 1 +
> drivers/gpu/drm/exynos/exynos_drm_fbdev.c | 4 +-
> drivers/gpu/drm/gma500/framebuffer.c | 29 +-
> drivers/gpu/drm/gma500/psb_device.c | 8 +-
> drivers/gpu/drm/gma500/psb_drv.c | 14 +-
> drivers/gpu/drm/i2c/ch7006_drv.c | 2 +-
> drivers/gpu/drm/i915/i915_debugfs.c | 15 +-
> drivers/gpu/drm/i915/intel_display.c | 21 +-
> drivers/gpu/drm/i915/intel_dp.c | 2 +
> drivers/gpu/drm/i915/intel_fb.c | 5 +-
> drivers/gpu/drm/i915/intel_lvds.c | 4 +-
> drivers/gpu/drm/i915/intel_overlay.c | 14 +-
> drivers/gpu/drm/i915/intel_sprite.c | 8 +-
> drivers/gpu/drm/mgag200/mgag200_fb.c | 1 +
> drivers/gpu/drm/mgag200/mgag200_main.c | 16 +-
> drivers/gpu/drm/nouveau/nouveau_bo.c | 22 +-
> drivers/gpu/drm/nouveau/nouveau_bo.h | 2 +
> drivers/gpu/drm/nouveau/nouveau_display.c | 10 +-
> drivers/gpu/drm/nouveau/nouveau_fbcon.c | 1 +
> drivers/gpu/drm/nouveau/nv04_display.c | 2 +-
> drivers/gpu/drm/nouveau/nv17_tv.c | 2 +-
> drivers/gpu/drm/nouveau/nv50_display.c | 10 +
> drivers/gpu/drm/radeon/radeon_cursor.c | 8 +-
> drivers/gpu/drm/radeon/radeon_display.c | 2 +-
> drivers/gpu/drm/radeon/radeon_fb.c | 2 +
> drivers/gpu/drm/shmobile/shmob_drm_drv.c | 4 +-
> drivers/gpu/drm/udl/udl_fb.c | 4 +-
> drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 2 +-
> drivers/gpu/drm/vmwgfx/vmwgfx_ioctl.c | 38 +-
> drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 87 ++--
> drivers/staging/omapdrm/omap_debugfs.c | 2 +
> drivers/staging/omapdrm/omap_fb.c | 16 +-
> drivers/staging/omapdrm/omap_fbdev.c | 8 +-
> include/drm/drmP.h | 13 +
> include/drm/drm_crtc.h | 30 ++
> 45 files changed, 830 insertions(+), 546 deletions(-)
>
> --
> 1.7.10.4
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
prev parent reply other threads:[~2013-01-11 23:17 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-10 20:47 [PATCH 00/36] kms locking rework Daniel Vetter
2013-01-10 20:47 ` [PATCH 01/35] drm: review locking rules in drm_crtc.c Daniel Vetter
2013-01-10 20:47 ` [PATCH 02/35] drm/doc: integrate drm_crtc.c kerneldoc Daniel Vetter
2013-01-10 20:47 ` [PATCH 03/35] drm/<drivers>: reorder framebuffer init sequence Daniel Vetter
2013-01-11 21:06 ` Rob Clark
2013-01-10 20:47 ` [PATCH 04/35] drm/vmwgfx: " Daniel Vetter
2013-01-10 20:47 ` [PATCH 05/35] drm/gma500: move fbcon restore to lastclose Daniel Vetter
2013-01-10 20:47 ` [PATCH 06/35] drm/nouveau: protect evo_wait/evo_kick sections with a channel mutex Daniel Vetter
2013-01-10 20:47 ` [PATCH 07/35] drm/nouveau: try to protect nbo->pin_refcount Daniel Vetter
2013-01-10 20:47 ` [PATCH 08/35] drm/<drivers>: Unified handling of unimplemented fb->create_handle Daniel Vetter
2013-01-18 15:00 ` Thierry Reding
2013-01-18 18:32 ` Daniel Vetter
2013-01-10 20:47 ` [PATCH 09/35] drm: encapsulate crtc->set_config calls Daniel Vetter
2013-01-10 20:47 ` [PATCH 10/35] drm: add drm_modeset_lock|unlock_all Daniel Vetter
2013-01-10 20:47 ` [PATCH 11/35] drm/i915: use drm_modeset_lock_all Daniel Vetter
2013-01-10 20:47 ` [PATCH 12/35] drm/gma500: " Daniel Vetter
2013-01-10 22:36 ` Alan Cox
2013-01-11 13:25 ` Daniel Vetter
2013-01-10 20:47 ` [PATCH 13/35] drm/ast: " Daniel Vetter
2013-01-10 20:47 ` [PATCH 14/35] drm/shmobile: " Daniel Vetter
2013-01-10 20:47 ` [PATCH 15/35] drm/vmgfx: " Daniel Vetter
2013-01-10 20:47 ` [PATCH 16/35] drm: add per-crtc locks Daniel Vetter
2013-01-10 20:47 ` [PATCH 17/35] drm: only take the crtc lock for ->cursor_set Daniel Vetter
2013-01-10 20:47 ` [PATCH 18/35] drm: only take the crtc lock for ->cursor_move Daniel Vetter
2013-01-10 20:48 ` [PATCH 19/35] drm: revamp locking around fb creation/destruction Daniel Vetter
2013-01-10 20:48 ` [PATCH 20/35] drm: create drm_framebuffer_lookup Daniel Vetter
2013-01-10 20:48 ` [PATCH 21/35] drm: revamp framebuffer cleanup interfaces Daniel Vetter
2013-01-10 20:48 ` [PATCH 22/35] drm: reference framebuffers which are on the idr Daniel Vetter
2013-01-10 20:48 ` [PATCH 23/35] drm: nest modeset locks within fpriv->fbs_lock Daniel Vetter
2013-01-10 20:48 ` [PATCH 24/35] drm: push modeset_lock_all into ->fb_create driver callbacks Daniel Vetter
2013-01-10 20:48 ` [PATCH 25/35] drm: don't take modeset locks in getfb ioctl Daniel Vetter
2013-01-10 20:48 ` [PATCH 26/35] drm: fb refcounting for dirtyfb_ioctl Daniel Vetter
2013-01-10 20:48 ` [PATCH 27/35] drm: refcounting for sprite framebuffers Daniel Vetter
2013-01-10 20:48 ` [PATCH 28/35] drm: refcounting for crtc framebuffers Daniel Vetter
2013-01-10 20:48 ` [PATCH 29/35] drm/i915: dump refcount into framebuffer debugfs file Daniel Vetter
2013-01-11 22:20 ` Rob Clark
2013-01-10 20:48 ` [PATCH 30/35] drm/vmwgfx: add proper framebuffer refcounting Daniel Vetter
2013-01-10 20:48 ` [PATCH 31/35] drm: optimize drm_framebuffer_remove Daniel Vetter
2013-01-10 20:48 ` [PATCH 32/35] drm: only grab the crtc lock for pageflips Daniel Vetter
2013-01-10 20:48 ` [PATCH 33/35] drm: don't hold crtc mutexes for connector ->detect callbacks Daniel Vetter
2013-01-10 20:48 ` [PATCH 34/35] drm/doc: updates for new framebuffer lifetime rules Daniel Vetter
2013-01-10 20:48 ` [PATCH 35/35] drm/fb_helper: check whether fbcon is bound Daniel Vetter
2013-01-10 20:48 ` [PATCH 36/36] drm/i915: wake up all pageflip waiters Daniel Vetter
2013-01-10 20:50 ` Daniel Vetter
2013-01-11 23:17 ` Rob Clark [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='CAF6AEGudQXgg6cMUM8Fo==0DdAraNEgn8MHcTYw_oNLosohfTw@mail.gmail.com' \
--to=robdclark@gmail.com \
--cc=daniel.vetter@ffwll.ch \
--cc=dri-devel@lists.freedesktop.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.