[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CADnq5_N6ezaNjPfKSr4siJ-nKDyuVZdQCP5azuMPWeKHrAQfwg@mail.gmail.com>
Date: Thu, 28 Jul 2016 12:14:59 -0400
From: Alex Deucher <alexdeucher@...il.com>
To: Tejun Heo <tj@...nel.org>
Cc: Bhaktipriya Shridhar <bhaktipriya96@...il.com>,
Alex Deucher <alexander.deucher@....com>,
Christian König <christian.koenig@....com>,
Maling list - DRI developers
<dri-devel@...ts.freedesktop.org>,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2] drm/radeon: Remove deprecated create_singlethread_workqueue
On Mon, Jul 18, 2016 at 8:48 PM, Tejun Heo <tj@...nel.org> wrote:
> On Sat, Jul 16, 2016 at 05:00:44PM +0530, Bhaktipriya Shridhar wrote:
>>
>> alloc_workqueue replaces deprecated create_singlethread_workqueue().
>>
>> Each hardware CRTC has a single flip work queue.
>> When a radeon_flip_work_func item is queued, it needs to be executed
>> ASAP because even a slight delay may cause the flip to be delayed by
>> one refresh cycle.
>>
>> Hence, a dedicated workqueue with WQ_HIGHPRI set, has been used here
>> since a delay can cause the outcome to miss the refresh cycle.
>>
>> Since there are only a fixed number of work items, explicit concurrency
>> limit is unnecessary here.
>>
>> Signed-off-by: Bhaktipriya Shridhar <bhaktipriya96@...il.com>
>
> Acked-by: Tejun Heo <tj@...nel.org>
>
Applied. thanks!
Alex
> Thanks.
>
> --
> tejun
> _______________________________________________
> dri-devel mailing list
> dri-devel@...ts.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Powered by blists - more mailing lists