Date: Sun, 13 Jan 2013 21:36:43 +0100 From: magnum <john.magnum@...hmail.com> To: john-dev@...ts.openwall.com Subject: Re: Changes to common-opencl.c On 13 Jan, 2013, at 19:09 , Claudio André <claudioandre.br@...il.com> wrote: > Em 13-01-2013 15:53, Claudio André escreveu: >> >> Could you please test it latter? You put a lot of specialized information there and i don't have the hardware. >> > > Maybe Bull has enough hardware. Anyway, it is suited for wide testing now, so please do it. Tried it on Bull and on OSX. I think it's starting to look really good already, I tried non-OpenCL formats, OpenCL ones, --test all/single formats and --list=opencl-devices. The latter is OK for now, we might come up with some cosmetical changes later but that is not important at all. Anyway all vendor-specific output seem to work for both nvidia and AMD. Personally I'd like to change Using device 0: GeForce GTX 570 to just Device 0: GeForce GTX 570 but that might not be the most important thing to bother with... On another less important note I think we should add the AMD specific name if available (list-devices shows this), so instead of just Device 1: Tahiti it could be Device 1: Tahiti (AMD Radeon HD 7900 Series) or perhaps Device 1: AMD Radeon HD 7900 Series (Tahiti) For parsing, try to follow how --groups are parsed in options.c and getopt.[ch]. It seems the provided functions can do most things we need. I haven't looked at it now but I'd expect some "documentation" within getopt.h. 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.