Thread (26 messages) 26 messages, 2 authors, 2025-06-11

Re: [PATCH v6 6/9] kselftest/arm64/mte: add address tag related macro and function

From: Mark Brown <broonie@kernel.org>
Date: 2025-06-11 11:58:09
Also in: linux-doc, lkml

On Wed, Jun 11, 2025 at 10:41:04AM +0100, Yeoreum Yun wrote:
Add address tag related macro and function to test MTE_FAR feature.
+void *mte_insert_atag(void *ptr)
+{
+	unsigned char atag;
+
+	srandom(time(NULL));
+	atag =  mtefar_support ? (random() % MT_ATAG_MASK) + 1 : 0;
+	return (void *)MT_SET_ATAG((unsigned long)ptr, atag);
+}
Ah, this is where the time.h inclusion came from - it just got split
into the wrong patch.  I would move that srandom() into the main()
function, we don't need to reset the RNG every time we generate a tag
and since time() has a resolution of a second depending on how fast the
machine is we might manage to end up setting exactly the same value for
every tag insertion the program does which would seem to defeat the
point of using a RNG here.

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help