lkml.org 
[lkml]   [2016]   [Jun]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [RFC] IB/srp: Remove create_workqueue
From
Date
On 06/07/2016 12:21 PM, Tejun Heo wrote:
> On Tue, Jun 07, 2016 at 11:32:42AM -0700, Bart Van Assche wrote:
>> On 06/07/2016 11:16 AM, Bhaktipriya Shridhar wrote:
>>> alloc_workqueue replaces deprecated create_workqueue().
>>>
>>> A dedicated workqueue has been used since the workqueue srp_remove_wq with
>>> workitem &target->remove_work, is a work queue for the SRP target removal.
>>> WQ_MEM_RECLAIM has been set to ensure forward progress under memory
>>> pressure.
>>> Since there are only a fixed number of work items, explicit
>>> concurrency limit is unnecessary here.
>>>
>>> Is the workqueue being used on a memory reclaim path?
>>> Does it require WQ_MEM_RECLAIM?
>>
>> Hello Bhaktipriya,
>>
>> srp_remove_wq is used for SRP target port removal work only. This work is
>> neither queued from inside a shrinker nor by the page writeback code so I
>> think it is safe to drop WQ_MEM_RECLAIM.
>
> It should be able to use system_wq then.

No. I have tried that but that resulted in a deadlock.

Bart.

\
 
 \ /
  Last update: 2016-06-07 22:01    [W:0.076 / U:0.260 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site