Date: Sat, 12 Jan 2013 00:38:29 +0100 From: magnum <john.magnum@...hmail.com> To: john-dev@...ts.openwall.com Subject: Re: SIGALRM and new 'done' method On 12 Jan, 2013, at 0:11 , magnum <john.magnum@...hmail.com> wrote: > On 11 Jan, 2013, at 23:59 , Claudio André <claudioandre.br@...il.com> wrote: >> Em 11-01-2013 20:39, magnum escreveu: >>> On another thought it looks from your patch like you have that commit. The problem then, is that bench_running is not set for some reason. It should be! >>> >>> magnum >> >> I tried a fake pull request and looked at bench.c and common-opencl.c. Everything seems to be ok. >> --- >> >> Previous to the patch, I put a printf("V: %d", bench_running) and i can see a lot of zeros and (at the end) some ones. I am not sure, but seems to me the error happened when bench_running change to 1. >> >> Anyway, we can deal with it latter when 'done' got created on bleeding. The ugly hack works for now. > > Yes, I just realized this is a race condition and your printf confirms that. It's a chain of events and conditions leading up to this. Your hack is not a good solution. I will look at it. Try reverting your hack and apply f8bf37d instead. Still a hack, but a better one (I hope). 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.