Re: [Intel-gfx] [PATCH] drm/i915: Be defensive and don't assume PSR has any commit to sync against

2018-09-05 Thread Pandiyan, Dhinakaran
On Wed, 2018-09-05 at 12:51 +0200, Maarten Lankhorst wrote: > Op 05-09-18 om 12:22 schreef Ville Syrjälä: > > On Tue, Sep 04, 2018 at 08:54:03PM +, Pandiyan, Dhinakaran > > wrote: > > > On Tue, 2018-09-04 at 19:12 +0100, Chris Wilson wrote: > > > > Quoting Ville Syrjälä (2018-09-04 19:06:29) >

Re: [Intel-gfx] [PATCH] drm/i915: Be defensive and don't assume PSR has any commit to sync against

2018-09-05 Thread Maarten Lankhorst
Op 05-09-18 om 12:22 schreef Ville Syrjälä: > On Tue, Sep 04, 2018 at 08:54:03PM +, Pandiyan, Dhinakaran wrote: >> On Tue, 2018-09-04 at 19:12 +0100, Chris Wilson wrote: >>> Quoting Ville Syrjälä (2018-09-04 19:06:29) On Tue, Sep 04, 2018 at 08:59:32PM +0300, Ville Syrjälä wrote: > On

Re: [Intel-gfx] [PATCH] drm/i915: Be defensive and don't assume PSR has any commit to sync against

2018-09-05 Thread Ville Syrjälä
On Tue, Sep 04, 2018 at 08:54:03PM +, Pandiyan, Dhinakaran wrote: > On Tue, 2018-09-04 at 19:12 +0100, Chris Wilson wrote: > > Quoting Ville Syrjälä (2018-09-04 19:06:29) > > > On Tue, Sep 04, 2018 at 08:59:32PM +0300, Ville Syrjälä wrote: > > > > On Tue, Sep 04, 2018 at 06:44:14PM +0100, Chris

Re: [Intel-gfx] [PATCH] drm/i915: Be defensive and don't assume PSR has any commit to sync against

2018-09-04 Thread Pandiyan, Dhinakaran
On Tue, 2018-09-04 at 19:12 +0100, Chris Wilson wrote: > Quoting Ville Syrjälä (2018-09-04 19:06:29) > > On Tue, Sep 04, 2018 at 08:59:32PM +0300, Ville Syrjälä wrote: > > > On Tue, Sep 04, 2018 at 06:44:14PM +0100, Chris Wilson wrote: > > > > Quoting Ville Syrjälä (2018-09-04 18:39:53) > > > > > O

Re: [Intel-gfx] [PATCH] drm/i915: Be defensive and don't assume PSR has any commit to sync against

2018-09-04 Thread Chris Wilson
Quoting Ville Syrjälä (2018-09-04 19:06:29) > On Tue, Sep 04, 2018 at 08:59:32PM +0300, Ville Syrjälä wrote: > > On Tue, Sep 04, 2018 at 06:44:14PM +0100, Chris Wilson wrote: > > > Quoting Ville Syrjälä (2018-09-04 18:39:53) > > > > On Tue, Sep 04, 2018 at 05:29:02PM +0100, Chris Wilson wrote: > >

Re: [Intel-gfx] [PATCH] drm/i915: Be defensive and don't assume PSR has any commit to sync against

2018-09-04 Thread Ville Syrjälä
On Tue, Sep 04, 2018 at 08:59:32PM +0300, Ville Syrjälä wrote: > On Tue, Sep 04, 2018 at 06:44:14PM +0100, Chris Wilson wrote: > > Quoting Ville Syrjälä (2018-09-04 18:39:53) > > > On Tue, Sep 04, 2018 at 05:29:02PM +0100, Chris Wilson wrote: > > > > If the previous modeset commit has completed and

Re: [Intel-gfx] [PATCH] drm/i915: Be defensive and don't assume PSR has any commit to sync against

2018-09-04 Thread Ville Syrjälä
On Tue, Sep 04, 2018 at 06:44:14PM +0100, Chris Wilson wrote: > Quoting Ville Syrjälä (2018-09-04 18:39:53) > > On Tue, Sep 04, 2018 at 05:29:02PM +0100, Chris Wilson wrote: > > > If the previous modeset commit has completed and is no longer part of > > > the crtc state, skip waiting for it. > > >

Re: [Intel-gfx] [PATCH] drm/i915: Be defensive and don't assume PSR has any commit to sync against

2018-09-04 Thread Chris Wilson
Quoting Ville Syrjälä (2018-09-04 18:39:53) > On Tue, Sep 04, 2018 at 05:29:02PM +0100, Chris Wilson wrote: > > If the previous modeset commit has completed and is no longer part of > > the crtc state, skip waiting for it. > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107792 > > F

Re: [Intel-gfx] [PATCH] drm/i915: Be defensive and don't assume PSR has any commit to sync against

2018-09-04 Thread Ville Syrjälä
On Tue, Sep 04, 2018 at 05:29:02PM +0100, Chris Wilson wrote: > If the previous modeset commit has completed and is no longer part of > the crtc state, skip waiting for it. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107792 > Fixes: c44301fce614 ("drm/i915: Allow control of PSR at r

[Intel-gfx] [PATCH] drm/i915: Be defensive and don't assume PSR has any commit to sync against

2018-09-04 Thread Chris Wilson
If the previous modeset commit has completed and is no longer part of the crtc state, skip waiting for it. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107792 Fixes: c44301fce614 ("drm/i915: Allow control of PSR at runtime through debugfs, v6") Signed-off-by: Chris Wilson Cc: Maarten L