RFR(S) 8197453 : Add support of extra problem list

Igor Ignatyev igor.ignatyev at oracle.com
Tue Feb 27 22:57:40 UTC 2018


Hi Katya,

the fix looks good to me.

-- Igor

> On Feb 27, 2018, at 9:45 AM, Ekaterina Pavlova <ekaterina.pavlova at oracle.com> wrote:
> 
> Jon,
> 
> thanks for the review.
> I have updated the webrev.
> 
> thanks,
> -katya
> 
> 
> On 2/26/18 12:02 PM, Jonathan Gibbons wrote:
>> If these new problem-list files are destined for use by jtreg, I would encourage adding a platform specifier on each line, after the bug number. If you want to mark the test as excluded on all platforms, the convention is to use "generic-all".
>> -- Jon
>> On 2/26/18 11:47 AM, Igor Ignatyev wrote:
>>> adding build-dev alias
>>> 
>>> -- Igor
>>> 
>>>> On Feb 8, 2018, at 3:08 PM, Ekaterina Pavlova <ekaterina.pavlova at oracle.com> wrote:
>>>> 
>>>> Hi all,
>>>> 
>>>> ProblemList.txt files used by makefiles for jtreg testing allow to specify list of tests to be excluded
>>>> from execution on all or specific platforms. However to test such features like Graal we want to be able
>>>> to specify list of failed tests which fail in particular JVM mode only.
>>>> Please review this change which adds support of extra problem list and introduces 2 Graal specific problem list files.
>>>> - test/hotspot/jtreg/ProblemList-graal.txt
>>>> - test/jdk/ProblemList-graal.txt
>>>> 
>>>> 
>>>>      JBS: https://bugs.openjdk.java.net/browse/JDK-8197453
>>>>   webrev: http://cr.openjdk.java.net/~epavlova//8197453/webrev.00/
>>>> testing: precheckin, tier1 and tier2 with empty EXTRA_PROBLEM_LISTS.
>>>>           testing in Graal mode with EXTRA_PROBLEM_LISTS=ProblemList-graal.txt
>>>> 
>>>> thanks,
>>>> -katya
>>>> 
>>>> p.s.
>>>> Igor Ignatyev volunteered to sponsor this change.
> 



More information about the build-dev mailing list