[drm/mgag200] e44e907dd8: phoronix-test-suite.glmark2.800x600.score -64.9% regression

[Emil Velikov]

Hi all,

On Thu, 4 Jun 2020 at 08:11, kernel test robot <rong.a.chen at intel.com> wrote:
>
> Greeting,
>
> FYI, we noticed a -64.9% regression of phoronix-test-suite.glmark2.800x600.score due to commit:
>
On one hand, I'm really happy to see performance testing happening
although this report is missing various crucial pieces of information.

> commit: e44e907dd8f937313d35615d799d54162c56d173 ("[PATCH v3 05/15] drm/mgag200: Split MISC register update into PLL selection, SYNC and I/O")
> url: https://github.com/0day-ci/linux/commits/Thomas-Zimmermann/drm-mgag200-Convert-to-atomic-modesetting/20200515-163744
> base: git://anongit.freedesktop.org/drm/drm-tip drm-tip
>
> in testcase: phoronix-test-suite
> on test machine: 16 threads Intel(R) Xeon(R) CPU X5570 @ 2.93GHz with 48G memory
> with following parameters:
>
>         need_x: true
Replace "need_x" with the Xorg version as seen in `Xorg -version'.

>         test: glmark2-1.1.0
>         cpufreq_governor: performance
>         ucode: 0x1d
>
> test-description: The Phoronix Test Suite is the most comprehensive testing and benchmarking platform available that provides an extensible framework for which new tests can be easily added.
> test-url: http://www.phoronix-test-suite.com/
>
Please remove the test description and url. They don't add any value.

Mention which Mesa version is used as well as on what GPU. The output
of lspci and glxinfo will help here.

For this particular test - there is no Mesa/upstream driver for this
GPU, so I imagine one of the swrast drivers was used. Which one -
swrast (classic, softpipe, llvmpipe, swr) or kms_swrast.
The output of `LD_DEBUG=libs glxinfo  |& grep _dri.so` will help here.

> commit:
>   bef2303526 ("drm/mgag200: Move mode-setting code into separate helper function")
>   e44e907dd8 ("drm/mgag200: Split MISC register update into PLL selection, SYNC and I/O")
>

Actually the offending commit has a subtle change of behaviour - it
adds an extra MGAREG_MISC_RAMMAPEN.
That is not documented and I've failed to spot it during review.

Thomas - shall we revert that line in itself or at least add an inline
comment why it is needed?

>
>   100 +---------------------------------------------------------------------+
>    90 |-+    +      +   +.+      +    +     +    +      +   :               |
>       | :    :      :   : :      :    :     :    :      :   :               |
>    80 |-:    :      :   : :      :    :     :    :      :   :               |
>    70 |-::   :     ::   :  :     :   ::     ::   :     ::  :                |
>       |: :  : :    : : :   :    : :  : :   : :  : :    : : :                |
>    60 |:+:  : :    : : :   :    : :  : :   : :  : :    : : :                |
>    50 |:+:  : :    : : :   :    : :  : :   : :  : :    : : :                |
>    40 |:+ : : :   :  : :    :   : : :  :   :  : : :   :  : :                |
>       |:  : : :   :  : :    :   : : :  :   :  : : :   :  : :O      O O O  O |
>    30 |:+ : : :   :  : :    :   : : :  :   :  : : :   :  : :                |
>    20 |-+ ::   :  :   :     :  :   ::   : :   ::   :  : O :                 |
>       |    :   : :    :      : :   :    : :    :   : :    :                 |
>    10 |-+  :   : :    :      : :   :    : :    :   : :    :                 |
>     0 +---------------------------------------------------------------------+
>
>
>                    phoronix-test-suite.glmark2.1024x768.score
>
>   70 +----------------------------------------------------------------------+
>      | +    +      +   +..+     +    +      +    +     +    +.+             |
>   60 |-:    :      :   :  :     :    :      :    :     :    :               |
>      | :    :      :   :  :     :    :      :    :     :    :               |
>   50 |-::   :     ::   :  :     ::   :      :   ::     ::   :               |
>      |: :  : :    : : :    :   : :  : :    : :  : :   : :  :                |
>   40 |:+:  : :    : : :    :   : :  : :    : :  : :   : :  :                |
>      |: :  : :    : : :    :   : :  : :    : :  : :   : :  :         O      |
>   30 |:+ : : :   :  : :    :   :  : : :    : : :  :   :  : :O      O   O  O |
>      |:  : : :   :  : :    :   :  : : :    : : :  :   :  : :                |
>   20 |:+ : : :   :  : :    :   :  : : :    : : :  :   :  : :                |
>      |   ::   :  :   :      : :   ::   :  :   ::   : : O ::                 |
>   10 |-+  :   : :    :      : :    :   :  :   :    : :    :                 |
>      |    :   : :  O :    O : : O  :   :  : O :    : :    :                 |
>    0 +----------------------------------------------------------------------+
>
>
> [*] bisect-good sample
> [O] bisect-bad  sample
>
Hmm I must be going blind - there isn't even a single * in either of the graphs.
Or perhaps my eyesight is fine and the legend or the graphs need fixing.

HTH
-Emil