-
Enhancement
-
Resolution: Fixed
-
P4
-
hs18
-
b04
-
generic
-
generic
Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
---|---|---|---|---|---|---|
JDK-2192536 | 7 | Christian Thalinger | P4 | Closed | Fixed | b92 |
JDK-2198013 | 6u23 | Christian Thalinger | P4 | Closed | Fixed | b01 |
JDK-2199964 | 6u22m | Christian Thalinger | P4 | Closed | Fixed | b01 |
JDK-2193548 | 6u21p | Christian Thalinger | P4 | Closed | Fixed | b03 |
Gary Benson wrote:
Hi all,
I recently started adding stack overflow checking to Shark,
which brought to light a bunch of deficiencies in Zero's
stack overflow code. Zero had this really odd, look-ahead
overflow handling which was pretty strange, easily fooled,
and in retrospect totally unnecessary. I'm pretty sure
whoever wrote it was an idiot.
This webrev removes the old overflow detection and replaces
it with something more straightforward and complete:
http://cr.openjdk.java.net/~gbenson/zero-stack-overflow-checks/
Note that a number of calls to Unimplemented have been removed.
It shouldn't be possible to have an unhandled stack overflow
error with the new code.
As a bonus, the new code is mildly faster (about 2%) due to
the checks being inlined.
I don't have a bug id for this.
Cheers,
Gary
Hi all,
I recently started adding stack overflow checking to Shark,
which brought to light a bunch of deficiencies in Zero's
stack overflow code. Zero had this really odd, look-ahead
overflow handling which was pretty strange, easily fooled,
and in retrospect totally unnecessary. I'm pretty sure
whoever wrote it was an idiot.
This webrev removes the old overflow detection and replaces
it with something more straightforward and complete:
http://cr.openjdk.java.net/~gbenson/zero-stack-overflow-checks/
Note that a number of calls to Unimplemented have been removed.
It shouldn't be possible to have an unhandled stack overflow
error with the new code.
As a bonus, the new code is mildly faster (about 2%) due to
the checks being inlined.
I don't have a bug id for this.
Cheers,
Gary
- backported by
-
JDK-2192536 Improved stack overflow handling for Zero
- Closed
-
JDK-2193548 Improved stack overflow handling for Zero
- Closed
-
JDK-2198013 Improved stack overflow handling for Zero
- Closed
-
JDK-2199964 Improved stack overflow handling for Zero
- Closed
- relates to
-
JDK-6962193 Rebrand some Zero files
- Resolved
-
JDK-6944473 6941224 misses new files
- Closed
(1 relates to)