[Intel-gfx] [PATCH v2 0/4] drm/exynos, intel: fix locking for flip/vbl event list
Inki Dae
inki.dae at samsung.com
Wed Nov 7 10:31:14 CET 2012
2012/11/2 Imre Deak <imre.deak at intel.com>
> The patchset adds the missing event_lock when accessing the
> vblank_event_list in drm_vblank_off() and as preparation for this
> also fixes a few other issues in the exynos driver.
>
> This is also a dependency for Rob Clark's drm_send_vblank_event()
> rework as that would trigger a warning for the unhold event_lock without
> this changeset.
>
> The exynos changes are only compile tested, the rest is tested on an
> Intel IVB machine on top of drm-intel-nightly + drm_send_vblank_event()
> rework, with i-g-t/flip_test.
>
>
Hi Imre,
Works fine. But we should wait for Rob's patch set to be merged to -next,
and this may be rebased on top of latest Rob's patch set again.
Thanks,
Inki Dae
> In v2:
> - Instead of fixing the event_lock vs. vblank_time_lock lockdep issue in
> drm_handle_vblank(), fix it in the exynos driver. This looks like the
> more logical thing to do and this way we also have a smaller impact on
> the rest of DRM code.
>
> Imre Deak (4):
> drm/exynos: hold event_lock while accessing pageflip_event_list
> drm/exynos: call drm_vblank_put for each queued flip event
> drm/exynos: fix lockdep for event_lock wrt. vbl_time_lock
> drm: hold event_lock while accessing vblank_event_list
>
> drivers/gpu/drm/drm_irq.c | 3 +++
> drivers/gpu/drm/exynos/exynos_drm_crtc.c | 5 +++++
> drivers/gpu/drm/exynos/exynos_drm_fimd.c | 20 +-------------------
> drivers/gpu/drm/exynos/exynos_drm_vidi.c | 20 +-------------------
> drivers/gpu/drm/exynos/exynos_mixer.c | 11 +----------
> 5 files changed, 11 insertions(+), 48 deletions(-)
>
> --
> 1.7.9.5
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20121107/88c1e277/attachment.html>
More information about the Intel-gfx
mailing list