Re: [Intel-gfx] [PATCH 05/14] drm/i915: Remove delay for idle_work

2019-05-02 Thread Chris Wilson
Quoting Tvrtko Ursulin (2019-05-02 14:51:31) > > On 02/05/2019 14:22, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2019-05-02 14:19:38) > >> > >> On 01/05/2019 12:45, Chris Wilson wrote: > >>> diff --git a/drivers/gpu/drm/i915/i915_gem_pm.c > >>> b/drivers/gpu/drm/i915/i915_gem_pm.c > >>>

Re: [Intel-gfx] [PATCH 05/14] drm/i915: Remove delay for idle_work

2019-05-02 Thread Tvrtko Ursulin
On 02/05/2019 14:22, Chris Wilson wrote: Quoting Tvrtko Ursulin (2019-05-02 14:19:38) On 01/05/2019 12:45, Chris Wilson wrote: diff --git a/drivers/gpu/drm/i915/i915_gem_pm.c b/drivers/gpu/drm/i915/i915_gem_pm.c index 49b0ce594f20..ae91ad7cb31e 100644 ---

Re: [Intel-gfx] [PATCH 05/14] drm/i915: Remove delay for idle_work

2019-05-02 Thread Chris Wilson
Quoting Tvrtko Ursulin (2019-05-02 14:19:38) > > On 01/05/2019 12:45, Chris Wilson wrote: > > diff --git a/drivers/gpu/drm/i915/i915_gem_pm.c > > b/drivers/gpu/drm/i915/i915_gem_pm.c > > index 49b0ce594f20..ae91ad7cb31e 100644 > > --- a/drivers/gpu/drm/i915/i915_gem_pm.c > > +++

Re: [Intel-gfx] [PATCH 05/14] drm/i915: Remove delay for idle_work

2019-05-02 Thread Tvrtko Ursulin
On 01/05/2019 12:45, Chris Wilson wrote: The original intent for the delay before running the idle_work was to provide a hysteresis to avoid ping-ponging the device runtime-pm. Since then we have also pulled in some memory management and general device management for parking. But with the

[Intel-gfx] [PATCH 05/14] drm/i915: Remove delay for idle_work

2019-05-01 Thread Chris Wilson
The original intent for the delay before running the idle_work was to provide a hysteresis to avoid ping-ponging the device runtime-pm. Since then we have also pulled in some memory management and general device management for parking. But with the inversion of the wakeref handling, GEM is no