Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1472734

[PATCH 1/9] fs/afs/vlocation: Remove deprecated create_singlethread_workqueue

Path csiph.com!eternal-september.org!feeder.eternal-september.org!aioe.org!bofh.it!news.nic.it!robomod
From Bhaktipriya Shridhar <bhaktipriya96@gmail.com>
Newsgroups linux.kernel
Subject [PATCH 1/9] fs/afs/vlocation: Remove deprecated create_singlethread_workqueue
Date Tue, 30 Aug 2016 19:50:03 +0200
Message-ID <sbVkD-7WI-69@gated-at.bofh.it> (permalink)
References <sbVkC-7WI-13@gated-at.bofh.it>
X-Original-To David Howells <dhowells@redhat.com>, Mark Fasheh <mfasheh@suse.com>, Joel Becker <jlbec@evilplan.org>, Michal Hocko <mhocko@suse.com>, Johannes Weiner <hannes@cmpxchg.org>, Vladimir Davydov <vdavydov@virtuozzo.com>, "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>, "David S. Miller" <davem@davemloft.net>, Eric Dumazet <edumazet@google.com>, Andrew Morton <akpm@linux-foundation.org>, Joseph Qi <joseph.qi@huawei.com>, "Norton.Zhu" <norton.zhu@huawei.com>, Jun Piao <piaojun@huawei.com>, xuejiufei <xuejiufei@huawei.com>
Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=ib8PBTIscvpuwrGz4tMs18u4K0TuPpjAqXpRtN6X40E=; b=CgiCdStCJO5qeBjG0RRgfXL/+Ulsjob3LCrCGbcXehosT+AVFEtle1YoMdh6hQHbpr JUbR5Yms50d7DORjzL5D1CwUqv4o+fU2jtO7ZmpNCtEF7aOtFMInl4AfdlkASjMQjaMm UwD2Ewid2FgxVSrQ9DKYbwoQ42zS4XqqcD+UTA+Yzd5f9x+Hs9SLtmsPhZWaP2a8u3pH LdNa3EAJQFsRY8fME3wla30wbzlCuMbHxN/St3zQ9ZixjYi2jDilHzNaOZHv9n83Z8pu F73y456rMHdlA6k3lIuVSJFcmTdflh6jbrKUczvWjLF0fosv60bi5usyaX95TW0rjVvQ f7Jg==
X-Google-Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=ib8PBTIscvpuwrGz4tMs18u4K0TuPpjAqXpRtN6X40E=; b=Gbx4UcIkA9mdsHHNOXNmWo3TCbN9sY+7lTAtNIbwEzkR4lLH91cuIuW1XELgcZN87n mhEcRdOyD93uilI48p0UyGkFUPuY+jZKMeNfUYV34/wukT47mIbdD6/i39vubWxSxUYm M/24dAQz8EF24Hc1PV7Lt1UNt5ceza4BKl0EbI/DTEBVEqYF4biHj5EHggiPTT7TacDZ H0ciwx6wip80CSzRW5gRPlcjV5S6jp8GTULhFN4NrJMy4bqB6wBGWztha+KS7xyfhmM+ ryTuJCMym4sJcL37yek19HgOdhve0WYqNqo3M9IkuNPvtJbcHmze5rLSmXSJEq+WV+xP HNmA==
X-Gm-Message-State AE9vXwONCkzf4PhmNX3157BNQb15eeJTG2VJSe8x9t2RMkbblF2b1QWYUDG4t7o6UnZHbg==
X-Received by 10.98.49.198 with SMTP id x189mr8335020pfx.135.1472579059119; Tue, 30 Aug 2016 10:44:19 -0700 (PDT)
MIME-Version 1.0
Content-Type text/plain; charset=us-ascii
Content-Disposition inline
User-Agent Mutt/1.5.23 (2014-03-12)
Sender robomod@news.nic.it
List-ID <linux-kernel.vger.kernel.org>
X-Mailing-List linux-kernel@vger.kernel.org
Approved robomod@news.nic.it
Lines 33
Organization linux.* mail to news gateway
X-Original-Cc ocfs2-devel@oss.oracle.com, linux-afs@lists.infradead.org, linux-kernel@vger.kernel.org, Tejun Heo <tj@kernel.org>
X-Original-Date Tue, 30 Aug 2016 23:14:16 +0530
X-Original-Message-ID <960cff8a55c83c0d8846306c1b286ec545e9e5f4.1472578959.git.bhaktipriya96@gmail.com>
X-Original-References <cover.1472578959.git.bhaktipriya96@gmail.com>
X-Original-Sender linux-kernel-owner@vger.kernel.org
Xref csiph.com linux.kernel:1472734

Show key headers only | View raw


The workqueue "afs_vlocation_update_worker" queues a single work item
&afs_vlocation_update and hence it doesn't require execution ordering.
Hence, alloc_workqueue has been used to replace the deprecated
create_singlethread_workqueue instance.

Since the workqueue is being used on a memory reclaim path, 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 <bhaktipriya96@gmail.com>
---
 fs/afs/vlocation.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/fs/afs/vlocation.c b/fs/afs/vlocation.c
index 5297678..45a8639 100644
--- a/fs/afs/vlocation.c
+++ b/fs/afs/vlocation.c
@@ -594,8 +594,8 @@ static void afs_vlocation_reaper(struct work_struct *work)
  */
 int __init afs_vlocation_update_init(void)
 {
-	afs_vlocation_update_worker =
-		create_singlethread_workqueue("kafs_vlupdated");
+	afs_vlocation_update_worker = alloc_workqueue("kafs_vlupdated",
+						      WQ_MEM_RECLAIM, 0);
 	return afs_vlocation_update_worker ? 0 : -ENOMEM;
 }

--
2.1.4

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 0/9] Remove deprecated create_singlethread_workqueue Bhaktipriya Shridhar <bhaktipriya96@gmail.com> - 2016-08-30 19:50 +0200
  [PATCH 1/9] fs/afs/vlocation: Remove deprecated  create_singlethread_workqueue Bhaktipriya Shridhar <bhaktipriya96@gmail.com> - 2016-08-30 19:50 +0200
  Re: [PATCH 0/9] Remove deprecated create_singlethread_workqueue David Howells <dhowells@redhat.com> - 2016-08-30 21:40 +0200
    Re: [PATCH 0/9] Remove deprecated create_singlethread_workqueue Bhaktipriya Shridhar <bhaktipriya96@gmail.com> - 2016-08-30 22:00 +0200

csiph-web