[kaffe] CVS kaffe (guilhem): Stack detection for unix-pthreads

[Dalibor Topic]

Fabien Renaud wrote:
> Fabien Renaud wrote:
> 
>> Kaffe CVS wrote:
>>
>>> PatchSet 4887 Date: 2004/07/04 14:19:54
>>> Author: guilhem
>>> Branch: HEAD
>>> Tag: (none) Log:
>>> Stack detection for unix-pthreads
>>>
>>>     * kaffe/kaffevm/systems/unix-pthreads/jsignal.h,
>>>     kaffe/kaffevm/systems/unix-pthreads/signal.c,
>>>     kaffe/kaffevm/systems/unix-pthreads/syscalls.c,
>>>     kaffe/kaffevm/systems/unix-pthreads/thread-impl.c:
>>>     Imported stack pointer detection in the unix-pthreads
>>>     implementation of kaffe threads.
>>>
>>> Members:     ChangeLog:1.2453->1.2454     
>>> kaffe/kaffevm/systems/unix-pthreads/jsignal.h:1.1->1.2     
>>> kaffe/kaffevm/systems/unix-pthreads/signal.c:1.3->1.4     
>>> kaffe/kaffevm/systems/unix-pthreads/syscalls.c:1.15->1.16 
>>
>> Hello,
>>
>> I have some problems with this patch :
>> signal.c: In function `nullException':
>> signal.c:102: error: `ctx' undeclared (first use in this function)
>> signal.c:102: error: (Each undeclared identifier is reported only once
>> signal.c:102: error: for each function it appears in.)
>> signal.c: In function `restoreAsyncSignalHandler':
>> signal.c:233: warning: unused parameter `sig'
>> signal.c:233: warning: unused parameter `handler'
>> signal.c: In function `restoreSyncSignalHandler':
>> signal.c:247: warning: unused parameter `sig'
>> signal.c:247: warning: unused parameter `handler'
>> signal.c: In function `stackOverflowDetector':
>> signal.c:437: warning: unused parameter `arm_r1'
>> signal.c:437: warning: unused parameter `arm_r2'
>> signal.c:437: warning: unused parameter `arm_r3'
>> signal.c:437: warning: unused parameter `sc'
>> signal.c: At top level:
>> signal.c:444: warning: no previous prototype for `kaffeNoopFunc'
>> signal.c: In function `detectStackBoundaries':
>> signal.c:479: warning: passing arg 1 of `kaffeNoopFunc' with different 
>> width due to prototype
>>
>>
>> Cheers,
>>
>> Fabien
> 
> 
> I have the problem with x86 too.
> My configure :
> 
> ./configure --disable-nls --enable-pure-java-math 
> --with-threads=unix-pthreads --with-engine=intrp --with-libffi 
> --with-jikes --without-x

Salut Fabien,

thanks a lot for the bug report. I've checked in a fix into the CVS.

cheers,
dalibor topic