RFR: 8339508: RenderPerf Test Application

Kevin Rushforth kcr at openjdk.org
Mon Sep 16 23:40:09 UTC 2024


On Mon, 16 Sep 2024 09:12:57 GMT, Lukasz Kostyra <lkostyra at openjdk.org> wrote:

>> This PR migrates RenderPerf performance test application from jfx-sandbox metal branch: https://github.com/openjdk/jfx-sandbox/tree/metal/tests/performance/animation/RenderPerfTest/src/renderperf
>> 
>> RenderPerf is a performance test application which draws provided number of "particles" (JFX nodes) on the screen and animates them. After some time (default 10 seconds of testing + warmup) the test automatically closes and reports FPS values. Test was developed as part of Metal backend for performance testing and we decided to integrate it with mainline for any potential further development and fixes.
>> 
>> Command line options for the test:
>> - `-n` - number of particles to draw, defaults to 1000
>> - `-t` - name of test to run, defaults to running all tests in random order. Can provide multiple tests, ex. `-t Rectangle Image`
>> - `-d` - test duration in seconds, defaults to 10. Can be set to 0, which means the test will run infinitely long and end only when user manually closes the stage, mostly useful for stability testing.
>> 
>> Some tests use `duke.png` image provided with this PR. This is an openly available Duke Wave image taken from https://wiki.openjdk.org/display/duke/Gallery and rescaled for test purposes.
>> 
>> PR consists of two commits:
>> - Base version of RenderPerfTest developed by @karthikpandelu 
>> - My commit adding `-d` flag and implementing plenty of other fixes as part of [JDK-8331570](https://bugs.openjdk.org/browse/JDK-8331570)
>
> Since this PR both migrates the base version (JDK-8339508) and adds test duration flag (JDK-8331570) I will link the latter issue as well.

@lukostyra Can you do a `/contributor add @karthikpandelu`?

Also, can you change the fix version of the secondary bug to tbd to address the following warning:

> [JDK-8331570](https://bugs.openjdk.org/browse/JDK-8331570): RenderPerf test: Add infinite test mode for stress-testing scenarios (Bug - P4)(⚠️ The fixVersion in this issue is [internal] but the fixVersion in .jcheck/conf is jfx24, a new backport will be created when this pr is integrated.)

Reviewers: @arapte @jayathirthrao

-------------

PR Comment: https://git.openjdk.org/jfx/pull/1568#issuecomment-2354214063


More information about the openjfx-dev mailing list