From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757031AbcH3Uvc (ORCPT ); Tue, 30 Aug 2016 16:51:32 -0400 Received: from mail-pa0-f67.google.com ([209.85.220.67]:33182 "EHLO mail-pa0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751574AbcH3Uv3 (ORCPT ); Tue, 30 Aug 2016 16:51:29 -0400 Date: Wed, 31 Aug 2016 02:21:26 +0530 From: Bhaktipriya Shridhar To: Mark Fasheh , Joel Becker , Andrew Morton , Joseph Qi , xuejiufei , "Norton.Zhu" , Jun Piao , "David S. Miller" , Eric Dumazet , Michal Hocko , Vladimir Davydov , Johannes Weiner , "Kirill A. Shutemov" Cc: ocfs2-devel@oss.oracle.com, linux-kernel@vger.kernel.org, Tejun Heo Subject: [PATCH v2 4/4] fs/ocfs2/dlm: Remove deprecated create_singlethread_workqueue Message-ID: <2b5ad8d6688effe1a9ddb2bc2082d26fbbe00302.1472590094.git.bhaktipriya96@gmail.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The workqueue "dlm_worker" queues a single work item &dlm->dispatched_work and thus it doesn't require execution ordering. Hence, alloc_workqueue has been used to replace the deprecated create_singlethread_workqueue instance. The WQ_MEM_RECLAIM flag has been set to ensure forward progress under memory pressure. Since there are fixed number of work items, explicit concurrency limit is unnecessary here. Signed-off-by: Bhaktipriya Shridhar --- Changes in v2: -No change fs/ocfs2/dlm/dlmdomain.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/ocfs2/dlm/dlmdomain.c b/fs/ocfs2/dlm/dlmdomain.c index 12e064b..dc30f38 100644 --- a/fs/ocfs2/dlm/dlmdomain.c +++ b/fs/ocfs2/dlm/dlmdomain.c @@ -1906,7 +1906,7 @@ static int dlm_join_domain(struct dlm_ctxt *dlm) } snprintf(wq_name, O2NM_MAX_NAME_LEN, "dlm_wq-%s", dlm->name); - dlm->dlm_worker = create_singlethread_workqueue(wq_name); + dlm->dlm_worker = alloc_workqueue(wq_name, WQ_MEM_RECLAIM, 0); if (!dlm->dlm_worker) { status = -ENOMEM; mlog_errno(status); -- 2.1.4