On 14/02/2018 05:08, Prahalad Kumar Narayanan wrote:
The intent of this fix and many similar fixes to come is to stabilize our test
suite. So running OpenGL tests on un supported configurations can be avoided
for now. Post the stabilization, we can inspect OpenGL failures/ artifacts
where OpenGL isn't the default pipeline.
But for this purpose we can use ProblemList and postpone execution on
windows until it will be fixed. But this case as well as adding of
additional keyword has one limitation, the test will be excluded but it
could be useful to run it using default pipeline.
To summarize: if we plan to fix it later then we can use ProblemList(it
is possible to exclude the test per platform), if we did not plan to fix
it in near future then we can drop an addition pipeline and cover a
default one.
It does not mean that we should close related bugs, for example the bug
for swing where the LAF is passed as an additional option to jtreg:
https://bugs.openjdk.java.net/browse/JDK-8068292
We could do the same thing for such bugs in java2d for non default piplines.
I am sure that even if we will drop this option we will not lose much,
because only some small amount(~50) of tests use it, so to cover the
opengl pipeline by all tests it is necessary to run the jtreg using
-Dsun.java2d.opengl=true
Hence, we not only need a provision to configure our automated build and test
systems to exclude OpenGL tests but also need a provision to execute tests
whenever required in future. The keywords approach helps to address both. Hence
this proposed fix.
Kindly let me know your views.
Thank you once again for your time
Have a good day
Prahalad N.
-----Original Message-----
From: Sergey Bylokhov
Sent: Wednesday, February 14, 2018 3:51 AM
To: Phil Race; Prahalad Kumar Narayanan; 2d-dev
Subject: Re: [OpenJDK 2D-Dev] [11] RFR: [JDK-8196451]
sun/java2d/OpenGL/DrawBufImgOp.java times out
On 13/02/2018 13:44, Phil Race wrote:
In this case I can say that it was nothing to do with being slow .. it
is a driver bug.
Even if behavior depends from the driver does not mean that it is a bug in
driver, we could do something wrong as well(since we have a bunch of artifacts
in opengl pipeline on windows/linux). But if we do not want to invest time to
investigate the reason of falure of this test on unsupported config(win+opengl)
then I assume the same decision will be done for other possible failures as
well.
I suggest to drop "-Dsun.java2d.opengl" option from our tests instead of
excluding the tests by keyword, so the tests will cover default system pipeline(which is
supported), and it will cover opengl on mac as well.
If it will be necessary to run the tests using some specific pipeline it will
be possible to pass a parameter to jtreg to run all the tests using one
pipeline.
-phil
On 02/13/2018 12:43 PM, Sergey Bylokhov wrote:
Hi, Prahalad, Phil.
Maybe it is possible to add a log from jtr file to the bug report,
usually it contains a stack trace on timeout. I am not sure that it
is a good idea to update any tests without reproducing it, when we
know that at least on one system it could be reproduced. The reasons
maybe a slow virtual system and we need to update a timeout.
On 13/02/2018 10:15, Prahalad Kumar Narayanan wrote:
Hello Everyone
Good day to you.
Request your time in reviewing a minor change for a test bug
[JDK-8196451] sun/java2d/OpenGL/DrawBufImgOp.java times out
https://bugs.openjdk.java.net/browse/JDK-8196451
Observation:
. The bug reports a test case- DrawBufImgOp, failing with
Timeout error on Win7 and OpenGL backend.
. The issue wasn't reproducible despite multiple test runs on
Win7 and Win10 machines.
. Hence OpenGL driver issues on the host machine is suspected
to have resulted in this Timeout.
Code change:
. The code changes add a new keyword- "opengl" to TEST.ROOT and
the respective test case.
. Since D3D is the preferred back-end on Windows, this keyword
will be helpful to filter OpenGL tests from execution in future.
. Note: There are many bugs that are similar to the current one.
But it's hard to propose a common fix as some of the test cases
execute with multiple @run tags (one for every back-end).
Kindly review the change and suggest your views
Link: http://cr.openjdk.java.net/~pnarayanan/8196451/webrev.00/
Thank you for your time
Have a good day
Prahalad N.
--
Best regards, Sergey.
--
Best regards, Sergey.