From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.0 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2166BC433E0 for ; Thu, 30 Jul 2020 10:08:20 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id E1F0220809 for ; Thu, 30 Jul 2020 10:08:19 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E1F0220809 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=windriver.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 7B2386B0006; Thu, 30 Jul 2020 06:08:19 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 7630C6B0007; Thu, 30 Jul 2020 06:08:19 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 677FF6B0008; Thu, 30 Jul 2020 06:08:19 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0171.hostedemail.com [216.40.44.171]) by kanga.kvack.org (Postfix) with ESMTP id 52A536B0006 for ; Thu, 30 Jul 2020 06:08:19 -0400 (EDT) Received: from smtpin16.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id 04DAC8248D51 for ; Thu, 30 Jul 2020 10:08:19 +0000 (UTC) X-FDA: 77094317118.16.mice43_4e15ee626f7a Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin16.hostedemail.com (Postfix) with ESMTP id B3B16100E6932 for ; Thu, 30 Jul 2020 10:08:18 +0000 (UTC) X-HE-Tag: mice43_4e15ee626f7a X-Filterd-Recvd-Size: 3537 Received: from mail1.windriver.com (mail1.windriver.com [147.11.146.13]) by imf21.hostedemail.com (Postfix) with ESMTP for ; Thu, 30 Jul 2020 10:08:17 +0000 (UTC) Received: from ALA-HCB.corp.ad.wrs.com (ala-hcb.corp.ad.wrs.com [147.11.189.41]) by mail1.windriver.com (8.15.2/8.15.2) with ESMTPS id 06UA89XB019930 (version=TLSv1 cipher=DHE-RSA-AES256-SHA bits=256 verify=FAIL); Thu, 30 Jul 2020 03:08:09 -0700 (PDT) Received: from pek-lpg-core1-vm1.wrs.com (128.224.156.106) by ALA-HCB.corp.ad.wrs.com (147.11.189.41) with Microsoft SMTP Server id 14.3.487.0; Thu, 30 Jul 2020 03:07:48 -0700 From: To: , , , , CC: , Subject: [PATCH v3] mm/slab.c: add node spinlock protect in __cache_free_alien Date: Thu, 30 Jul 2020 18:19:34 +0800 Message-ID: <20200730101934.38343-1-qiang.zhang@windriver.com> X-Mailer: git-send-email 2.26.2 MIME-Version: 1.0 Content-Type: text/plain X-Rspamd-Queue-Id: B3B16100E6932 X-Spamd-Result: default: False [0.00 / 100.00] X-Rspamd-Server: rspam03 Content-Transfer-Encoding: quoted-printable X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: From: Zhang Qiang for example: node0 cpu0 cpu1 slab_dead_cpu >mutex_lock(&slab_mutex) >cpuup_canceled slab_dead_cpu >mask =3D cpumask_of_node(node) >mutex_lock(&slab_mu= tex) >n =3D get_node(cachep0, node0) >spin_lock_irq(n&->list_lock) >if (!cpumask_empty(mask)) =3D=3D true >spin_unlock_irq(&n->list_lock) >goto free_slab .... >mutex_unlock(&slab_mutex) .... >cpuup_canceled >mask =3D cpumask_of_node(node) kmem_cache_free(cachep0 ) >n =3D get_node(cachep0, node0) >__cache_free_alien(cachep0 ) >spin_lock_irq(n&->list_lock) >n =3D get_node(cachep0, node0) >if (!cpumask_empty(mask)) =3D=3D= false >if (n->alien && n->alien[page_node]) >alien =3D n->alien >alien =3D n->alien[page_node] >n->alien =3D NULL >.... >spin_unlock_irq(&n->list_lock) >.... Due to multiple cpu offline, The same cache in a node may be operated in parallel,the "n->alien" should be protect. Fixes: 6731d4f12315 ("slab: Convert to hotplug state machine") Signed-off-by: Zhang Qiang --- v1->v2->v3: change submission information and fixes tags. mm/slab.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/mm/slab.c b/mm/slab.c index a89633603b2d..290523c90b4e 100644 --- a/mm/slab.c +++ b/mm/slab.c @@ -759,8 +759,10 @@ static int __cache_free_alien(struct kmem_cache *cac= hep, void *objp, =20 n =3D get_node(cachep, node); STATS_INC_NODEFREES(cachep); + spin_lock(&n->list_lock); if (n->alien && n->alien[page_node]) { alien =3D n->alien[page_node]; + spin_unlock(&n->list_lock); ac =3D &alien->ac; spin_lock(&alien->lock); if (unlikely(ac->avail =3D=3D ac->limit)) { @@ -769,14 +771,15 @@ static int __cache_free_alien(struct kmem_cache *ca= chep, void *objp, } ac->entry[ac->avail++] =3D objp; spin_unlock(&alien->lock); - slabs_destroy(cachep, &list); } else { + spin_unlock(&n->list_lock); n =3D get_node(cachep, page_node); spin_lock(&n->list_lock); free_block(cachep, &objp, 1, page_node, &list); spin_unlock(&n->list_lock); - slabs_destroy(cachep, &list); } + + slabs_destroy(cachep, &list); return 1; } =20 --=20 2.26.2