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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 82B05C001DF for ; Wed, 2 Aug 2023 03:46:05 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id E7209280122; Tue, 1 Aug 2023 23:46:03 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id E2085280112; Tue, 1 Aug 2023 23:46:03 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id C737E280122; Tue, 1 Aug 2023 23:46:03 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id B9DF6280112 for ; Tue, 1 Aug 2023 23:46:03 -0400 (EDT) Received: from smtpin24.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id 843CF1A06DC for ; Wed, 2 Aug 2023 03:46:03 +0000 (UTC) X-FDA: 81077776206.24.2281FB5 Received: from dggsgout11.his.huawei.com (unknown [45.249.212.51]) by imf17.hostedemail.com (Postfix) with ESMTP id 05AD34000F for ; Wed, 2 Aug 2023 03:46:00 +0000 (UTC) Authentication-Results: imf17.hostedemail.com; dkim=none; dmarc=none; spf=none (imf17.hostedemail.com: domain of thunder.leizhen@huaweicloud.com has no SPF policy when checking 45.249.212.51) smtp.mailfrom=thunder.leizhen@huaweicloud.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1690947961; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=EfgGCO1mKe1gEwuSBXTk2lHs1P0pykmK+rRtL4vrqQ0=; b=sm60R3NxaYltVv3VL/l35j+T6pc7xfQCRGmxLjb+//slGJPR5ukjSVZUeRoyAww5gIKMgi /5c1g6qKLEz9J9ugz29ECW7grkH+L5C0K0RELYbvoIsr9DYJEmKxuZCmWN3gQ/3kpNbqz1 pz8GbpqMcnNBf/ChqWEJMlPlya1k+Yk= ARC-Authentication-Results: i=1; imf17.hostedemail.com; dkim=none; dmarc=none; spf=none (imf17.hostedemail.com: domain of thunder.leizhen@huaweicloud.com has no SPF policy when checking 45.249.212.51) smtp.mailfrom=thunder.leizhen@huaweicloud.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1690947961; a=rsa-sha256; cv=none; b=VSucmbjjIzIQjZmD3z2mJJoAjuDUAd7dZadRKlxWL5nA0oiDrxyVswLBKT9jTHMjIv15mq gAAX4Y3bGmEMmFvkOJoTQTYe1LC5FAS8lqOjmXg/6uJE3cr2OzNcZdhdtkFanlXrSzUETS j0UmH/XAod5h5eaYavILhCtC63OIYIE= Received: from mail02.huawei.com (unknown [172.30.67.143]) by dggsgout11.his.huawei.com (SkyGuard) with ESMTP id 4RFyYs3s3vz4f3pHV for ; Wed, 2 Aug 2023 11:45:53 +0800 (CST) Received: from huaweicloud.com (unknown [10.174.178.55]) by APP4 (Coremail) with SMTP id gCh0CgD3X7Nq0clky7F2PQ--.46299S6; Wed, 02 Aug 2023 11:45:54 +0800 (CST) From: thunder.leizhen@huaweicloud.com To: Christoph Lameter , Pekka Enberg , David Rientjes , Joonsoo Kim , Andrew Morton , Vlastimil Babka , Roman Gushchin , Hyeonggon Yoo <42.hyeyoo@gmail.com>, linux-mm@kvack.org, "Paul E . McKenney" , Frederic Weisbecker , Neeraj Upadhyay , Joel Fernandes , Josh Triplett , Boqun Feng , Steven Rostedt , Mathieu Desnoyers , Lai Jiangshan , Zqiang , rcu@vger.kernel.org, linux-kernel@vger.kernel.org, openeuler Subject: [PATCH v3 2/2] rcu: Dump memory object info if callback function is invalid Date: Wed, 2 Aug 2023 11:45:17 +0800 Message-Id: <20230802034518.1115-3-thunder.leizhen@huaweicloud.com> X-Mailer: git-send-email 2.37.3.windows.1 In-Reply-To: <20230802034518.1115-1-thunder.leizhen@huaweicloud.com> References: <20230802034518.1115-1-thunder.leizhen@huaweicloud.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-CM-TRANSID:gCh0CgD3X7Nq0clky7F2PQ--.46299S6 X-Coremail-Antispam: 1UD129KBjvJXoWxtF47WrW7Kw1fury3tFWrZrb_yoW7Xr48pr ykuFyUKw4kXFyrtay7Zw18WrWUA39Ygayaka95Crn3Cw4Ykw10gFyqyF12qrWYqFyrK34a qF1YqF43Kw48AwUanT9S1TB71UUUUUUqnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDU0xBIdaVrnRJUUUBEb4IE77IF4wAFF20E14v26rWj6s0DM7CY07I20VC2zVCF04k2 6cxKx2IYs7xG6r1S6rWUM7CIcVAFz4kK6r1j6r18M28IrcIa0xkI8VA2jI8067AKxVWUXw A2048vs2IY020Ec7CjxVAFwI0_Xr0E3s1l8cAvFVAK0II2c7xJM28CjxkF64kEwVA0rcxS w2x7M28EF7xvwVC0I7IYx2IY67AKxVWDJVCq3wA2z4x0Y4vE2Ix0cI8IcVCY1x0267AKxV W8Jr0_Cr1UM28EF7xvwVC2z280aVAFwI0_GcCE3s1l84ACjcxK6I8E87Iv6xkF7I0E14v2 6rxl6s0DM2AIxVAIcxkEcVAq07x20xvEncxIr21l5I8CrVACY4xI64kE6c02F40Ex7xfMc Ij6xIIjxv20xvE14v26r1j6r18McIj6I8E87Iv67AKxVWUJVW8JwAm72CE4IkC6x0Yz7v_ Jr0_Gr1lF7xvr2IYc2Ij64vIr41lFIxGxcIEc7CjxVA2Y2ka0xkIwI1lw4CEc2x0rVAKj4 xxMxAIw28IcxkI7VAKI48JMxC20s026xCaFVCjc4AY6r1j6r4UMI8I3I0E5I8CrVAFwI0_ Jr0_Jr4lx2IqxVCjr7xvwVAFwI0_JrI_JrWlx4CE17CEb7AF67AKxVW8ZVWrXwCIc40Y0x 0EwIxGrwCI42IY6xIIjxv20xvE14v26r1j6r1xMIIF0xvE2Ix0cI8IcVCY1x0267AKxVWx JVW8Jr1lIxAIcVCF04k26cxKx2IYs7xG6r1j6r1xMIIF0xvEx4A2jsIE14v26r1j6r4UMI IF0xvEx4A2jsIEc7CjxVAFwI0_Gr0_Gr1UYxBIdaVFxhVjvjDU0xZFpf9x07UtR67UUUUU = X-CM-SenderInfo: hwkx0vthuozvpl2kv046kxt4xhlfz01xgou0bp/ X-CFilter-Loop: Reflected X-Rspamd-Queue-Id: 05AD34000F X-Rspam-User: X-Rspamd-Server: rspam02 X-Stat-Signature: bf3we189r7rp1bfpcf3gxbbkooquaaa1 X-HE-Tag: 1690947960-810118 X-HE-Meta: U2FsdGVkX1+0QzzlC/6ZK3GOBJjqRGRkVNjZg9WtDSliHTlHm7ulRLjPpC93NVQopd4NOP31hNwh48N6UdNQg8TE75XUc960vD3BKSUnSNXGWae8x2/EJCFxtZju2S6IZVbyFx1iAZFMM51ti56n1ndwreQ8MelYIkGPIQb6oDHWtMzKmpo06VLH5et+UpajSzoA2UHbTCpqsgm3Xtkrf8fnKqxqMeZ9XHX1HExliSu56Hn+PHqu+ZTfg/7X2VciMqz1WDl0hSXq8uiAG4z8olGFsNhhaVvxzzr+oxT2lfCuEakSLD+TwE7aR5/iTLLYohSlA5SMW1/pttvtrn3rmr51y+EUfYnCEs4JJVhfDK86Nf870A/d14B7iceHs7BEi8xTpcGaKZiHQGS7gGguHq+TcH6JskfaOKokgG/V5tn9fMGosgWziSJ1TWUDT0TC8jh9FTTczW6CS5C5DgjKflp/5SRECT4syjq75+XvsilZgKFS6TyN/7GmsThdZVbJkFW+jltXZEH7IDqtCULiQKCbqE+ZhCY7x49NvKUDiPQ4gXPns7vQ8GQIEaiw3QrDI2Fkuc16IN5ha2IMCooGIxWD2YR61stzDiBclTdsR4kcakPkGGZ6dbImuDsXvSsqpvHsJzOZg9lcLlYex/4lVgdyb32mim+UB3Fto5V83PpNRNhtRGqXOQLjKBFIIixsWN29wRunruPfx3WqX7Z1Bs8UdnmLfxf8bHh2NLg84fS2nQYhs/ZvFNSF+QEi/PW0CBImAFRkrnGWLxufU53vNqKFY9AvSxReoTJ6boYUmr5GeYGV17Vhngh+HHTGmf/m4fJ4bFRWJU4EWh+8ioXD4NXTM8rQ29eFQ8l29/riQa55AoCmZDTuadX94oTxGluMwDBQExb/oXNI0YkkV8UiO9fFqXyN3OsQQpzeHtlC0KO121TbwcBZbpCST/RvI1JCN1n0LyZKzuqdEQobmgo W9t4tPRt tHTy0drwDtvDlgTRILC1dPkOO8wU+LHM4fAgBFuQf2aH+ObubnWNn8VdixDp80m59wdCnlkYB+K0PWJxwh/m8+SKhHb3/YX+/ZUR4BS2fbEhrEzQDSDEFXEYIIraNI25wq2m84V6UKIKkPAx07iItJBPvjXPcmVBPp5Jph9Ku5tTtapWTz4jRDdn5Z6cWjAu0SwBT6BBVVuufOekhvG7KlTGCzxYpEe3yQre+t486q8N7tMDKZEaFjfCu7fvmbkjqT67ljPYImYudJgGttNOl3m8unTS9L2muCWNAdvEeVYShFcyLJZHL4URA14OIbqxu1jPqvt00UfBxCXk= 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: Zhen Lei When a structure containing an RCU callback rhp is (incorrectly) freed and reallocated after rhp is passed to call_rcu(), it is not unusual for rhp->func to be set to NULL. This defeats the debugging prints used by __call_rcu_common() in kernels built with CONFIG_DEBUG_OBJECTS_RCU_HEAD=y, which expect to identify the offending code using the identity of this function. And in kernels build without CONFIG_DEBUG_OBJECTS_RCU_HEAD=y, things are even worse, as can be seen from this splat: Unable to handle kernel NULL pointer dereference at virtual address 0 ... ... PC is at 0x0 LR is at rcu_do_batch+0x1c0/0x3b8 ... ... (rcu_do_batch) from (rcu_core+0x1d4/0x284) (rcu_core) from (__do_softirq+0x24c/0x344) (__do_softirq) from (__irq_exit_rcu+0x64/0x108) (__irq_exit_rcu) from (irq_exit+0x8/0x10) (irq_exit) from (__handle_domain_irq+0x74/0x9c) (__handle_domain_irq) from (gic_handle_irq+0x8c/0x98) (gic_handle_irq) from (__irq_svc+0x5c/0x94) (__irq_svc) from (arch_cpu_idle+0x20/0x3c) (arch_cpu_idle) from (default_idle_call+0x4c/0x78) (default_idle_call) from (do_idle+0xf8/0x150) (do_idle) from (cpu_startup_entry+0x18/0x20) (cpu_startup_entry) from (0xc01530) This commit therefore adds calls to mem_dump_obj(rhp) to output some information, for example: slab kmalloc-256 start ffff410c45019900 pointer offset 0 size 256 This provides the rough size of the memory block and the offset of the rcu_head structure, which as least provides at least a few clues to help locate the problem. If the problem is reproducible, additional slab debugging can be enabled, for example, CONFIG_DEBUG_SLAB=y, which can provide significantly more information. Signed-off-by: Zhen Lei Signed-off-by: Paul E. McKenney --- kernel/rcu/rcu.h | 9 +++++++++ kernel/rcu/srcutiny.c | 1 + kernel/rcu/srcutree.c | 1 + kernel/rcu/tasks.h | 1 + kernel/rcu/tiny.c | 1 + kernel/rcu/tree.c | 1 + 6 files changed, 14 insertions(+) diff --git a/kernel/rcu/rcu.h b/kernel/rcu/rcu.h index d1dcb09750efbd6..30a7b0a79ae7a67 100644 --- a/kernel/rcu/rcu.h +++ b/kernel/rcu/rcu.h @@ -10,6 +10,7 @@ #ifndef __LINUX_RCU_H #define __LINUX_RCU_H +#include #include /* @@ -248,6 +249,14 @@ static inline void debug_rcu_head_unqueue(struct rcu_head *head) } #endif /* #else !CONFIG_DEBUG_OBJECTS_RCU_HEAD */ +static inline void debug_rcu_head_callback(struct rcu_head *rhp) +{ + if (unlikely(!rhp->func)) { + if (kmem_valid_obj(rhp)) + kmem_dump_obj(rhp); + } +} + extern int rcu_cpu_stall_suppress_at_boot; static inline bool rcu_stall_is_suppressed_at_boot(void) diff --git a/kernel/rcu/srcutiny.c b/kernel/rcu/srcutiny.c index 336af24e0fe358a..c38e5933a5d6937 100644 --- a/kernel/rcu/srcutiny.c +++ b/kernel/rcu/srcutiny.c @@ -138,6 +138,7 @@ void srcu_drive_gp(struct work_struct *wp) while (lh) { rhp = lh; lh = lh->next; + debug_rcu_head_callback(rhp); local_bh_disable(); rhp->func(rhp); local_bh_enable(); diff --git a/kernel/rcu/srcutree.c b/kernel/rcu/srcutree.c index f1a905200fc2f79..833a8f848a90ae6 100644 --- a/kernel/rcu/srcutree.c +++ b/kernel/rcu/srcutree.c @@ -1710,6 +1710,7 @@ static void srcu_invoke_callbacks(struct work_struct *work) rhp = rcu_cblist_dequeue(&ready_cbs); for (; rhp != NULL; rhp = rcu_cblist_dequeue(&ready_cbs)) { debug_rcu_head_unqueue(rhp); + debug_rcu_head_callback(rhp); local_bh_disable(); rhp->func(rhp); local_bh_enable(); diff --git a/kernel/rcu/tasks.h b/kernel/rcu/tasks.h index 7294be62727b12c..148ac6a464bfb12 100644 --- a/kernel/rcu/tasks.h +++ b/kernel/rcu/tasks.h @@ -538,6 +538,7 @@ static void rcu_tasks_invoke_cbs(struct rcu_tasks *rtp, struct rcu_tasks_percpu raw_spin_unlock_irqrestore_rcu_node(rtpcp, flags); len = rcl.len; for (rhp = rcu_cblist_dequeue(&rcl); rhp; rhp = rcu_cblist_dequeue(&rcl)) { + debug_rcu_head_callback(rhp); local_bh_disable(); rhp->func(rhp); local_bh_enable(); diff --git a/kernel/rcu/tiny.c b/kernel/rcu/tiny.c index 42f7589e51e09e7..fec804b7908032d 100644 --- a/kernel/rcu/tiny.c +++ b/kernel/rcu/tiny.c @@ -97,6 +97,7 @@ static inline bool rcu_reclaim_tiny(struct rcu_head *head) trace_rcu_invoke_callback("", head); f = head->func; + debug_rcu_head_callback(head); WRITE_ONCE(head->func, (rcu_callback_t)0L); f(head); rcu_lock_release(&rcu_callback_map); diff --git a/kernel/rcu/tree.c b/kernel/rcu/tree.c index 7c79480bfaa04e4..927c5ba0ae42269 100644 --- a/kernel/rcu/tree.c +++ b/kernel/rcu/tree.c @@ -2135,6 +2135,7 @@ static void rcu_do_batch(struct rcu_data *rdp) trace_rcu_invoke_callback(rcu_state.name, rhp); f = rhp->func; + debug_rcu_head_callback(rhp); WRITE_ONCE(rhp->func, (rcu_callback_t)0L); f(rhp); -- 2.34.1