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 54233C0015E for ; Thu, 3 Aug 2023 10:19:17 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id A571128022B; Thu, 3 Aug 2023 06:19:16 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id A079A2801EB; Thu, 3 Aug 2023 06:19:16 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 8DB0A28022B; Thu, 3 Aug 2023 06:19:16 -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 7F0212801EB for ; Thu, 3 Aug 2023 06:19:16 -0400 (EDT) Received: from smtpin12.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id 41A5AC0785 for ; Thu, 3 Aug 2023 10:19:16 +0000 (UTC) X-FDA: 81082395912.12.382BA4B Received: from dggsgout12.his.huawei.com (unknown [45.249.212.56]) by imf04.hostedemail.com (Postfix) with ESMTP id 9BB714000C for ; Thu, 3 Aug 2023 10:19:12 +0000 (UTC) Authentication-Results: imf04.hostedemail.com; dkim=none; spf=none (imf04.hostedemail.com: domain of thunder.leizhen@huaweicloud.com has no SPF policy when checking 45.249.212.56) smtp.mailfrom=thunder.leizhen@huaweicloud.com; dmarc=none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1691057953; 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-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=qH2Y3/xOMWgj3UzJQddM1wfI9s1KetFUbEB44vBr0Q0=; b=Bk3MLIqYZNjpXwVJ9KETLDXdzpaxIylm1kICgbp6lhnv0cPFdtPbiKeuRpog/U0q3o0aT6 c0Nb3n24ortpFO/gMV6Q770MgHVllpTnpg9fl6FhhBWUbe2mWLIrNoqelgxuk5hwZZ8tXm Aeu3nLGakIgZMkJC72+3M8bt0eP7oB0= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1691057953; a=rsa-sha256; cv=none; b=muqNSQi+fQl0wOqCo9mhbvWFd9LftZ/eUhh4qYnaECasMc3vxxgpHAswlLCWHl6Lqo+jMR XUuEGvhBeEsXcfx9MxMYkT5iHk6PC+d8PCtL4TWPPM8mh4qKAs5hLBqvawNuSKI8mOw4bJ uH7vTQwRQ5i030F+KIvuGupd5Xw85tc= ARC-Authentication-Results: i=1; imf04.hostedemail.com; dkim=none; spf=none (imf04.hostedemail.com: domain of thunder.leizhen@huaweicloud.com has no SPF policy when checking 45.249.212.56) smtp.mailfrom=thunder.leizhen@huaweicloud.com; dmarc=none Received: from mail02.huawei.com (unknown [172.30.67.143]) by dggsgout12.his.huawei.com (SkyGuard) with ESMTP id 4RGlF44sYhz4f46G6 for ; Thu, 3 Aug 2023 18:19:04 +0800 (CST) Received: from huaweicloud.com (unknown [10.174.178.55]) by APP4 (Coremail) with SMTP id gCh0CgD3mp4Uf8tk4+TaPQ--.35755S5; Thu, 03 Aug 2023 18:19:07 +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 Cc: Zhen Lei Subject: [PATCH v5 1/3] mm: Remove kmem_valid_obj() Date: Thu, 3 Aug 2023 18:17:52 +0800 Message-Id: <20230803101754.1149-2-thunder.leizhen@huaweicloud.com> X-Mailer: git-send-email 2.37.3.windows.1 In-Reply-To: <20230803101754.1149-1-thunder.leizhen@huaweicloud.com> References: <20230803101754.1149-1-thunder.leizhen@huaweicloud.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-CM-TRANSID:gCh0CgD3mp4Uf8tk4+TaPQ--.35755S5 X-Coremail-Antispam: 1UD129KBjvJXoWxZr1ftry5Jr4fXr1fKF1rZwb_yoWrCrWkpw nxGr9xJFW8Jr1xGrsrJF1kurW5Zr4kuF17Ca9aqw18Ar1UXrs7ur1kG3s2qF98GFW8XF10 ya1vkF43uryUArDanT9S1TB71UUUUUUqnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDU0xBIdaVrnRJUUUBKb4IE77IF4wAFF20E14v26rWj6s0DM7CY07I20VC2zVCF04k2 6cxKx2IYs7xG6r1S6rWUM7CIcVAFz4kK6r1j6r18M28IrcIa0xkI8VA2jI8067AKxVWUGw A2048vs2IY020Ec7CjxVAFwI0_Xr0E3s1l8cAvFVAK0II2c7xJM28CjxkF64kEwVA0rcxS w2x7M28EF7xvwVC0I7IYx2IY67AKxVWDJVCq3wA2z4x0Y4vE2Ix0cI8IcVCY1x0267AKxV W8Jr0_Cr1UM28EF7xvwVC2z280aVAFwI0_GcCE3s1l84ACjcxK6I8E87Iv6xkF7I0E14v2 6rxl6s0DM2AIxVAIcxkEcVAq07x20xvEncxIr21l5I8CrVACY4xI64kE6c02F40Ex7xfMc Ij6xIIjxv20xvE14v26r1j6r18McIj6I8E87Iv67AKxVWUJVW8JwAm72CE4IkC6x0Yz7v_ Jr0_Gr1lF7xvr2IYc2Ij64vIr41lFIxGxcIEc7CjxVA2Y2ka0xkIwI1lw4CEc2x0rVAKj4 xxMxAIw28IcxkI7VAKI48JMxC20s026xCaFVCjc4AY6r1j6r4UMI8I3I0E5I8CrVAFwI0_ Jr0_Jr4lx2IqxVCjr7xvwVAFwI0_JrI_JrWlx4CE17CEb7AF67AKxVW8ZVWrXwCIc40Y0x 0EwIxGrwCI42IY6xIIjxv20xvE14v26r1j6r1xMIIF0xvE2Ix0cI8IcVCY1x0267AKxVW8 JVWxJwCI42IY6xAIw20EY4v20xvaj40_Jr0_JF4lIxAIcVC2z280aVAFwI0_Jr0_Gr1lIx AIcVC2z280aVCY1x0267AKxVW8JVW8JrUvcSsGvfC2KfnxnUUI43ZEXa7IU1D73DUUUUU= = X-CM-SenderInfo: hwkx0vthuozvpl2kv046kxt4xhlfz01xgou0bp/ X-CFilter-Loop: Reflected X-Rspamd-Queue-Id: 9BB714000C X-Rspam-User: X-Stat-Signature: srm9cg9wb58d8r8aisny3o7nmn5jk4qj X-Rspamd-Server: rspam03 X-HE-Tag: 1691057952-699981 X-HE-Meta: U2FsdGVkX18wxTS087fl2eIMGfzrQ67X7c8VhA9kK+nmXttusxkkcKeeG5l+3F4J0cT03wh+WsBUoPWkASDCqAJ5hXkMier/AMtUKpEtOGpS4vTgpP4EAvySo3KmvKNO1bcJurS5oJfqf9RBjIjuoVkd5Ww6a+KiqOSeou3D14SO8zEtv9LUlMMuorEl7OhTrOwJsOifUBJHvX3k7aQKnQxKy9mCFXabRf4DNQa/Td5ecIsPXSxtmn6Zta4IZnwScWmpc3PqXtb+t0wNS+G3aTiEcto0I2vhAKrh+MiZw3MfCJFOfIfG2+GtBrXUiQkPSkkP6X2xeAFYcVMoT2H1Kz8oNsB74jkw2Obp9tnFTmCcE9PV0nL/23DzThazVjso2PmP4Bir16eAph5sH8vfGgqTiMIb7CnBPgnJyXVEbXhxogeiodE/8tBZqb1wgxLc4UgM3IVd/PJFAtwQBT8n98wZswCBdA3mHlLhs3ReKRIVAWmO/Ca+B2hVw9Hn5M0GRDMx6rU1l0wGz1yDziT7759Ko1uFL8+yvXuyBSzuFN71TikoV5vX1l797PCJCCRlmxoNiTfC0qX1CMZjDMkuM1lEwYvNNXbFCTZ3je09sl/yNhluiHZ8nBud5/fbeeKmCQbM6hUPakNiOC6DFayii2/qu6HX/1SzsUO19BJVr71+PEL+PzXCdmOg7g/rYx3OGolI5cwRWBqh3xHgBJkEZzDOw1Dz67HyhD3jxDMi8fir3Icj8CLdJ4XsJtYhm7db8qvtfCTEcnt3EGMVJVgu60FKCHllK4KrFW2jNonWDuxrgaEiMA8Wd9J4pqmw7XF6e606xuEJ+0aUc+wEpSG3CvfMPmVgSEVYpD5Tap8ZEJMPMbCUZTeRCgdOzTGoQcwxjvAIirboVjjSBfdwTE3QVur4YkCPH8Ouz6GzIE+xYC2tfBopKk0ho0ZNQv4TH27x3opBUDDpGtjoh/vd0gj jW0KDj8q 8Y3R8/dcCZLKCXoAKLXHFaLBIyFnSaxEwXUcetC26IWDRUICr5jDtiR/KS7u6+Hvk6dbsmrsmwhIBbw8ES46OeXq/8+wO7+gwJC9ODhQDw5J4ojcSRRcVWrsrPVqWRU7waROuLWM48qX8LmSFVz+P5IipOvATLyNs5GSgi1NqSOHAWsZ0q6Z+ibEHdOiHWIEoscCt+ZCyDFcdYQYDHJJze6yDikdByX6LiSD2CSZ4Llu4H/Rc0WChTmGQey2r3BpYVE+xLhRdIwpVyArtlRW1Tr81T1xUSS986bhgZe5cQW8MzFHrbj6VVuu2u1WP9yMCIDasQsZDd8dDXNSQqV1W0EdLaThGrIXuIFJNslLm8lLfz2m+HMYEWO77RA== 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 Function kmem_dump_obj() will splat if passed a pointer to a non-slab object. So no one will call it directly. It is always necessary to call kmem_valid_obj() first to determine whether the passed pointer to a valid slab object. Then merging kmem_valid_obj() into kmem_dump_obj() will make the code more concise. So convert kmem_dump_obj() to work the same way as vmalloc_dump_obj(). After this, no one calls kmem_valid_obj() anymore, and it can be safely removed. Suggested-by: Matthew Wilcox Signed-off-by: Zhen Lei Reviewed-by: Matthew Wilcox (Oracle) Acked-by: Vlastimil Babka --- include/linux/slab.h | 5 +++-- mm/slab_common.c | 41 +++++++++++------------------------------ mm/util.c | 4 +--- 3 files changed, 15 insertions(+), 35 deletions(-) diff --git a/include/linux/slab.h b/include/linux/slab.h index 848c7c82ad5ad0b..d8ed2e810ec4448 100644 --- a/include/linux/slab.h +++ b/include/linux/slab.h @@ -244,8 +244,9 @@ DEFINE_FREE(kfree, void *, if (_T) kfree(_T)) size_t ksize(const void *objp); #ifdef CONFIG_PRINTK -bool kmem_valid_obj(void *object); -void kmem_dump_obj(void *object); +bool kmem_dump_obj(void *object); +#else +static inline bool kmem_dump_obj(void *object) { return false; } #endif /* diff --git a/mm/slab_common.c b/mm/slab_common.c index d1555ea2981ac51..ee6ed6dd7ba9fa5 100644 --- a/mm/slab_common.c +++ b/mm/slab_common.c @@ -528,26 +528,6 @@ bool slab_is_available(void) } #ifdef CONFIG_PRINTK -/** - * kmem_valid_obj - does the pointer reference a valid slab object? - * @object: pointer to query. - * - * Return: %true if the pointer is to a not-yet-freed object from - * kmalloc() or kmem_cache_alloc(), either %true or %false if the pointer - * is to an already-freed object, and %false otherwise. - */ -bool kmem_valid_obj(void *object) -{ - struct folio *folio; - - /* Some arches consider ZERO_SIZE_PTR to be a valid address. */ - if (object < (void *)PAGE_SIZE || !virt_addr_valid(object)) - return false; - folio = virt_to_folio(object); - return folio_test_slab(folio); -} -EXPORT_SYMBOL_GPL(kmem_valid_obj); - static void kmem_obj_info(struct kmem_obj_info *kpp, void *object, struct slab *slab) { if (__kfence_obj_info(kpp, object, slab)) @@ -566,11 +546,11 @@ static void kmem_obj_info(struct kmem_obj_info *kpp, void *object, struct slab * * and, if available, the slab name, return address, and stack trace from * the allocation and last free path of that object. * - * This function will splat if passed a pointer to a non-slab object. - * If you are not sure what type of object you have, you should instead - * use mem_dump_obj(). + * Return: %true if the pointer is to a not-yet-freed object from + * kmalloc() or kmem_cache_alloc(), either %true or %false if the pointer + * is to an already-freed object, and %false otherwise. */ -void kmem_dump_obj(void *object) +bool kmem_dump_obj(void *object) { char *cp = IS_ENABLED(CONFIG_MMU) ? "" : "/vmalloc"; int i; @@ -578,13 +558,13 @@ void kmem_dump_obj(void *object) unsigned long ptroffset; struct kmem_obj_info kp = { }; - if (WARN_ON_ONCE(!virt_addr_valid(object))) - return; + /* Some arches consider ZERO_SIZE_PTR to be a valid address. */ + if (object < (void *)PAGE_SIZE || !virt_addr_valid(object)) + return false; slab = virt_to_slab(object); - if (WARN_ON_ONCE(!slab)) { - pr_cont(" non-slab memory.\n"); - return; - } + if (!slab) + return false; + kmem_obj_info(&kp, object, slab); if (kp.kp_slab_cache) pr_cont(" slab%s %s", cp, kp.kp_slab_cache->name); @@ -621,6 +601,7 @@ void kmem_dump_obj(void *object) pr_info(" %pS\n", kp.kp_free_stack[i]); } + return true; } EXPORT_SYMBOL_GPL(kmem_dump_obj); #endif diff --git a/mm/util.c b/mm/util.c index dd12b9531ac4cad..ddfbb22dc1876d3 100644 --- a/mm/util.c +++ b/mm/util.c @@ -1063,10 +1063,8 @@ void mem_dump_obj(void *object) { const char *type; - if (kmem_valid_obj(object)) { - kmem_dump_obj(object); + if (kmem_dump_obj(object)) return; - } if (vmalloc_dump_obj(object)) return; -- 2.34.1