On Fri Sep 05 10:17:30 2008, doughera wrote: > > This patch takes the following small steps: > > First, I replaced the > retval = *(int*)i->si_ptr; > line by > retval = *(int*)i->si_value.sival_ptr; > I've never fiddled with this stuff before, but it looks like the > si_ptr > #define is not necessarily very portable. I replaced it by the > si_value > and sival_ptr fields, which I hope are slightly more portable. > > Second, it moves the small step of moving the logic of finding > SIGRTMIN + 1 into the C program, where it's trivial. > > Next, it adds in an else branch for the case where the program builds > but does not run successfully. > > Finally, made sure the 'aio' and 'HAS_AIO' entries are always set to > some value. > > I haven't looked for race conditions at all, but with this patch, I'll > at > least be more confident that it's trying to do what we think it's > trying > to do. >
Thanks, Andy. Applied in r30800 after testing on Linux and Darwin.