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. Thanks for the explanation! -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html