ppccap.c: fix compiler warning and perform sanity check outside signal masking.
authorAndy Polyakov <appro@openssl.org>
Tue, 29 Dec 2009 11:18:16 +0000 (11:18 +0000)
committerAndy Polyakov <appro@openssl.org>
Tue, 29 Dec 2009 11:18:16 +0000 (11:18 +0000)
ppc64-mont.pl: clarify comment and fix spelling.

crypto/bn/asm/ppc64-mont.pl
crypto/ppccap.c

index 985a2b2..51e6bbb 100644 (file)
 # 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.
index 8590159..2c3e1f2 100644 (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;