Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Sat, 19 Jan 2013 14:30:06 +0100
From: magnum <>
Subject: Re: New shared find_best_gws

On 16 Jan, 2013, at 3:38 , Claudio André <> wrote:
> Hi, i created a new shared find_best_GWS.
> It uses:
> - format->set_salt, format->set_key, format->crypt_all, etc.
> - requires create_clobj and release_clobj (you have to give pointers to them so find_best_GWS can (m)alloc necessary memory).
> - (maybe) a new crypt_all_benchmark function (with unique events for all OpenCL commands).
> It can handle a set of formats for sure. Anyway, it will be nice if new formats authors try it (at least to show its problems).
> As usual, you can dance naked using legacy code and it still works.

Good stuff. I see you also added a opencl_find_best_lws() function. Is that a new version of opencl_find_best_workgroup() with more flexibility (eg. for split-kernel formats)?

I think we might want to adopt core JtR style and move/add comments to common-opencl.h. For example, this comment in the .c file might be better put in the .h file:

 *   NOTE: Requirements for using this function:
 * - Your kernel (or main kernel) should be crypt_kernel.
 * - Use profilingEvent in your crypt_all() when enqueueing crypt_kernel.
 * - Do not use profilingEvent for transfers or other subkernels.
 * - For split kernels, use firstEvent and lastEvent instead.

And similar how-to comments should be added for some/many of the other functions. For deprecated functions (which we might end up making static or rationalize away eventually) we could point to the newer/better alternative. Obviously we shouldn't bother with this for unstable, only bleeding.

This would be of great help for developers trying to catch up, including myself!


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.