PR/57594 CVS commit: [netbsd-10] src/crypto/external/bsd/openssl/dist/crypto

["Martin Husemann" <martin%netbsd.org@localhost>]

The following reply was made to PR port-sparc/57594; it has been noted by GNATS.

From: "Martin Husemann" <martin%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/57594 CVS commit: [netbsd-10] src/crypto/external/bsd/openssl/dist/crypto
Date: Wed, 6 Sep 2023 08:00:12 +0000

 Module Name:	src
 Committed By:	martin
 Date:		Wed Sep  6 08:00:12 UTC 2023
 
 Modified Files:
 	src/crypto/external/bsd/openssl/dist/crypto [netbsd-10]: sparccpuid.S
 
 Log Message:
 Pull up following revision(s) (requested by mrg in ticket #349):
 
 	crypto/external/bsd/openssl/dist/crypto/sparccpuid.S: revision 1.11
 
 fix SPARC v8/v9 detection code.
 
 this code uses a trick where the encoding on both v8 and v9 are
 the same, and are not illegal instructions, but that the v9 one
 has a detectable difference than v8.
 
 the idea is that we perform a "subcc" (set condition codes) which
 sets "%ccr" on v9, which is an unimplemented "%asr2" on v8, then
 we read %ccr (v9) or %asr2 (v8), which will always be 0x99 on v9,
 and .. is non-trapping but impleentation defined on v8.
 
 for many implementations this returns the value of the %y reg.
 
 as nothing actually sets %y in this path, it remains the value it
 was most recently set to by something (anything), and if it just
 happens to be 0x99 then the v9 paths will be taken on v8.
 
 fix this by clearing the %y register before the potential read.
 
 fixes PR port-sparc/57594.  tested on ss20 and in qemu.  this
 version of the patch has been submitted upstream.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.9.6.1 -r1.9.6.2 \
     src/crypto/external/bsd/openssl/dist/crypto/sparccpuid.S
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.