Received: by 2002:a05:6a10:1287:0:0:0:0 with SMTP id d7csp3637561pxv; Mon, 19 Jul 2021 05:19:08 -0700 (PDT) X-Google-Smtp-Source: ABdhPJycXvpO01McavsPsEmpZJ7E8jI/FPAgv44Y2DrMheFvK+py42y1dOBi83LVwP6tnHVDjMhi X-Received: by 2002:a92:d141:: with SMTP id t1mr16500913ilg.171.1626697147933; Mon, 19 Jul 2021 05:19:07 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1626697147; cv=none; d=google.com; s=arc-20160816; b=a7V7VMS7X/nyeFL/x6cq/fG0ZjjdJZ6iMNhpB8Dof8zqbL/GRlGYr4mLQeSfdKn9JL PTQwCAlkCCsbeqBp6Vkem8Z4yXuVeXu/boSfQ0H2nwgZiOqg8YrCtjmYzOkZ847PS/W2 atDrvLA8CIjFBW6VjhHHPth+6NaAu2Q1njRwdrSJw3Kud/N62L3N6/vGTngeURuyQKQC KP4WaPL+LgTt9+OdHtXi/fIJY2RwM7/gaRWnCTFHfWGl0COMGSundPr8oA3BpaxIJkhK UW2C+Eun8xO0Fegf+yvv6Ql7Sr0JYchZfW+ZOTDv+jw5Knfz4copENmIn1lr0mTzn5Cf TSEg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject; bh=kbvn/14Jc6h3g00mCJXeCZtLa6u81CoEsHZ4NaO1KJM=; b=m8QhMLhE//Tn3gEjAS/D05++r1h/ivF/vb02rE7PfLHWWzhRPpsj7HpuXvJC9+XdnQ 5YXZ/UDhDX2Q+pd+BVEpYvzVNgfVNx2nrYRcNRa/WAo2GefK6a7WDE4qlSgSV0NtOINc R0wfRo2GKHBlmpxpyIqO3uyZ+LhFcVN+N5LSQjTAcNmpkgsWrQHiYY0ZlVRdlqYT8Y7Z KYJoQ/STodt/zDVdb/mtXMvlrw1ixJRbr6QJTd/UyUf+TQduLuE+7UIAEUSKF/S5fDVq Bbjv8hWcfViM8wBI030UYZMIAwJGaS5ssT/499ea+LNZoB6RmzGOv4ycDFy4OGUasFIN 9uSw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=huawei.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id p7si11574595iop.92.2021.07.19.05.18.56; Mon, 19 Jul 2021 05:19:07 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=huawei.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236925AbhGSLhF (ORCPT + 99 others); Mon, 19 Jul 2021 07:37:05 -0400 Received: from szxga02-in.huawei.com ([45.249.212.188]:7391 "EHLO szxga02-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236918AbhGSLhE (ORCPT ); Mon, 19 Jul 2021 07:37:04 -0400 Received: from dggeme703-chm.china.huawei.com (unknown [172.30.72.53]) by szxga02-in.huawei.com (SkyGuard) with ESMTP id 4GT13c3yQpz7whb; Mon, 19 Jul 2021 20:14:04 +0800 (CST) Received: from [10.174.177.180] (10.174.177.180) by dggeme703-chm.china.huawei.com (10.1.199.99) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2; Mon, 19 Jul 2021 20:17:41 +0800 Subject: Re: linux-5.13.2: warning from kernel/rcu/tree_plugin.h:359 To: Oleksandr Natalenko , Matthew Wilcox CC: Boqun Feng , Zhouyi Zhou , , linux-kernel , , Chris Clayton , Chris Rankin , Josh Triplett , Steven Rostedt , Mathieu Desnoyers , Lai Jiangshan , Joel Fernandes , rcu , Andrew Morton , Linux-MM , "Huang, Ying" , References: <2145858.R0O0FObHBN@natalenko.name> <688a2cb1-5cd8-2c00-889c-4d48021371f8@huawei.com> <5812280.fcLxn8YiTP@natalenko.name> From: Miaohe Lin Message-ID: <8c2317d0-0956-8f05-d6bf-bcd381c8fcee@huawei.com> Date: Mon, 19 Jul 2021 20:17:41 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.6.0 MIME-Version: 1.0 In-Reply-To: <5812280.fcLxn8YiTP@natalenko.name> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 8bit X-Originating-IP: [10.174.177.180] X-ClientProxiedBy: dggems706-chm.china.huawei.com (10.3.19.183) To dggeme703-chm.china.huawei.com (10.1.199.99) X-CFilter-Loop: Reflected Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2021/7/19 20:12, Oleksandr Natalenko wrote: > On pondělí 19. července 2021 14:08:37 CEST Miaohe Lin wrote: >> On 2021/7/19 19:59, Oleksandr Natalenko wrote: >>> On pondělí 19. července 2021 13:50:07 CEST Miaohe Lin wrote: >>>> On 2021/7/19 19:22, Matthew Wilcox wrote: >>>>> On Mon, Jul 19, 2021 at 07:12:58PM +0800, Miaohe Lin wrote: >>>>>> When in the commit 2799e77529c2a, we're using the percpu_ref to >>>>>> serialize >>>>>> against concurrent swapoff, i.e. there's percpu_ref inside >>>>>> get_swap_device() instead of rcu_read_lock(). Please see commit >>>>>> 63d8620ecf93 ("mm/swapfile: use percpu_ref to serialize against >>>>>> concurrent swapoff") for detail. >>>>> >>>>> Oh, so this is a backport problem. 2799e77529c2 was backported without >>>>> its prerequisite 63d8620ecf93. Greg, probably best to just drop >>>> >>>> Yes, they're posted as a patch set: >>>> >>>> https://lkml.kernel.org/r/20210426123316.806267-1-linmiaohe@huawei.com >>>> >>>>> 2799e77529c2 from all stable trees; the race described is not very >>>>> important (swapoff vs reading a page back from that swap device). >>>>> . >>>> >>>> The swapoff races with reading a page back from that swap device should >>>> be >>>> really uncommon as most users only do swapoff when the system is going to >>>> shutdown. >>>> >>>> Sorry for the trouble! >>> >>> git log --oneline v5.13..v5.13.3 --author="Miaohe Lin" >>> 11ebc09e50dc mm/zswap.c: fix two bugs in zswap_writeback_entry() >>> 95d192da198d mm/z3fold: use release_z3fold_page_locked() to release locked >>> z3fold page >>> ccb7848e2344 mm/z3fold: fix potential memory leak in z3fold_destroy_pool() >>> 9f7229c901c1 mm/huge_memory.c: don't discard hugepage if other processes >>> are mapping it >>> f13259175e4f mm/huge_memory.c: add missing read-only THP checking in >>> transparent_hugepage_enabled() >>> afafd371e7de mm/huge_memory.c: remove dedicated macro >>> HPAGE_CACHE_INDEX_MASK a533a21b692f mm/shmem: fix shmem_swapin() race >>> with swapoff >>> c3b39134bbd0 swap: fix do_swap_page() race with swapoff >>> >>> Do you suggest reverting "mm/shmem: fix shmem_swapin() race with swapoff" >>> as well? >> >> This patch also rely on its prerequisite 63d8620ecf93. I think we should >> either revert any commit in this series or just backport the entire series. > > Then why not just pick up 2 more patches instead of dropping 2 patches. Greg, > could you please make sure the whole series from [1] gets pulled? > > Thanks. > > [1] https://lkml.kernel.org/r/20210426123316.806267-1-linmiaohe@huawei.com > I browsed the previous backport notifying email and found that these two patches are also backported into 5.10 and 5.12. So we might also need to make sure the whole series from [1] gets pulled in 5.10 and 5.12. Thanks.