wbrawner/openssl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

ppccap.c: fix compiler warning and perform sanity check outside signal masking.

Browse source 
ppc64-mont.pl: clarify comment and fix spelling.
This commit is contained in:
Andy Polyakov 2009-12-29 11:18:16 +00:00
parent 6a9d28f9e4
commit 70b76d392f
2 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
crypto/bn/asm/ppc64-mont.pl
View file

@ -56,10 +56,10 @@
# that asynchronous signals have to be blocked upon entry to this
# subroutine. Signal masking (and complementary unmasking) has quite
# an impact on performance, naturally larger for shorter keys. It's
# so severe that shorter key performance can be as low as 1/3 of
# so severe that 512-bit key performance can be as low as 1/3 of
# expected one. This is why this routine can be engaged for longer
# key operations only, see crypto/ppccap.c for further details.
# Alternative is to break dependance on upper halves on GPRs...
# Alternative is to break dependence on upper halves on GPRs...
# MacOS X is an exception from this and doesn't require signal
# masking, and that's where above improvement coefficients were
# collected.

4
crypto/ppccap.c
View file

@ -25,7 +25,7 @@ int bn_mul_mont(BN_ULONG *rp, const BN_ULONG *ap, const BN_ULONG *bp, const BN_U
#else
/* boundary of 32 was experimentally determined on
Linux 2.6.22, might have to be adjusted on AIX... */
if ((num>=32) && (OPENSSL_ppccap_P&PPC_FPU64))
if (num>=32 && (num&3)==0 && (OPENSSL_ppccap_P&PPC_FPU64))
{
sigset_t oset;
int ret;
@ -50,6 +50,8 @@ int bn_mul_mont(BN_ULONG *rp, const BN_ULONG *ap, const BN_ULONG *bp, const BN_U
static sigjmp_buf ill_jmp;
static void ill_handler (int sig) { siglongjmp(ill_jmp,sig); }
void OPENSSL_ppc64_probe(void);
void OPENSSL_cpuid_setup(void)
{
char *e;