diff mbox series

[v2,18/34] kasan, vmalloc: drop outdated VM_KASAN comment

Message ID	4d44c09c5999cf4767803724eb47581294f4341c.1638825394.git.andreyknvl@google.com (mailing list archive)
State	New, archived
Headers	show Return-Path: <linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org> From: andrey.konovalov@linux.dev To: Marco Elver <elver@google.com>, Alexander Potapenko <glider@google.com>, Vincenzo Frascino <vincenzo.frascino@arm.com>, Catalin Marinas <catalin.marinas@arm.com>, Peter Collingbourne <pcc@google.com> Cc: Andrey Konovalov <andreyknvl@gmail.com>, Dmitry Vyukov <dvyukov@google.com>, Andrey Ryabinin <ryabinin.a.a@gmail.com>, kasan-dev@googlegroups.com, Andrew Morton <akpm@linux-foundation.org>, linux-mm@kvack.org, Will Deacon <will@kernel.org>, Mark Rutland <mark.rutland@arm.com>, linux-arm-kernel@lists.infradead.org, Evgenii Stepanov <eugenis@google.com>, linux-kernel@vger.kernel.org, Andrey Konovalov <andreyknvl@google.com> Subject: [PATCH v2 18/34] kasan, vmalloc: drop outdated VM_KASAN comment Date: Mon, 6 Dec 2021 22:43:55 +0100 Message-Id: <4d44c09c5999cf4767803724eb47581294f4341c.1638825394.git.andreyknvl@google.com> In-Reply-To: <cover.1638825394.git.andreyknvl@google.com> References: <cover.1638825394.git.andreyknvl@google.com> MIME-Version: 1.0 Precedence: list Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" <linux-arm-kernel-bounces@lists.infradead.org> Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org
Series	kasan, vmalloc, arm64: add vmalloc tagging support for SW/HW_TAGS \| expand [v2,00/34] kasan, vmalloc, arm64: add vmalloc tagging support for SW/HW_TAGS [v2,01/34] kasan, page_alloc: deduplicate should_skip_kasan_poison [v2,02/34] kasan, page_alloc: move tag_clear_highpage out of kernel_init_free_pages [v2,03/34] kasan, page_alloc: merge kasan_free_pages into free_pages_prepare [v2,04/34] kasan, page_alloc: simplify kasan_poison_pages call site [v2,05/34] kasan, page_alloc: init memory of skipped pages on free [v2,06/34] kasan: drop skip_kasan_poison variable in free_pages_prepare [v2,07/34] mm: clarify __GFP_ZEROTAGS comment [v2,08/34] kasan: only apply __GFP_ZEROTAGS when memory is zeroed [v2,09/34] kasan, page_alloc: refactor init checks in post_alloc_hook [v2,10/34] kasan, page_alloc: merge kasan_alloc_pages into post_alloc_hook [v2,11/34] kasan, page_alloc: combine tag_clear_highpage calls in post_alloc_hook [v2,12/34] kasan, page_alloc: move SetPageSkipKASanPoison in post_alloc_hook [v2,13/34] kasan, page_alloc: move kernel_init_free_pages in post_alloc_hook [v2,14/34] kasan, page_alloc: simplify kasan_unpoison_pages call site [v2,15/34] kasan: clean up metadata byte definitions [v2,16/34] kasan: define KASAN_VMALLOC_INVALID for SW_TAGS [v2,17/34] kasan, x86, arm64, s390: rename functions for modules shadow [v2,18/34] kasan, vmalloc: drop outdated VM_KASAN comment [v2,19/34] kasan: reorder vmalloc hooks [v2,20/34] kasan: add wrappers for vmalloc hooks [v2,21/34] kasan, vmalloc: reset tags in vmalloc functions [v2,22/34] kasan, fork: don't tag stacks allocated with vmalloc [v2,23/34] kasan, vmalloc: add vmalloc support to SW_TAGS [v2,24/34] kasan, vmalloc, arm64: mark vmalloc mappings as pgprot_tagged [v2,25/34] kasan, vmalloc: don't unpoison VM_ALLOC pages before mapping [v2,26/34] kasan, page_alloc: allow skipping unpoisoning for HW_TAGS [v2,27/34] kasan, page_alloc: allow skipping memory init for HW_TAGS [v2,28/34] kasan, vmalloc: add vmalloc support to HW_TAGS [v2,29/34] kasan: mark kasan_arg_stacktrace as __initdata [v2,30/34] kasan: simplify kasan_init_hw_tags [v2,31/34] kasan: add kasan.vmalloc command line flag [v2,32/34] arm64: select KASAN_VMALLOC for SW/HW_TAGS modes [v2,33/34] kasan: documentation updates [v2,34/34] kasan: improve vmalloc tests

Commit Message

andrey.konovalov@linux.dev Dec. 6, 2021, 9:43 p.m. UTC

From: Andrey Konovalov <andreyknvl@google.com>

The comment about VM_KASAN in include/linux/vmalloc.c is outdated.
VM_KASAN is currently only used to mark vm_areas allocated for
kernel modules when CONFIG_KASAN_VMALLOC is disabled.

Drop the comment.

Signed-off-by: Andrey Konovalov <andreyknvl@google.com>
---
 include/linux/vmalloc.h | 11 -----------
 1 file changed, 11 deletions(-)

diff mbox series

Patch

diff --git a/include/linux/vmalloc.h b/include/linux/vmalloc.h
index 6e022cc712e6..b22369f540eb 100644
--- a/include/linux/vmalloc.h
+++ b/include/linux/vmalloc.h
@@ -28,17 +28,6 @@  struct notifier_block;		/* in notifier.h */
 #define VM_MAP_PUT_PAGES	0x00000200	/* put pages and free array in vfree */
 #define VM_NO_HUGE_VMAP		0x00000400	/* force PAGE_SIZE pte mapping */
 
-/*
- * VM_KASAN is used slightly differently depending on CONFIG_KASAN_VMALLOC.
- *
- * If IS_ENABLED(CONFIG_KASAN_VMALLOC), VM_KASAN is set on a vm_struct after
- * shadow memory has been mapped. It's used to handle allocation errors so that
- * we don't try to poison shadow on free if it was never allocated.
- *
- * Otherwise, VM_KASAN is set for kasan_module_alloc() allocations and used to
- * determine which allocations need the module shadow freed.
- */
-
 /* bits [20..32] reserved for arch specific ioremap internals */
 
 /*