Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1340690055.39821.YahooMailNeo@web161305.mail.bf1.yahoo.com>
Date: Mon, 25 Jun 2012 22:54:15 -0700 (PDT)
From: deepika dutta <deepikadutta_19@...oo.com>
To: john-dev@...ts.openwall.com
Subject: Re: mschap-v2 conversion

hi, thanks for testing the patch, I will look into the openmp related details in the code and make the next patch fully functional (atleast for generic build for now).  Cheers,
Deepika


________________________________
 From: magnum <john.magnum@...hmail.com>
To: john-dev@...ts.openwall.com 
Sent: Tuesday, June 26, 2012 3:51 AM
Subject: Re: [john-dev] mschap-v2 conversion
 
On 2012-06-25 21:30, deepika dutta wrote:
> I have done the change for key (xoring it with 0x80) before giving to
> DES_bs_set_key(), now it passes the test suite cases without any
> failure. The updated patch is attached.

I trust you about the Test Suite so did not check now. But something seems to be amiss with OMP:

Jumbo-5:
Benchmarking: MSCHAPv2 C/R MD4 DES [mschapv2]... DONE
Many salts:    2330K c/s real, 2330K c/s virtual
Only one salt:    1570K c/s real, 1570K c/s virtual

Benchmarking: MSCHAPv2 C/R MD4 DES [mschapv2]... (2xOMP) DONE
Many salts:    4200K c/s real, 2121K c/s virtual
Only one salt:    2717K c/s real, 1372K c/s virtual

Bitslice:
Benchmarking: MSCHAPv2 C/R MD4 DES [DES_BS_MSCHAPv2]... DONE
Many salts:    4488K c/s real, 4488K c/s virtual
Only one salt:    2280K c/s real, 2280K c/s virtual

Benchmarking: MSCHAPv2 C/R MD4 DES [DES_BS_MSCHAPv2]... (2xOMP) DONE
Many salts:    2946K c/s real, 1480K c/s virtual
Only one salt:    2077K c/s real, 1044K c/s virtual


The very last benchmark of the above indicates a bug: "Virtual" for OMP should ideally be close to the one for non-OMP. And real should ideally approach 2x virtual. In any case it should not be slower than non-OMP.

Also, for your next patch I think it's time you add a copyright notice like explained in http://openwall.info/wiki/john/licensing

magnum
Content of type "text/html" skipped

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.