Re: patch to improve AES-NI performance

2013-08-27 Thread Outback Dingo
On Sun, Aug 25, 2013 at 10:19 AM, Ollivier Robert robe...@keltia.freenix.fr wrote: According to Ollivier Robert: You are right, I wanted to say r226837 which is the code one. FYI I've finally merged r226837,r226839 as r254856 in stable/9 as it is a prerequesite to apply jmg's patch. I've

Re: patch to improve AES-NI performance

2013-08-27 Thread John-Mark Gurney
Outback Dingo wrote this message on Tue, Aug 27, 2013 at 19:01 -0400: On Sun, Aug 25, 2013 at 10:19 AM, Ollivier Robert robe...@keltia.freenix.fr wrote: According to Ollivier Robert: You are right, I wanted to say r226837 which is the code one. FYI I've finally merged r226837,r226839

Re: patch to improve AES-NI performance

2013-08-25 Thread Ollivier Robert
According to Ollivier Robert: You are right, I wanted to say r226837 which is the code one. FYI I've finally merged r226837,r226839 as r254856 in stable/9 as it is a prerequesite to apply jmg's patch. I've asked re@ whether they would consider this for 9.2. It is very late in the 9.2 release

Re: patch to improve AES-NI performance

2013-08-23 Thread Ollivier Robert
According to John-Mark Gurney on Thu, Aug 22, 2013 at 01:20:27PM -0700: I have developed a patch to improve AES-NI performance. If you took the AES-XTS algorithm into userland (no cryptodev or geli usage), these changes improve the performance over 10x in my tests (from ~150MB/sec to over 2GB

Re: patch to improve AES-NI performance

2013-08-23 Thread Mike Tancsa
On 8/23/2013 11:16 AM, Ollivier Robert wrote: According to John-Mark Gurney on Thu, Aug 22, 2013 at 01:20:27PM -0700: I have developed a patch to improve AES-NI performance. If you took the AES-XTS algorithm into userland (no cryptodev or geli usage), these changes improve the performance

Re: patch to improve AES-NI performance

2013-08-23 Thread John-Mark Gurney
Ollivier Robert wrote this message on Fri, Aug 23, 2013 at 17:16 +0200: According to John-Mark Gurney on Thu, Aug 22, 2013 at 01:20:27PM -0700: I have developed a patch to improve AES-NI performance. If you took the AES-XTS algorithm into userland (no cryptodev or geli usage

Re: patch to improve AES-NI performance

2013-08-23 Thread John-Mark Gurney
Mike Tancsa wrote this message on Fri, Aug 23, 2013 at 11:26 -0400: On 8/23/2013 11:16 AM, Ollivier Robert wrote: According to John-Mark Gurney on Thu, Aug 22, 2013 at 01:20:27PM -0700: I have developed a patch to improve AES-NI performance. If you took the AES-XTS algorithm into userland

Re: patch to improve AES-NI performance

2013-08-23 Thread Mike Tancsa
On 8/23/2013 2:05 PM, John-Mark Gurney wrote: Speeding up userland AES is very interesting to me for a couple of apps. If there is a proper way I should test on RELENG_9, please let me know as I am few boxes that I would be happy to test/deploy on. My patch would only effect userland

Re: patch to improve AES-NI performance

2013-08-23 Thread Ollivier Robert
According to John-Mark Gurney: pjd did not merge (as he said at the time of commit) r226839 r226839. Is there any objection to merge these two (and possibly 247061 as well -- copyright update)? You repeated r226839 twice. What is the correct second revision? You are right, I wanted to say

Re: patch to improve AES-NI performance

2013-08-23 Thread John-Mark Gurney
Mike Tancsa wrote this message on Fri, Aug 23, 2013 at 14:19 -0400: On 8/23/2013 2:05 PM, John-Mark Gurney wrote: Speeding up userland AES is very interesting to me for a couple of apps. If there is a proper way I should test on RELENG_9, please let me know as I am few boxes that I would

Re: patch to improve AES-NI performance

2013-08-23 Thread Dag-Erling Smørgrav
John-Mark Gurney j...@funkthat.com writes: Mike Tancsa m...@sentex.net writes: John-Mark Gurney j...@funkthat.com writes: My patch would only effect userland applications that use /dev/crypto... For me its ssh which I think does, no ? It looks like it uses OpenSSL for it's crypto, not

Re: patch to improve AES-NI performance

2013-08-23 Thread John-Mark Gurney
Dag-Erling Smrgrav wrote this message on Fri, Aug 23, 2013 at 21:30 +0200: John-Mark Gurney j...@funkthat.com writes: Mike Tancsa m...@sentex.net writes: John-Mark Gurney j...@funkthat.com writes: My patch would only effect userland applications that use /dev/crypto... For me its ssh

patch to improve AES-NI performance

2013-08-22 Thread John-Mark Gurney
I have developed a patch to improve AES-NI performance. If you took the AES-XTS algorithm into userland (no cryptodev or geli usage), these changes improve the performance over 10x in my tests (from ~150MB/sec to over 2GB/sec). In tests of geli on gnop, the performance improvement is more