From patchwork Wed Jan 24 08:40:14 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Miaohe Lin X-Patchwork-Id: 13528800 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 22300C46CD2 for ; Wed, 24 Jan 2024 08:40:39 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 7BCEF6B0075; Wed, 24 Jan 2024 03:40:38 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 76D086B007B; Wed, 24 Jan 2024 03:40:38 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 659616B007D; Wed, 24 Jan 2024 03:40:38 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id 51FC66B0075 for ; Wed, 24 Jan 2024 03:40:38 -0500 (EST) Received: from smtpin10.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay07.hostedemail.com (Postfix) with ESMTP id E7B90160159 for ; Wed, 24 Jan 2024 08:40:37 +0000 (UTC) X-FDA: 81713558514.10.0379196 Received: from szxga03-in.huawei.com (szxga03-in.huawei.com [45.249.212.189]) by imf24.hostedemail.com (Postfix) with ESMTP id 603F618000F for ; Wed, 24 Jan 2024 08:40:34 +0000 (UTC) Authentication-Results: imf24.hostedemail.com; dkim=none; dmarc=pass (policy=quarantine) header.from=huawei.com; spf=pass (imf24.hostedemail.com: domain of linmiaohe@huawei.com designates 45.249.212.189 as permitted sender) smtp.mailfrom=linmiaohe@huawei.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1706085636; a=rsa-sha256; cv=none; b=sxYTYufWa7XRPDAQEpva+kiBc/cbfPvw2r46UcVdZhXeGrL5gV2ah20FXKOMxQZIoioy4z GBIqtU/hEc8y7Ihvv0FVJYVJIg6zZN2HPBdNza2ZNYMb/f4N2HOndcKr5EB9vSsnF22N// l0F3N0yXhmRfGo7eur8npzczbrXPVCY= ARC-Authentication-Results: i=1; imf24.hostedemail.com; dkim=none; dmarc=pass (policy=quarantine) header.from=huawei.com; spf=pass (imf24.hostedemail.com: domain of linmiaohe@huawei.com designates 45.249.212.189 as permitted sender) smtp.mailfrom=linmiaohe@huawei.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1706085636; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding:in-reply-to: references; bh=zs7C/e7+EHbK1/qUFsb9vXV9nCGnOOzy5GU+JHOv8zg=; b=ZKihx7huW8gztLtWF3RwC25yuK9rRBOufGMXCzu7+Lepi8y0TSIQ8ygvvzHVswRKfEdjxW EM/GVmgvHQ+b1goZQx1u7DtPbqrrL8B9RnlVoekXBQELtNMwIDb03HlsENgBRchs37+HY6 rmxt9GHSE6aMxxBGHMQYssSE8PuSUSg= Received: from mail.maildlp.com (unknown [172.19.88.105]) by szxga03-in.huawei.com (SkyGuard) with ESMTP id 4TKcp15lpbzNlRk; Wed, 24 Jan 2024 16:39:37 +0800 (CST) Received: from canpemm500002.china.huawei.com (unknown [7.192.104.244]) by mail.maildlp.com (Postfix) with ESMTPS id 93F5A14011A; Wed, 24 Jan 2024 16:40:30 +0800 (CST) Received: from huawei.com (10.173.135.154) by canpemm500002.china.huawei.com (7.192.104.244) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Wed, 24 Jan 2024 16:40:30 +0800 From: Miaohe Lin To: , , CC: , , Subject: [PATCH v2] mm/memory-failure: fix crash in split_huge_page_to_list from soft_offline_page Date: Wed, 24 Jan 2024 16:40:14 +0800 Message-ID: <20240124084014.1772906-1-linmiaohe@huawei.com> X-Mailer: git-send-email 2.33.0 MIME-Version: 1.0 X-Originating-IP: [10.173.135.154] X-ClientProxiedBy: dggems706-chm.china.huawei.com (10.3.19.183) To canpemm500002.china.huawei.com (7.192.104.244) X-Rspamd-Server: rspam08 X-Rspamd-Queue-Id: 603F618000F X-Stat-Signature: 97jm5r73x9xfhw6yubc4b548kn1j8wg4 X-Rspam-User: X-HE-Tag: 1706085634-563185 X-HE-Meta: U2FsdGVkX19w3v9MGB0JZJ9XgZUK08LTKOmfFk1eFlWfGv6nV/ILSIGwCmxKYElYRjIKbmwfvVoMsEvpd/R3YOt+w2pCVPTL82RDsTDbpL8ptyWll0+IU84pSf03anO2KOeXiA7eWHUcVdXYAcYvLp43CIuSIvT2mEFY2UdzkqEX4X8qJNpvRyEBbDKuQw+IojUn2H6owytcQ0YgHe8Rql7qIkNObdLM/Vvg9zIfsW/9eg9W8c6Q8AhLF68IJyUxWSCwGnOA8GoHFGU5eXnvUxd9jl455FhVL/CGFZ6RasmlyUoHluZEZnxhNDbXfKRgoBovN4iTihX30P7k7i4z+O3/cAbzVieXnL3DE0GoJN+I/5MNrPc/lfOW2dpoi7meN0QP9+cKy+89j416fAHNrDgiwSmVY14iY4c1LHehZZNK7dnhzk1P3az5e/Qa2BJHvV4ITjRSoCQc+JfuGYrWdplXwMkN+wi/UYy8eOmQRgG+AdU+mDaRTktOVw5QQuXxYRe0aJ8NSEy52K1dzXU4qGJU8s+rcK3ReCUVy+4GpEdhZK5roLe9ExpPquSMBx9RlFiiu2fPfQrIDbTx6shKmbJlGPthKiamHx/KPZzGDcqy3oOimVSWNL6lJDTf8qAOzXXHnlaX/CbRoIUu+LYxnnUS3kVSOVxWsLTkkynWTJTkfVkTUM+RjgOGfw69Y83szEjsl28s/DnE4ZBY5bTaQz2O/C5H6VuFOcmDnxNt6hdSFcwP1UuktAbhNI6ZbrYmgnpBPVN8DWn/TcY1mJ3FmIum6O/aQn0SdTd96x8KfUa5/giufGvGuZJE0vykL6s6UlH0xlMBOVYDuuVU9BpI5E5PjF09qvfl6VHyMpSM8hbXagnOcWmxDrHp/+kMx/FZqnWing8FEEs= 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: List-Subscribe: List-Unsubscribe: When I did soft offline stress test, a machine was observed to crash with the following message: kernel BUG at include/linux/memcontrol.h:554! invalid opcode: 0000 [#1] PREEMPT SMP NOPTI CPU: 5 PID: 3837 Comm: hwpoison.sh Not tainted 6.7.0-next-20240112-00001-g8ecf3e7fb7c8-dirty #97 Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS rel-1.14.0-0-g155821a1990b-prebuilt.qemu.org 04/01/2014 RIP: 0010:folio_memcg+0xaf/0xd0 Code: 10 5b 5d c3 cc cc cc cc 48 c7 c6 08 b1 f2 b2 48 89 ef e8 b4 c5 f8 ff 90 0f 0b 48 c7 c6 d0 b0 f2 b2 48 89 ef e8 a2 c5 f8 ff 90 <0f> 0b 48 c7 c6 08 b1 f2 b2 48 89 ef e8 90 c5 f8 ff 90 0f 0b 66 66 RSP: 0018:ffffb6c043657c98 EFLAGS: 00000296 RAX: 000000000000004b RBX: ffff932bc1d1e401 RCX: ffff933abfb5c908 RDX: 0000000000000000 RSI: 0000000000000027 RDI: ffff933abfb5c900 RBP: ffffea6f04019080 R08: ffffffffb3338ce8 R09: 0000000000009ffb R10: 00000000000004dd R11: ffffffffb3308d00 R12: ffffea6f04019080 R13: ffffea6f04019080 R14: 0000000000000001 R15: ffffb6c043657da0 FS: 00007f6c60f6b740(0000) GS:ffff933abfb40000(0000) knlGS:0000000000000000 CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 CR2: 0000559c3bc8b980 CR3: 0000000107f1c000 CR4: 00000000000006f0 Call Trace: ? die+0x32/0x90 ? do_trap+0xde/0x110 ? folio_memcg+0xaf/0xd0 ? do_error_trap+0x60/0x80 ? folio_memcg+0xaf/0xd0 ? exc_invalid_op+0x53/0x70 ? folio_memcg+0xaf/0xd0 ? asm_exc_invalid_op+0x1a/0x20 ? folio_memcg+0xaf/0xd0 ? folio_memcg+0xae/0xd0 split_huge_page_to_list+0x4d/0x1380 ? sysvec_apic_timer_interrupt+0xf/0x80 try_to_split_thp_page+0x3a/0xf0 soft_offline_page+0x1ea/0x8a0 soft_offline_page_store+0x52/0x90 kernfs_fop_write_iter+0x118/0x1b0 vfs_write+0x30b/0x430 ksys_write+0x5e/0xe0 do_syscall_64+0xb0/0x1b0 entry_SYSCALL_64_after_hwframe+0x6d/0x75 RIP: 0033:0x7f6c60d14697 Code: 10 00 f7 d8 64 89 02 48 c7 c0 ff ff ff ff eb b7 0f 1f 00 f3 0f 1e fa 64 8b 04 25 18 00 00 00 85 c0 75 10 b8 01 00 00 00 0f 05 <48> 3d 00 f0 ff ff 77 51 c3 48 83 ec 28 48 89 54 24 18 48 89 74 24 RSP: 002b:00007ffe9b72b8d8 EFLAGS: 00000246 ORIG_RAX: 0000000000000001 RAX: ffffffffffffffda RBX: 000000000000000c RCX: 00007f6c60d14697 RDX: 000000000000000c RSI: 0000559c3bc8b980 RDI: 0000000000000001 RBP: 0000559c3bc8b980 R08: 00007f6c60dd1460 R09: 000000007fffffff R10: 0000000000000000 R11: 0000000000000246 R12: 000000000000000c R13: 00007f6c60e1a780 R14: 00007f6c60e16600 R15: 00007f6c60e15a00 The problem is that page->mapping is overloaded with slab->slab_list or slabs fields now, so slab pages could be taken as non-LRU movable pages if field slabs contains PAGE_MAPPING_MOVABLE or slab_list->prev is set to LIST_POISON2. These slab pages will be treated as thp later leading to crash in split_huge_page_to_list(). Signed-off-by: Miaohe Lin Fixes: 130d4df57390 ("mm/sl[au]b: rearrange struct slab fields to allow larger rcu_head") Reviewed-by: Matthew Wilcox (Oracle) --- v2: Check PageSlab() first to leave the rest code alone per Matthew. --- mm/memory-failure.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/mm/memory-failure.c b/mm/memory-failure.c index 636280d04008..9349948f1abf 100644 --- a/mm/memory-failure.c +++ b/mm/memory-failure.c @@ -1377,6 +1377,9 @@ void ClearPageHWPoisonTakenOff(struct page *page) */ static inline bool HWPoisonHandlable(struct page *page, unsigned long flags) { + if (PageSlab(page)) + return false; + /* Soft offline could migrate non-LRU movable pages */ if ((flags & MF_SOFT_OFFLINE) && __PageMovable(page)) return true;