|
Message-ID: <20120317041526.GA16046@openwall.com> Date: Sat, 17 Mar 2012 08:15:26 +0400 From: Solar Designer <solar@...nwall.com> To: john-dev@...ts.openwall.com Subject: Re: SSH thread-safety On Sat, Mar 17, 2012 at 08:54:29AM +0530, Dhiru Kholia wrote: > On Sat, Mar 17, 2012 at 7:38 AM, Dhiru Kholia <dhiru.kholia@...il.com> wrote: > > I have an (untested idea). Why not copy PEM_do_header function from > > OpenSSL and remove all thread-unsafe error handling which we don't use > > anyway? > > I have implemented this idea. Performance is back and no random > segfaults :-). Please give the latest code a shot. Thanks! Wow, that was quick. So I got an "undeclared" error about EVP_CIPH_FLAG_CUSTOM_CIPHER. I simply #if 0'ed that piece, and the thing compiled and seems to work. The speed is slightly better than it was before this round of changes (that is, before your introduction of locking). :-) I am getting almost 700k c/s on 8-core. I am running the high thread count tests now. My biggest concern is that the code is highly non-portable to different versions of OpenSSL now. I am not sure what we can do about that. Thanks, Alexander
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.