Thread (88 messages) 88 messages, 4 authors, 2020-10-01

Re: [PATCH v3 32/39] kasan: define KASAN_GRANULE_SIZE for HW_TAGS

From: Andrey Konovalov <hidden>
Date: 2020-10-01 21:59:23
Also in: linux-mm, lkml

On Thu, Oct 1, 2020 at 7:58 PM [off-list ref] wrote:
On Fri, Sep 25, 2020 at 12:50AM +0200, Andrey Konovalov wrote:
quoted
Hardware tag-based KASAN has granules of MTE_GRANULE_SIZE. Define
KASAN_GRANULE_SIZE to MTE_GRANULE_SIZE for CONFIG_KASAN_HW_TAGS.

Signed-off-by: Andrey Konovalov <redacted>
Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com>
Reviewed-by: Marco Elver <elver@google.com>
quoted
---
Change-Id: I5d1117e6a991cbca00d2cfb4ba66e8ae2d8f513a
---
 mm/kasan/kasan.h | 6 ++++++
 1 file changed, 6 insertions(+)
diff --git a/mm/kasan/kasan.h b/mm/kasan/kasan.h
index 9c73f324e3ce..bd51ab72c002 100644
--- a/mm/kasan/kasan.h
+++ b/mm/kasan/kasan.h
@@ -5,7 +5,13 @@
 #include <linux/kasan.h>
 #include <linux/stackdepot.h>

+#if defined(CONFIG_KASAN_GENERIC) || defined(CONFIG_KASAN_SW_TAGS)
 #define KASAN_GRANULE_SIZE   (1UL << KASAN_SHADOW_SCALE_SHIFT)
+#else
+#include <asm/mte-kasan.h>
+#define KASAN_GRANULE_SIZE   (MTE_GRANULE_SIZE)
Why braces? Shouldn't MTE_GRANULE_SIZE already have braces?
Will fix in v4, thanks!
quoted
+#endif
+
 #define KASAN_GRANULE_MASK   (KASAN_GRANULE_SIZE - 1)
 #define KASAN_GRANULE_PAGE   (KASAN_GRANULE_SIZE << PAGE_SHIFT)

--
2.28.0.681.g6f77f65b4e-goog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help