[kaffe] Re: Bug Status of Kaffe

[Mark Wielaard]

--=-Txtba8p7JNFz8j7gDuhQ
Content-Type: text/plain
Content-Transfer-Encoding: 7bit

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


--=-Txtba8p7JNFz8j7gDuhQ
Content-Disposition: attachment; filename=warning.patch
Content-Transfer-Encoding: base64
Content-Type: text/x-patch; name=warning.patch; charset=ANSI_X3.4-1968

SW5kZXg6IGthZmZlL2thZmZldm0vam5pLmMNCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0NClJDUyBmaWxlOiAvY3ZzL2th
ZmZlL2thZmZlL2thZmZlL2thZmZldm0vam5pLmMsdg0KcmV0cmlldmluZyByZXZpc2lvbiAxLjk3
DQpkaWZmIC11IC1yMS45NyBqbmkuYw0KLS0tIGthZmZlL2thZmZldm0vam5pLmMJMyBOb3YgMjAw
MyAwNToyOTozMSAtMDAwMAkxLjk3DQorKysga2FmZmUva2FmZmV2bS9qbmkuYwkyNyBEZWMgMjAw
MyAyMDoyMTozNSAtMDAwMA0KQEAgLTMwMjQsNiArMzAyNCwxMiBAQA0KIAlyZXR1cm4gKHJldCk7
DQogfQ0KIA0KK3N0YXRpYyB2b2lkKg0KK0thZmZlX0dldFByaW1pdGl2ZUFycmF5Q3JpdGljYWwo
Sk5JRW52KiBlbnYsIGphcnJheSBhcnIsIGpib29sKiBpc2NvcHkpDQorew0KKyAgcmV0dXJuIChL
YWZmZV9HZXRCeXRlQXJyYXlFbGVtZW50cyhlbnYsIChqYnl0ZUFycmF5KWFyciwgaXNjb3B5KSk7
DQorfQ0KKw0KIHN0YXRpYyBqY2hhcioNCiBLYWZmZV9HZXRDaGFyQXJyYXlFbGVtZW50cyhKTklF
bnYqIGVudiwgamNoYXJBcnJheSBhcnIsIGpib29sKiBpc2NvcHkpDQogew0KQEAgLTMxNTksNiAr
MzE2NSwxMiBAQA0KIH0NCiANCiBzdGF0aWMgdm9pZA0KK0thZmZlX1JlbGVhc2VQcmltaXRpdmVB
cnJheUNyaXRpY2FsKEpOSUVudiogZW52LCBqYnl0ZUFycmF5IGFyciwgamJ5dGUqIGVsZW1zLCBq
aW50IG1vZGUpDQorew0KKyAgS2FmZmVfUmVsZWFzZUJ5dGVBcnJheUVsZW1lbnRzKGVudiwgKGpi
eXRlQXJyYXkpYXJyLCAoamJ5dGUqKWVsZW1zLCBtb2RlKTsNCit9DQorDQorc3RhdGljIHZvaWQN
CiBLYWZmZV9SZWxlYXNlQ2hhckFycmF5RWxlbWVudHMoSk5JRW52KiBlbnYsIGpjaGFyQXJyYXkg
YXJyLCBqY2hhciogZWxlbXMsIGppbnQgbW9kZSkNCiB7DQogCUJFR0lOX0VYQ0VQVElPTl9IQU5E
TElOR19WT0lEKCk7DQpAQCAtNDQyNSw4ICs0NDM3LDggQEANCiAJS2FmZmVfR2V0SmF2YVZNLA0K
IAlOVUxMLA0KIAlOVUxMLA0KLQlOVUxMLA0KLQlOVUxMLA0KKwlLYWZmZV9HZXRQcmltaXRpdmVB
cnJheUNyaXRpY2FsLA0KKwlLYWZmZV9SZWxlYXNlUHJpbWl0aXZlQXJyYXlDcml0aWNhbCwNCiAJ
TlVMTCwNCiAJTlVMTCwNCiAJTlVMTCwNCkluZGV4OiBrYWZmZS9rYWZmZXZtL2ppdDMvbWFjaGlu
ZS5jDQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09DQpSQ1MgZmlsZTogL2N2cy9rYWZmZS9rYWZmZS9rYWZmZS9rYWZmZXZt
L2ppdDMvbWFjaGluZS5jLHYNCnJldHJpZXZpbmcgcmV2aXNpb24gMS40NA0KZGlmZiAtdSAtcjEu
NDQgbWFjaGluZS5jDQotLS0ga2FmZmUva2FmZmV2bS9qaXQzL21hY2hpbmUuYwkyIE5vdiAyMDAz
IDE3OjUxOjU5IC0wMDAwCTEuNDQNCisrKyBrYWZmZS9rYWZmZXZtL2ppdDMvbWFjaGluZS5jCTI3
IERlYyAyMDAzIDIwOjIxOjM2IC0wMDAwDQpAQCAtNjExLDcgKzYxMSwxMiBAQA0KIAkJCWUtPnN0
YXJ0X3BjID0gZ2V0SW5zblBDKGUtPnN0YXJ0X3BjLCBjb2RlSW5mbywgY29kZSkgKyAodWludHAp
Y29kZS0+Y29kZTsNCiAJCQllLT5lbmRfcGMgPSBnZXRJbnNuUEMoZS0+ZW5kX3BjLCBjb2RlSW5m
bywgY29kZSkgKyAodWludHApY29kZS0+Y29kZTsNCiAJCQllLT5oYW5kbGVyX3BjID0gZ2V0SW5z
blBDKGUtPmhhbmRsZXJfcGMsIGNvZGVJbmZvLCBjb2RlKSArICh1aW50cCljb2RlLT5jb2RlOw0K
LQkJCWFzc2VydCAoZS0+c3RhcnRfcGMgPD0gZS0+ZW5kX3BjKTsNCisJCQlpZiAoZS0+c3RhcnRf
cGMgPiBlLT5lbmRfcGMpDQorCQkJICBwcmludGYoIldBUk5JTkcgc3RhcnRfcGM9JWQgZW5kX3Bj
PSVkIGluICVzLiVzKCVzKVxuIiwNCisJCQkJIGUtPnN0YXJ0X3BjLCBlLT5lbmRfcGMsDQorCQkJ
CSBDTEFTU19DTkFNRShtZXRoLT5jbGFzcyksDQorCQkJCSBtZXRoLT5uYW1lLT5kYXRhLA0KKwkJ
CQkgTUVUSE9EX1NJR0QobWV0aCkpOw0KIAkJfQ0KIAl9DQogDQo=

--=-Txtba8p7JNFz8j7gDuhQ--