RFR: 8263460: DynamicArchiveRelocationTest.java fails in product VM

Daniel D.Daugherty dcubed at openjdk.java.net
Fri Mar 12 17:23:07 UTC 2021


On Fri, 12 Mar 2021 07:46:10 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> Please review this test bug. 
>> 
>> The handling of -XX:ArchiveRelocationMode=1 is different between fastdebug and product builds:
>> 
>> - fastdebug: (to improve test coverage) -- we always map at the default location, then unmap, and then map again at a random location.
>> - product: (for faster performance) -- don't attempt to map at the default location. Just map at random location
>> 
>> Therefore, the "unmap" messages are not available in product mode.
>> 
>> I added comments to explain what output is expected, and rearranged the regexp for better matching.
>
> The special handling for Windows is not needed anymore?
> 
> I have the feeling this could be made simpler in general (not in this fix). E.g. instead of mapping, then unmapping, which causes the execution paths between non-product and product to be quite different, why not just reserve a "rogue" allocation in the middle of the soon-to-map region? Would that not be a more reliable and realistic test?
> 
> Cheers, Thomas

@ioilam - Any chance that this fix will get integrated before the weekend?
I'm interested in taking the two failures per Tier6 off the board for the weekend.

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

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


More information about the hotspot-runtime-dev mailing list