From owner-freebsd-stable@FreeBSD.ORG Wed Nov 17 19:19:08 2010 Return-Path: Delivered-To: freebsd-stable@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id A1CF4106566C for ; Wed, 17 Nov 2010 19:19:08 +0000 (UTC) (envelope-from mike@sentex.net) Received: from smarthost1.sentex.ca (smarthost1-6.sentex.ca [IPv6:2607:f3e0:0:1::12]) by mx1.freebsd.org (Postfix) with ESMTP id DD5BB8FC1C for ; Wed, 17 Nov 2010 19:19:07 +0000 (UTC) Received: from [IPv6:2607:f3e0:0:4:808a:bf97:8eb:f23b] ([IPv6:2607:f3e0:0:4:808a:bf97:8eb:f23b]) by smarthost1.sentex.ca (8.14.4/8.14.4) with ESMTP id oAHJIwGj072293 (version=TLSv1/SSLv3 cipher=DHE-RSA-CAMELLIA256-SHA bits=256 verify=NO); Wed, 17 Nov 2010 14:18:58 -0500 (EST) (envelope-from mike@sentex.net) Message-ID: <4CE42A9A.5030001@sentex.net> Date: Wed, 17 Nov 2010 14:18:50 -0500 From: Mike Tancsa User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.12) Gecko/20101027 Thunderbird/3.1.6 MIME-Version: 1.0 To: Kostik Belousov , freebsd-stable@freebsd.org References: <20101115211350.GE2392@deviant.kiev.zoral.com.ua> <4CE1FDBA.9030403@sentex.net> <20101116094330.GH2392@deviant.kiev.zoral.com.ua> <4CE300DE.8010304@sentex.net> <20101116221926.GN2392@deviant.kiev.zoral.com.ua> <4CE333EF.10406@sentex.net> <20101117163541.GR2392@deviant.kiev.zoral.com.ua> In-Reply-To: <20101117163541.GR2392@deviant.kiev.zoral.com.ua> X-Enigmail-Version: 1.1.1 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.67 on IPv6:2607:f3e0:0:1::12 Cc: Subject: Re: Call for testers: FPU changes X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 17 Nov 2010 19:19:08 -0000 On 11/17/2010 11:35 AM, Kostik Belousov wrote: > Meantime, the similar change may be beneficial for padlock(4) too. > f you are going to test it, please note that most likely, openssl padlock > engine does not use padlock(4), I do not know for sure. > > diff --git a/sys/crypto/via/padlock.c b/sys/crypto/via/padlock.c > index 77e059b..ba63093 100644 > --- a/sys/crypto/via/padlock.c > +++ b/sys/crypto/via/padlock.c Patch applied cleanly Full results at the bottom of http://www.tancsa.com/fpu.html On large blocks, version 1 vs the above patch show no significant difference. This is with openssl using the cryptodev engine. I also compared to the openssl padlock engine which gave interesting results! 0(via)# cat version1.txt | sed -e 's/k//g' | awk '{print $6}' > 1 0(via)# cat version2.txt | sed -e 's/k//g' | awk '{print $6}' > 2 0(via)# ministat 1 2 x 1 + 2 N Min Max Median Avg Stddev x 30 2591851.6 6645345.1 4326340.6 4227917.6 1083181.2 + 30 2574883.9 8830282.8 4033610.4 4241195.6 1519334.8 No difference proven at 95.0% confidence 0(via)# cat version1.txt | sed -e 's/k//g' | awk '{print $5}' > 1 0(via)# cat version2.txt | sed -e 's/k//g' | awk '{print $5}' > 2 0(via)# ministat 1 2 N Min Max Median Avg Stddev x 30 1124673.3 2320883.7 1527677.1 1550631.9 295165.4 + 30 1069788.2 2508865.7 1594506.2 1588193.2 389414.33 No difference proven at 95.0% confidence 0(via)#