Re: [Intel-gfx] [PATCH] drm/atomic-helper: Simplify commit tracking locking

2017-06-27 Thread Daniel Vetter
On Wed, Jun 21, 2017 at 10:30:51AM -0400, Sean Paul wrote: > On Wed, Jun 21, 2017 at 11:16:27AM +0200, Daniel Vetter wrote: > > The crtc->commit_lock only protects commit_list and commit_entry. If > > we chase the pointer from the drm_atomic_state update structure, then > > we don't need any locks

Re: [Intel-gfx] [PATCH] drm/atomic-helper: Simplify commit tracking locking

2017-06-21 Thread Sean Paul
On Wed, Jun 21, 2017 at 11:16:27AM +0200, Daniel Vetter wrote: > The crtc->commit_lock only protects commit_list and commit_entry. If > we chase the pointer from the drm_atomic_state update structure, then > we don't need any locks (since we hold a reference already). > > Simplify the locking acco

[PATCH] drm/atomic-helper: Simplify commit tracking locking

2017-06-21 Thread Daniel Vetter
The crtc->commit_lock only protects commit_list and commit_entry. If we chase the pointer from the drm_atomic_state update structure, then we don't need any locks (since we hold a reference already). Simplify the locking accordingly. Noticed while reviewing a patch from Boris. Cc: Boris Brezillo