From nobody Sun Jun 28 02:51:47 2026 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 911A9C433F5 for ; Tue, 15 Feb 2022 18:39:45 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S243144AbiBOSjy (ORCPT ); Tue, 15 Feb 2022 13:39:54 -0500 Received: from mxb-00190b01.gslb.pphosted.com ([23.128.96.19]:38912 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232323AbiBOSjx (ORCPT ); Tue, 15 Feb 2022 13:39:53 -0500 Received: from out1.migadu.com (out1.migadu.com [IPv6:2001:41d0:2:863f::]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 56CEFDAAEF for ; Tue, 15 Feb 2022 10:39:42 -0800 (PST) X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1644950380; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=pwV+5NwZYvQrQ1JgfEXwuJfZqiKqAoxKp8Wr6tUObj4=; b=tWXktW8oDUS1MhFseJtb3CLIKZuWLrPDlv2sQlq+eyNkoNG1e/ZPG6PDlwMn2tYaAx5EW6 9UtA67ZGSmseXId+wLFPydBzmgeXSAb/AdIjZo9BxnBvVkt5eJSjVaeN/RrpBRQIffZ97l +19DsKFp5Hnxmwaped+jsGsDYi/30LE= From: andrey.konovalov@linux.dev To: Andrew Morton Cc: Andrey Konovalov , Marco Elver , Alexander Potapenko , Dmitry Vyukov , Andrey Ryabinin , kasan-dev@googlegroups.com, linux-mm@kvack.org, linux-kernel@vger.kernel.org, Andrey Konovalov Subject: [PATCH mm] fix for "kasan: improve vmalloc tests" Date: Tue, 15 Feb 2022 19:39:38 +0100 Message-Id: <865c91ba49b90623ab50c7526b79ccb955f544f0.1644950160.git.andreyknvl@google.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Migadu-Flow: FLOW_OUT X-Migadu-Auth-User: linux.dev Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" From: Andrey Konovalov vmap_tags() and vm_map_ram_tags() pass invalid page array size to vm_map_ram() and vm_unmap_ram(). It's supposed to be 1, but it's 1 << order =3D=3D 2 currently. Remove order variable (it can only be 0 with the current code) and hardcode the number of pages in these tests. Signed-off-by: Andrey Konovalov Acked-by: Marco Elver --- lib/test_kasan.c | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/lib/test_kasan.c b/lib/test_kasan.c index 491a82006f06..8416161d5177 100644 --- a/lib/test_kasan.c +++ b/lib/test_kasan.c @@ -1149,7 +1149,6 @@ static void vmap_tags(struct kunit *test) { char *p_ptr, *v_ptr; struct page *p_page, *v_page; - size_t order =3D 1; =20 /* * This test is specifically crafted for the software tag-based mode, @@ -1159,12 +1158,12 @@ static void vmap_tags(struct kunit *test) =20 KASAN_TEST_NEEDS_CONFIG_ON(test, CONFIG_KASAN_VMALLOC); =20 - p_page =3D alloc_pages(GFP_KERNEL, order); + p_page =3D alloc_pages(GFP_KERNEL, 1); KUNIT_ASSERT_NOT_ERR_OR_NULL(test, p_page); p_ptr =3D page_address(p_page); KUNIT_ASSERT_NOT_ERR_OR_NULL(test, p_ptr); =20 - v_ptr =3D vmap(&p_page, 1 << order, VM_MAP, PAGE_KERNEL); + v_ptr =3D vmap(&p_page, 1, VM_MAP, PAGE_KERNEL); KUNIT_ASSERT_NOT_ERR_OR_NULL(test, v_ptr); =20 /* @@ -1186,14 +1185,13 @@ static void vmap_tags(struct kunit *test) KUNIT_EXPECT_PTR_EQ(test, p_page, v_page); =20 vunmap(v_ptr); - free_pages((unsigned long)p_ptr, order); + free_pages((unsigned long)p_ptr, 1); } =20 static void vm_map_ram_tags(struct kunit *test) { char *p_ptr, *v_ptr; struct page *page; - size_t order =3D 1; =20 /* * This test is specifically crafted for the software tag-based mode, @@ -1201,12 +1199,12 @@ static void vm_map_ram_tags(struct kunit *test) */ KASAN_TEST_NEEDS_CONFIG_ON(test, CONFIG_KASAN_SW_TAGS); =20 - page =3D alloc_pages(GFP_KERNEL, order); + page =3D alloc_pages(GFP_KERNEL, 1); KUNIT_ASSERT_NOT_ERR_OR_NULL(test, page); p_ptr =3D page_address(page); KUNIT_ASSERT_NOT_ERR_OR_NULL(test, p_ptr); =20 - v_ptr =3D vm_map_ram(&page, 1 << order, -1); + v_ptr =3D vm_map_ram(&page, 1, -1); KUNIT_ASSERT_NOT_ERR_OR_NULL(test, v_ptr); =20 KUNIT_EXPECT_GE(test, (u8)get_tag(v_ptr), (u8)KASAN_TAG_MIN); @@ -1216,8 +1214,8 @@ static void vm_map_ram_tags(struct kunit *test) *p_ptr =3D 0; *v_ptr =3D 0; =20 - vm_unmap_ram(v_ptr, 1 << order); - free_pages((unsigned long)p_ptr, order); + vm_unmap_ram(v_ptr, 1); + free_pages((unsigned long)p_ptr, 1); } =20 static void vmalloc_percpu(struct kunit *test) --=20 2.25.1