Thread (2 messages) 2 messages, 2 authors, 2018-07-31
DORMANTno replies REVIEWED: 1 (1M)

[PATCH] arm64: drop unused kernel_neon_begin_partial() macro

From: Dave.Martin@arm.com (Dave Martin)
Date: 2018-07-31 09:10:59

On Tue, Jul 31, 2018 at 09:35:45AM +0200, Ard Biesheuvel wrote:
When kernel mode NEON was first introduced to the arm64 kernel,
every call to kernel_neon_begin()/_end() stacked resp. unstacked
the entire NEON register file, making it worthwile to reduce the
number of used NEON registers to a bare minimum, and only stack
those. kernel_neon_begin_partial() was introduced for this purpose,
but after the refactoring for SVE and other changes, it no longer
exists and was simply #define'd to kernel_neon_begin() directly.

In the mean time, all users have been updated, so let's remove
the fallback macro.

Signed-off-by: Ard Biesheuvel <redacted>
Makes sense, and nobody seems to use it any more.

Reviewed-by: Dave Martin <Dave.Martin@arm.com>
quoted hunk ↗ jump to hunk
---
 arch/arm64/include/asm/neon.h | 7 -------
 1 file changed, 7 deletions(-)
diff --git a/arch/arm64/include/asm/neon.h b/arch/arm64/include/asm/neon.h
index f922eaf780f9..fb9d137256a6 100644
--- a/arch/arm64/include/asm/neon.h
+++ b/arch/arm64/include/asm/neon.h
@@ -19,11 +19,4 @@
 void kernel_neon_begin(void);
 void kernel_neon_end(void);
 
-/*
- * Temporary macro to allow the crypto code to compile. Note that the
- * semantics of kernel_neon_begin_partial() are now different from the
- * original as it does not allow being called in an interrupt context.
- */
-#define kernel_neon_begin_partial(num_regs)	kernel_neon_begin()
-
 #endif /* ! __ASM_NEON_H */
-- 
2.18.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel at 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