RFR: 8253717: Relocate stack overflow code out of thread.hpp/cpp [v6]

Thomas Stuefe stuefe at openjdk.java.net
Thu Oct 8 06:57:12 UTC 2020


On Thu, 8 Oct 2020 06:52:05 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> This change moves the significant amount of stack overflow related code (with ascii art!) out of thread files into a
>> new file.  Many of the functions are static functions and some go through JavaThread::_stack_overflow_state where
>> needed.   All functions are moved and not modified except for qualification.  I also added a delegating constructor to
>> JavaThread::JavaThread so reordered the assignments as initializers from JavaThread::initialize.
>> Tested with tier1-6 and builds on arm32, ppc, s390 and zero.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Revised comment

@coleenp: Looks all good to me now. Thanks for that work!

@dholmes-ora : I knew remember now the discussion as well. Seeing that it was only in February this is embarrassing :)
I still find the fact the base values are off by one odd but having them at crooked values (page ends) would be odd
too, so maybe its just a matter of brushing up comments.

-------------

Marked as reviewed by stuefe (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/522


More information about the hotspot-dev mailing list