RFR: 8252148: vmError::controlled_crash should be #ifdef ASSERT and move tests to gtest [v4]

Thomas Stuefe stuefe at openjdk.java.net
Mon Dec 14 06:01:57 UTC 2020


On Fri, 11 Dec 2020 22:19:23 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Removed the ".*" in unittest.hpp and moved to the tests instead.
>
> Marked as reviewed by iklam (Reviewer).

Hi Coleen,

sorry, found another issue. In `print_error_for_unit_test` multiple threads may race and overwrite each others _detail_fmt buffer. Later on we block all threads but the first to report an error. But not here yet. So, maybe just use a C-heap allocated buffer or a stack buffer. I also don't think it has to be 1024, I guess 128 or so would be enough. We can make it larger if needed.

Cheers, Thomas

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

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


More information about the hotspot-dev mailing list