[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 14 Feb 2013 23:38:19 +0100
From: magnum <john.magnum@...hmail.com>
To: john-dev@...ts.openwall.com
Subject: Re: mschapv2-bitsliced conversion
On 14 Feb, 2013, at 19:44 , magnum <john.magnum@...hmail.com> wrote:
> On 14 Feb, 2013, at 11:14 , deepika dutta <deepikadutta_19@...oo.com> wrote:
>> Good this worked with the previous patch. I think the changes you did should be fine as they are in sync with what already existed in those files. Assembly language is hard to understand without experience. Anyways, there is a small problem in the previous patch w.r.t. multiple definitions of P.
>>
>> -- P is extern in DES_bs.h,
>> -- it is defined in DES_bs.c when C code is to be used and defined in assembly files when assembly code is to be used.
>> -- it is defined in DES_bs_b.c also for both cases.
>>
>> Though it didn't create any problem but I feel it is not a good approach to programming. Therefore, I had used extern in assembly file and did all the stuff which didn't work.
>
> Apparently, extern is not recognized by all assemblers.
>
>> But just now I re-looked the code, the latest patch should actually be discarded as just removing the DES_bs_vector P[64] from DES_bs_b.c in previous patch will work fine with keeping one definition of P.
>
> OK, so it will be declared from including the header. I haven't tried this yet, will do.
When I tried this, OMP stopped working (at least for 64-bit SSE). Not sure why.
magnum
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.
