[Intel-gfx] [PATCH 1/2] drm/i915: Fix hw state verifier access to crtc->state.

2017-05-11 Thread Maarten Lankhorst
We shouldn't inspect crtc->state, instead grab the crtc state. At this point the hw state verifier should be able to run even if crtc->state has been updated (which cannot currently happen). Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/i915/intel_display.c | 24 ++--

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Fix hw state verifier access to crtc->state.

2017-05-11 Thread Daniel Vetter
On Thu, May 11, 2017 at 10:28:43AM +0200, Maarten Lankhorst wrote: > We shouldn't inspect crtc->state, instead grab the crtc state. > At this point the hw state verifier should be able to run even if > crtc->state has been updated (which cannot currently happen). > > Signed-off-by: Maarten Lankhor

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Fix hw state verifier access to crtc->state.

2017-05-11 Thread Maarten Lankhorst
Op 11-05-17 om 11:23 schreef Daniel Vetter: > On Thu, May 11, 2017 at 10:28:43AM +0200, Maarten Lankhorst wrote: >> We shouldn't inspect crtc->state, instead grab the crtc state. >> At this point the hw state verifier should be able to run even if >> crtc->state has been updated (which cannot curre

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Fix hw state verifier access to crtc->state.

2017-05-15 Thread Daniel Vetter
On Thu, May 11, 2017 at 11:41:22AM +0200, Maarten Lankhorst wrote: > Op 11-05-17 om 11:23 schreef Daniel Vetter: > > On Thu, May 11, 2017 at 10:28:43AM +0200, Maarten Lankhorst wrote: > >> We shouldn't inspect crtc->state, instead grab the crtc state. > >> At this point the hw state verifier should

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Fix hw state verifier access to crtc->state.

2017-05-16 Thread Maarten Lankhorst
Op 15-05-17 om 15:52 schreef Daniel Vetter: > On Thu, May 11, 2017 at 11:41:22AM +0200, Maarten Lankhorst wrote: >> Op 11-05-17 om 11:23 schreef Daniel Vetter: >>> On Thu, May 11, 2017 at 10:28:43AM +0200, Maarten Lankhorst wrote: We shouldn't inspect crtc->state, instead grab the crtc state.