[v7,26/38] s390/cpacf: Unpoison the results of cpacf_trng()

Message ID	20240621113706.315500-27-iii@linux.ibm.com (mailing list archive)
State	New
Headers	show Return-Path: <owner-linux-mm@kvack.org> From: Ilya Leoshkevich <iii@linux.ibm.com> To: Alexander Gordeev <agordeev@linux.ibm.com>, Alexander Potapenko <glider@google.com>, Andrew Morton <akpm@linux-foundation.org>, Christoph Lameter <cl@linux.com>, David Rientjes <rientjes@google.com>, Heiko Carstens <hca@linux.ibm.com>, Joonsoo Kim <iamjoonsoo.kim@lge.com>, Marco Elver <elver@google.com>, Masami Hiramatsu <mhiramat@kernel.org>, Pekka Enberg <penberg@kernel.org>, Steven Rostedt <rostedt@goodmis.org>, Vasily Gorbik <gor@linux.ibm.com>, Vlastimil Babka <vbabka@suse.cz> Cc: Christian Borntraeger <borntraeger@linux.ibm.com>, Dmitry Vyukov <dvyukov@google.com>, Hyeonggon Yoo <42.hyeyoo@gmail.com>, kasan-dev@googlegroups.com, linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-s390@vger.kernel.org, linux-trace-kernel@vger.kernel.org, Mark Rutland <mark.rutland@arm.com>, Roman Gushchin <roman.gushchin@linux.dev>, Sven Schnelle <svens@linux.ibm.com>, Ilya Leoshkevich <iii@linux.ibm.com> Subject: [PATCH v7 26/38] s390/cpacf: Unpoison the results of cpacf_trng() Date: Fri, 21 Jun 2024 13:35:10 +0200 Message-ID: <20240621113706.315500-27-iii@linux.ibm.com> In-Reply-To: <20240621113706.315500-1-iii@linux.ibm.com> References: <20240621113706.315500-1-iii@linux.ibm.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: owner-linux-mm@kvack.org Precedence: bulk
Series	kmsan: Enable on s390 \| expand [v7,00/38] kmsan: Enable on s390 [v7,02/38] kmsan: Make the tests compatible with kmsan.panic=1 [v7,04/38] kmsan: Increase the maximum store size to 4096 [v7,05/38] kmsan: Fix is_bad_asm_addr() on arches with overlapping address spaces [v7,07/38] kmsan: Remove a useless assignment from kmsan_vmap_pages_range_noflush() [v7,08/38] kmsan: Remove an x86-specific #include from kmsan.h [v7,09/38] kmsan: Expose kmsan_get_metadata() [v7,11/38] kmsan: Allow disabling KMSAN checks for the current task [v7,14/38] kmsan: Use ALIGN_DOWN() in kmsan_get_metadata() [v7,16/38] kmsan: Expose KMSAN_WARN_ON() [v7,17/38] mm: slub: Let KMSAN access metadata [v7,18/38] mm: slub: Disable KMSAN when checking the padding bytes [v7,19/38] mm: kfence: Disable KMSAN when checking the canary [v7,20/38] lib/zlib: Unpoison DFLTCC output buffers [v7,21/38] kmsan: Accept ranges starting with 0 on s390 [v7,22/38] s390/boot: Turn off KMSAN [v7,23/38] s390: Use a larger stack for KMSAN [v7,24/38] s390/boot: Add the KMSAN runtime stub [v7,25/38] s390/checksum: Add a KMSAN check [v7,26/38] s390/cpacf: Unpoison the results of cpacf_trng() [v7,27/38] s390/cpumf: Unpoison STCCTM output buffer [v7,28/38] s390/diag: Unpoison diag224() output buffer [v7,31/38] s390/mm: Define KMSAN metadata for vmalloc and modules [v7,32/38] s390/string: Add KMSAN support [v7,33/38] s390/traps: Unpoison the kernel_stack_overflow()'s pt_regs [v7,35/38] s390/uaccess: Add the missing linux/instrumented.h #include [v7,36/38] s390/unwind: Disable KMSAN checks [v7,37/38] s390/kmsan: Implement the architecture-specific functions [v7,38/38] kmsan: Enable on s390

Message ID

20240621113706.315500-27-iii@linux.ibm.com (mailing list archive)

State

New

Headers

From: Ilya Leoshkevich <iii@linux.ibm.com>
To: Alexander Gordeev <agordeev@linux.ibm.com>,
        Alexander Potapenko <glider@google.com>,
        Andrew Morton <akpm@linux-foundation.org>,
        Christoph Lameter <cl@linux.com>,
 David Rientjes <rientjes@google.com>,
        Heiko Carstens <hca@linux.ibm.com>,
        Joonsoo Kim <iamjoonsoo.kim@lge.com>, Marco Elver <elver@google.com>,
        Masami Hiramatsu <mhiramat@kernel.org>,
        Pekka Enberg <penberg@kernel.org>,
        Steven Rostedt <rostedt@goodmis.org>,
        Vasily Gorbik <gor@linux.ibm.com>, Vlastimil Babka <vbabka@suse.cz>
Cc: Christian Borntraeger <borntraeger@linux.ibm.com>,
        Dmitry Vyukov <dvyukov@google.com>,
        Hyeonggon Yoo <42.hyeyoo@gmail.com>, kasan-dev@googlegroups.com,
        linux-kernel@vger.kernel.org, linux-mm@kvack.org,
        linux-s390@vger.kernel.org, linux-trace-kernel@vger.kernel.org,
        Mark Rutland <mark.rutland@arm.com>,
        Roman Gushchin <roman.gushchin@linux.dev>,
        Sven Schnelle <svens@linux.ibm.com>,
        Ilya Leoshkevich <iii@linux.ibm.com>
Subject: [PATCH v7 26/38] s390/cpacf: Unpoison the results of cpacf_trng()
Date: Fri, 21 Jun 2024 13:35:10 +0200
Message-ID: <20240621113706.315500-27-iii@linux.ibm.com>
In-Reply-To: <20240621113706.315500-1-iii@linux.ibm.com>
References: <20240621113706.315500-1-iii@linux.ibm.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Sender: owner-linux-mm@kvack.org
Precedence: bulk

Series

kmsan: Enable on s390 | expand

Commit Message

Ilya Leoshkevich June 21, 2024, 11:35 a.m. UTC

Prevent KMSAN from complaining about buffers filled by cpacf_trng()
being uninitialized.

Tested-by: Alexander Gordeev <agordeev@linux.ibm.com>
Reviewed-by: Alexander Potapenko <glider@google.com>
Acked-by: Heiko Carstens <hca@linux.ibm.com>
Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>
---
 arch/s390/include/asm/cpacf.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/s390/include/asm/cpacf.h b/arch/s390/include/asm/cpacf.h
index c786538e397c..dae8843b164f 100644
--- a/arch/s390/include/asm/cpacf.h
+++ b/arch/s390/include/asm/cpacf.h
@@ -12,6 +12,7 @@ 
 #define _ASM_S390_CPACF_H
 
 #include <asm/facility.h>
+#include <linux/kmsan-checks.h>
 
 /*
  * Instruction opcodes for the CPACF instructions
@@ -542,6 +543,8 @@  static inline void cpacf_trng(u8 *ucbuf, unsigned long ucbuf_len,
 		: [ucbuf] "+&d" (u.pair), [cbuf] "+&d" (c.pair)
 		: [fc] "K" (CPACF_PRNO_TRNG), [opc] "i" (CPACF_PRNO)
 		: "cc", "memory", "0");
+	kmsan_unpoison_memory(ucbuf, ucbuf_len);
+	kmsan_unpoison_memory(cbuf, cbuf_len);
 }
 
 /**

[v7,26/38] s390/cpacf: Unpoison the results of cpacf_trng()

Commit Message

Patch