linux-kernel - Re: [PATCH 10/14] alpha: remove DECpc AXP150 (Jensen) support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Fri, 3 May 2024 18:00:28 +0100
From: Al Viro <viro@...iv.linux.org.uk>
To: Arnd Bergmann <arnd@...nel.org>
Cc: linux-alpha@...r.kernel.org, Arnd Bergmann <arnd@...db.de>,
	Richard Henderson <richard.henderson@...aro.org>,
	Ivan Kokshaysky <ink@...assic.park.msu.ru>,
	Matt Turner <mattst88@...il.com>, Marc Zyngier <maz@...nel.org>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	"Paul E. McKenney" <paulmck@...nel.org>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 10/14] alpha: remove DECpc AXP150 (Jensen) support

On Fri, May 03, 2024 at 10:11:21AM +0200, Arnd Bergmann wrote:
> diff --git a/arch/alpha/kernel/traps.c b/arch/alpha/kernel/traps.c
> index 7fc72aeb7398..a9a8e9ab0f52 100644
> --- a/arch/alpha/kernel/traps.c
> +++ b/arch/alpha/kernel/traps.c
> @@ -30,39 +30,6 @@
>  
>  #include "proto.h"
>  
> -/* Work-around for some SRMs which mishandle opDEC faults.  */
> -
> -static int opDEC_fix;
> -
> -static void
> -opDEC_check(void)
> -{
> -	__asm__ __volatile__ (
> -	/* Load the address of... */
> -	"	br	$16, 1f\n"
> -	/* A stub instruction fault handler.  Just add 4 to the
> -	   pc and continue.  */
> -	"	ldq	$16, 8($sp)\n"
> -	"	addq	$16, 4, $16\n"
> -	"	stq	$16, 8($sp)\n"
> -	"	call_pal %[rti]\n"
> -	/* Install the instruction fault handler.  */
> -	"1:	lda	$17, 3\n"
> -	"	call_pal %[wrent]\n"
> -	/* With that in place, the fault from the round-to-minf fp
> -	   insn will arrive either at the "lda 4" insn (bad) or one
> -	   past that (good).  This places the correct fixup in %0.  */
> -	"	lda %[fix], 0\n"
> -	"	cvttq/svm $f31,$f31\n"
> -	"	lda %[fix], 4"
> -	: [fix] "=r" (opDEC_fix)
> -	: [rti] "n" (PAL_rti), [wrent] "n" (PAL_wrent)
> -	: "$0", "$1", "$16", "$17", "$22", "$23", "$24", "$25");
> -
> -	if (opDEC_fix)
> -		printk("opDEC fixup enabled.\n");
> -}
> -

That should be moved into commit that removes the caller...