Re: [Intel-gfx] [PATCH v2 4/5] drm/i915: Change locking for struct_mutex, v2.

2015-11-02 Thread Maarten Lankhorst
Op 02-11-15 om 14:06 schreef Chris Wilson: > On Mon, Nov 02, 2015 at 01:57:59PM +0100, Maarten Lankhorst wrote: >> struct_mutex is being locked for every plane in intel_prepare_plane_fb and >> intel_cleanup_plane_fb. This can be optimized by acquiring struct_mutex first >> before calling the

[Intel-gfx] [PATCH v2 4/5] drm/i915: Change locking for struct_mutex, v2.

2015-11-02 Thread Maarten Lankhorst
struct_mutex is being locked for every plane in intel_prepare_plane_fb and intel_cleanup_plane_fb. This can be optimized by acquiring struct_mutex first before calling the atomic helpers. This way the lock only needs to be acquired twice in ->atomic_commit(). Once for pinning new framebuffers at

Re: [Intel-gfx] [PATCH v2 4/5] drm/i915: Change locking for struct_mutex, v2.

2015-11-02 Thread Chris Wilson
On Mon, Nov 02, 2015 at 01:57:59PM +0100, Maarten Lankhorst wrote: > struct_mutex is being locked for every plane in intel_prepare_plane_fb and > intel_cleanup_plane_fb. This can be optimized by acquiring struct_mutex first > before calling the atomic helpers. This way the lock only needs to be