RFR(XS): 8191504: CDSTestUtils.isUnableToMap() should check OptionalData region mapping failure

mikhailo mikhailo.seledtsov at oracle.com
Tue Nov 28 00:08:35 UTC 2017


Looks good,

Misha


On 11/27/2017 03:25 PM, Jiangli Zhou wrote:
> Hi,
>
> Please review following small test fix. CDS has multiple archived memory regions, which are mapped individually at runtime. CDSTestUtils.isUnableToMap() checks for mapping failures, but missing the ‘OptionalData’ memory region. The fix in the webrev adds the check for ‘OptionalData’ region. It also updated the error checking for archived heap memory region.
>
> webrev: http://cr.openjdk.java.net/~jiangli/8191504/webrev.00/ <http://cr.openjdk.java.net/~jiangli/8191504/webrev.00/>
> bug: https://bugs.openjdk.java.net/browse/JDK-8191504?filter=14921 <https://bugs.openjdk.java.net/browse/JDK-8191504?filter=14921>
>
> Tested with tier1, trer2, tier3, tier4, and tier5.
>
> Thanks,
> Jiangli



More information about the hotspot-runtime-dev mailing list