RFR: 8133646: Internal Error: x86/vm/macroAssembler_x86.cpp:886 DEBUG MESSAGE: StubRoutines::call_stub: threads must correspond
david.holmes at oracle.com
Wed Aug 26 02:17:19 UTC 2015
Thanks for the review.
On 26/08/2015 2:28 AM, Coleen Phillimore wrote:
> Can you add this duplicate code into a verify_thread() function? This
> would help if we are ever able to combine common code in
I'm so grateful you posed that as a question not a request. :) Answer: no.
The duplicated structure is pre-existing, and the error messages differ
so the context would have to be passed in. I'm totally unfamiliar with
playing in the macroassembler so would be unsure how to factor this out
into a separate function without messing up register usage, or using the
correct "variables" from the "calling context".
Also note that for some reason these sanity checks don't exist on
x86_32, nor sparc (only aarch64 also has it presumably based on the
x86_64 version). I think the time to factor this out would be if/when we
do the code combining.
> On 8/25/15 2:07 AM, David Holmes wrote:
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8133646
>> webrev: http://cr.openjdk.java.net/~dholmes/8133646/webrev/
>> The error raised in this bug was not reproducible - it was only seen
>> on one machine which was subsequently re-imaged to run Windows. A
>> similar Solaris system did not reproduce the bug.
>> Rather than just close this as not-reproducible I want to push the
>> enhanced diagnostics that Vladimir provided here:
>> which I've added to the two chunks of code that perform this thread
>> sanity check.
More information about the hotspot-dev