[11u] RFR: 8212028: Use run-test makefile framework for testing in Oracle's Mach5

[Langer, Christoph]

Hi Severin,

good catch, thank you.

Adjusted webrev: http://cr.openjdk.java.net/~clanger/webrevs/8212028.11u-dev.1/

Best regards
Christoph

> -----Original Message-----
> From: Severin Gehwolf <sgehwolf at redhat.com>
> Sent: Donnerstag, 10. Oktober 2019 15:15
> To: Langer, Christoph <christoph.langer at sap.com>; jdk-updates-
> dev at openjdk.java.net; build-dev <build-dev at openjdk.java.net>
> Subject: Re: [11u] RFR: 8212028: Use run-test makefile framework for testing
> in Oracle's Mach5
> 
> Hi Christoph,
> 
> On Thu, 2019-10-10 at 12:16 +0000, Langer, Christoph wrote:
> > Hi,
> >
> > please help reviewing this backport of a build infrastructure change to
> jdk11u.
> >
> > One reason for doing this is parity with Oracle's 11.0.6 but the patch also
> contains some test improvements which will help stabilizing 11u testing. This
> mainly means increasing some test timeouts for a few tests that tend to
> timeout on slow or busy test hardware.
> >
> > The patch mostly applies, apart from these two rejects:
> > make/RunTests.gmk:
> http://cr.openjdk.java.net/~clanger/webrevs/8212028.11u-dev.0/8212028-
> RunTests.gmk.rej
> > make/RunTestsPrebuilt.gmk:
> http://cr.openjdk.java.net/~clanger/webrevs/8212028.11u-dev.0/8212028-
> RunTestsPrebuilt.gmk.rej
> > So, when reviewing, please have the most thorough look at these places.
> >
> > I ran the fix through our nightly tests and didn't find regressions. I'm
> planning to push backport JDK-8213005 together with this change, as it fixes a
> regression for it.
> >
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8212028
> > Original Change: http://hg.openjdk.java.net/jdk/jdk/rev/28375a1de254
> > Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8212028.11u-
> dev.0/
> 
>  222 langtools_JTREG_MAX_MEM := 768m
>  223
>  224 langtools_JTREG_MAX_MEM := 768m
> 
> Lines 222 and 224 in RunTests.gmk are duplicates.
> 
> Otherwise seems fine.
> 
> Thanks,
> Severin