RFR(L): 10: JDK-8177728 - [TESTBUG] Improve CDS test utils
jiangli.zhou at oracle.com
Tue Apr 18 18:09:00 UTC 2017
Looks good. No new webrev needed for me after you fix the typos that Calvin found.
> On Apr 13, 2017, at 7:33 PM, Mikhailo Seledtsov <mikhailo.seledtsov at oracle.com> wrote:
> Thank you for review. I fixed TransformRelatedClasses.java to use new CDSTestUtils.
> Here are updated webrevs that address feedback from you and Calvin.
> Thank you,
> On 4/4/17, 5:53 PM, Jiangli Zhou wrote:
>> Hi Misha,
>> Looks good. TransformRelatedClasses.java was not changed to use the utility methods. Was that on purpose?
>>> On Mar 28, 2017, at 3:09 PM, mikhailo<mikhailo.seledtsov at oracle.com> wrote:
>>> Please review this enhancement to CDS tests (RFE)
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8177728
>>> Here is a brief summary of the improvements:
>>> - create/use CDS utility methods for common patterns in CDS related tests:
>>> - creating an testlist
>>> - creating an archive, checking result
>>> - executing JVM with the archive
>>> - checking results and common error patterns
>>> - use -Xshare:on when executing CDS tests, and use test utilities
>>> to filter out failures due to inability to map shared archive.
>>> This is a more deterministic way to execute CDS tests.
>>> - additional improvements came up as part of this work,
>>> such as jdk.test.lib.Utils:getTestName()
>>> This work also lays ground for future improvements in this area.
>>> 1. Locally: executed affected tests on Linux-x64
>>> 2. Automated multi-platform testing
>>> Running the affected tests on a set of supported platforms.
>>> IN Progress...
>>> Thank you,
More information about the hotspot-runtime-dev