RFR: 8253794: TestAbortVMOnSafepointTimeout never timeouts [v2]

[Robbin Ehn]

On Mon, 5 Oct 2020 15:14:16 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

>> Robbin Ehn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update with input from reviews
>
> src/hotspot/share/prims/whitebox.cpp line 77:
> 
>> 75: #include "runtime/jniHandles.inline.hpp"
>> 76: #include "runtime/os.hpp"
>> 77: #include "runtime/safepoint.hpp"
> 
> I don't think you need this include change anymore.

Fixed

> test/hotspot/jtreg/runtime/Safepoint/TestAbortVMOnSafepointTimeout.java line 74:
> 
>> 72:             Integer waitTime = Integer.parseInt(args[0]);
>> 73:             WhiteBox wb = WhiteBox.getWhiteBox();
>> 74:             // While no safepoint timeout.
> 
> Perhaps: // Loop here to cause a safepoint timeout.

Fixed

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

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