Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <fdc69306459e425b6053257baf948f5a@smtp.hushmail.com>
Date: Thu, 15 Oct 2015 23:02:56 +0200
From: magnum <john.magnum@...hmail.com>
To: john-dev@...ts.openwall.com
Subject: Re: 64-bit rotate on AMD GCN

On 2015-10-15 22:25, Pavel Semjanov wrote:
> On 12.10.2015 20:41, magnum wrote:
>> On 2015-10-10 06:52, Solar Designer wrote:
>>> #define ror(x, n)       ((n) < 32 ? (amd_bitalign((uint)((x) >> 32),
>>> (uint)(x), (uint)(n)) | ((ulong)amd_bitalign((uint)(x), (uint)((x) >>
>>> 32), (uint)(n)) << 32)) : (amd_bitalign((uint)(x), (uint)((x) >> 32),
>>> (uint)(n) - 32) | ((ulong)amd_bitalign((uint)((x) >> 32), (uint)(x),
>>> (uint)(n) - 32) << 32)))
>>
>> Thanks, this went into
>> https://github.com/magnumripper/JohnTheRipper/issues/1819 and most or
>> all applicable formats now use the above.
>
> Not working on small numbers and rotate by 8, like ror (0x220, 8).
> I guess it's bitalign error. The only one mention I found is:
> https://community.amd.com/thread/158878

What device and driver version(s) did you see that with? I recall Atom 
told me he'd seen rotate() fail with numbers divisible by 8.  I'm pretty 
sure he meant the OpenCL function but it could be the same underlying 
bug. That was in June last year so maybe Cat 14.4 or something. I never 
saw that very bug surface though.

sigma0 has a rotate by 8 but I see no problems with 15.7. I just tested 
bull's 13.4 and it works fine (although it's a *LOT* slower than 15.7 
with Myrice's formats and it can't even build any of Claudio's SHA-512 
formats).

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.