|
Message-ID: <CAKGDhHX+2dB9jJ9esnDD_S8kRUep-ifVJoMGrRJ=j30PD65u=w@mail.gmail.com> Date: Mon, 6 Jul 2015 21:02:33 +0200 From: Agnieszka Bielec <bielecagnieszka8@...il.com> To: john-dev@...ts.openwall.com Subject: Re: PHC: Lyra2 on GPU 2015-07-06 20:48 GMT+02:00 magnum <john.magnum@...hmail.com>: > On 2015-07-06 18:53, Agnieszka Bielec wrote: >> >> 2015-07-06 10:25 GMT+02:00 Solar Designer <solar@...nwall.com>: >>> >>> On Mon, Jul 06, 2015 at 01:15:54AM +0200, magnum wrote: >>>> >>>> We could change it to always pass "db" to reset(). It could still *be* >>>> NULL but we'd never call it with an explicit NULL. >>>> (...) >>>> This would solve this issue but a side-effect is reset() can no longer >>>> tell whether we're about to self-test before a crack or actually run >>>> one. For resolving that we could simply change >>>> >>>> void fmt_reset(struct db_main *db); >>>> >>>> ...to >>>> >>>> void fmt_reset(struct db_main *db, int self_test); >>>> >>>> ...and a crack run would change to: >>>> reset(db, 1) >>>> self-tests >>>> reset(db, 0) >>>> crack mode >>>> >>>> Does this make sense? >>> >>> >>> Yes, this makes sense to me. Would we actually need to add this "int >>> self_test"? The few formats that care would be able to count the >>> reset() calls on their own, perhaps with the counter reset on init(). >> >> >> first printf() is in opencl_init, second in autotune_run() >> maybe just modify these functions to only printf once and call these >> functions as now it looks like in the code? > > > If I understand you right, you are looking at hiding the problem instead of > fixing it? the problem is only that something is printf'ed 2 times and user can be confused (as I think) or there is something more? opencl_init can be moved to init (if build_opts aren't changing), and maybe i can not-autotune for tests but I was just thinking about something else
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.