[kaffe] Re: Bug Status of Kaffe

[Mark Wielaard]

Hi,

On Thu, 2003-12-25 at 03:12, Ben Burton wrote:
> > > #210716 jython causes kaffe to fail with assert error
> > >
> > >         ,----
> > >         | Version: 1:1.1.1-1
> > >         |
> > >         | > After removing the JNI lines from jython shell script (see
> > >         | > issue #207998) kaffe dies with kaffe-bin: machine.c:620:
> > >         | > installMethodCode: Assertion `e->start_pc <= e->end_pc'
> > >         | > failed.
> [...]
> But this crash is not a debian-specific bug.  The bug here is that jython
> causes kaffe to crash on startup *after* all of the required classes have
> been found - it has nothing to do with the library path.
> [..]
> This is where this crash comes up.  Once you have kaffe finding all of the
> JNI libraries that it should (including the ones used with jython and the
> kaffe bootstrap classes), it then crashes with 'e->start_pc <= e->end_pc'
> failed.
> 
> So this is definitely a kaffe issue, not a debian-specific issue or a JNI
> path issue.

Seen the same crash when using the CVS view from Eclipse 3.0M4 on kaffe.
<http://kaffe.org/pipermail/kaffe/2003-October/044318.html>
I just removed the assert.

The problem is actually the byte code generated by the compiler since it
should not generate an exception table entry whose start_pc is smaller
(or equal - the assert is actually wrong) to end_pc.

The attached patch turns the assert into a printf WARNING which can help
debug the byte code. In my eclipse case it says:
WARNING start_pc=164661273 end_pc=164661236 in
org/eclipse/team/internal/core/streams/TimeoutInputStream.runThread(()V)
WARNING start_pc=164661775 end_pc=164661236 in
org/eclipse/team/internal/core/streams/TimeoutInputStream.runThread(()V)
WARNING start_pc=161990681 end_pc=161990644 in
org/eclipse/team/internal/core/streams/TimeoutOutputStream.runThread(()V)
WARNING start_pc=161991183 end_pc=161990644 in
org/eclipse/team/internal/core/streams/TimeoutOutputStream.runThread(()V)

(Also included the other things needed to patch/hack around for eclipse)

Cheers,

Mark

-------------- next part --------------
A non-text attachment was scrubbed...
Name: warning.patch
Type: text/x-patch
Size: 2033 bytes
Desc: not available
Url : http://kaffe.org/pipermail/kaffe/attachments/20031227/8a230022/attachment-0002.bin