Re: [Intel-gfx] [PATCH v2 2/3] drm/i915: add a dedicated workqueue inside drm_i915_private

2023-05-28 Thread Coelho, Luciano
On Fri, 2023-05-26 at 14:30 +0300, Jani Nikula wrote: > On Wed, 24 May 2023, Tvrtko Ursulin wrote: > > On 24/05/2023 10:05, Luca Coelho wrote: > > > In order to avoid flush_scheduled_work() usage, add a dedicated > > > workqueue in the drm_i915_private structure. In this way, we don't > > > need

Re: [Intel-gfx] [PATCH v2 2/3] drm/i915: add a dedicated workqueue inside drm_i915_private

2023-05-26 Thread Jani Nikula
On Wed, 24 May 2023, Tvrtko Ursulin wrote: > On 24/05/2023 10:05, Luca Coelho wrote: >> In order to avoid flush_scheduled_work() usage, add a dedicated >> workqueue in the drm_i915_private structure. In this way, we don't >> need to use the system queue anymore. >> >> This change is mostly mecha

Re: [Intel-gfx] [PATCH v2 2/3] drm/i915: add a dedicated workqueue inside drm_i915_private

2023-05-24 Thread Coelho, Luciano
On Wed, 2023-05-24 at 12:00 +0100, Tvrtko Ursulin wrote: > On 24/05/2023 10:05, Luca Coelho wrote: > > In order to avoid flush_scheduled_work() usage, add a dedicated > > workqueue in the drm_i915_private structure. In this way, we don't > > need to use the system queue anymore. > > > > This chan

Re: [Intel-gfx] [PATCH v2 2/3] drm/i915: add a dedicated workqueue inside drm_i915_private

2023-05-24 Thread Tvrtko Ursulin
On 24/05/2023 12:00, Tvrtko Ursulin wrote: On 24/05/2023 10:05, Luca Coelho wrote: 8<   if (pool_free_older_than(pool, HZ)) -    schedule_delayed_work(&pool->work, -  round_jiffies_up_relative(HZ)); +    queue_delayed_work(gt->i915->unordered_wq, &pool->work

Re: [Intel-gfx] [PATCH v2 2/3] drm/i915: add a dedicated workqueue inside drm_i915_private

2023-05-24 Thread Tvrtko Ursulin
On 24/05/2023 10:05, Luca Coelho wrote: In order to avoid flush_scheduled_work() usage, add a dedicated workqueue in the drm_i915_private structure. In this way, we don't need to use the system queue anymore. This change is mostly mechanical and based on Tetsuo's original patch[1]. Link: htt

[Intel-gfx] [PATCH v2 2/3] drm/i915: add a dedicated workqueue inside drm_i915_private

2023-05-24 Thread Luca Coelho
In order to avoid flush_scheduled_work() usage, add a dedicated workqueue in the drm_i915_private structure. In this way, we don't need to use the system queue anymore. This change is mostly mechanical and based on Tetsuo's original patch[1]. Link: https://patchwork.freedesktop.org/series/114608