Mesa (main): radeonsi: change rounding mode to round to even
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Mon Mar 7 11:10:11 UTC 2022
Module: Mesa
Branch: main
Commit: 9c49550163ffd0753c55117f7ac2e800b1691e1c
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=9c49550163ffd0753c55117f7ac2e800b1691e1c
Author: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer at amd.com>
Date: Fri Mar 4 11:23:20 2022 +0100
radeonsi: change rounding mode to round to even
Use ROUND_TO_EVEN instead of TRUNCATE; this matches what pal and radv do.
This fixes the spec at ext_framebuffer_multisample@turn-on-off tests.
Cc: mesa-stable
Reviewed-by: Marek Olšák <marek.olsak at amd.com>
Reviewed-by: Mihai Preda <mhpreda at gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/15240>
---
.../radeonsi/ci/navi10-piglit-quick-fail.csv | 4 -
.../drivers/radeonsi/ci/radeonsi-stoney-fails.txt | 4 -
.../radeonsi/ci/raven-piglit-quick-fail.csv | 4 -
.../ci/sienna_cichlid-piglit-quick-fail.csv | 4 -
.../drivers/radeonsi/ci/traces-radeonsi.yml | 102 ++++++++++-----------
.../radeonsi/ci/vega20-piglit-quick-fail.csv | 4 -
src/gallium/drivers/radeonsi/si_state_viewport.c | 2 +-
7 files changed, 52 insertions(+), 72 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/ci/navi10-piglit-quick-fail.csv b/src/gallium/drivers/radeonsi/ci/navi10-piglit-quick-fail.csv
index 960e56867de..2f1a873d416 100644
--- a/src/gallium/drivers/radeonsi/ci/navi10-piglit-quick-fail.csv
+++ b/src/gallium/drivers/radeonsi/ci/navi10-piglit-quick-fail.csv
@@ -109,10 +109,6 @@ spec at egl_chromium_sync_control@conformance,Fail
spec at egl_chromium_sync_control@conformance at eglGetSyncValuesCHROMIUM_msc_and_sbc_test,Fail
spec at egl_ext_protected_content@conformance,Fail
spec at ext_framebuffer_blit@fbo-blit-check-limits,Fail
-spec at ext_framebuffer_multisample@turn-on-off 2,Fail
-spec at ext_framebuffer_multisample@turn-on-off 4,Fail
-spec at ext_framebuffer_multisample@turn-on-off 6,Fail
-spec at ext_framebuffer_multisample@turn-on-off 8,Fail
spec at ext_image_dma_buf_import@ext_image_dma_buf_import-sample_uyvy,Fail
spec at ext_image_dma_buf_import@ext_image_dma_buf_import-sample_yuyv,Fail
spec at glsl-1.20@compiler at invalid-vec4-array-to-vec3-array-conversion.vert,Fail
diff --git a/src/gallium/drivers/radeonsi/ci/radeonsi-stoney-fails.txt b/src/gallium/drivers/radeonsi/ci/radeonsi-stoney-fails.txt
index 0d5d40a87a9..bd8ee34b45c 100644
--- a/src/gallium/drivers/radeonsi/ci/radeonsi-stoney-fails.txt
+++ b/src/gallium/drivers/radeonsi/ci/radeonsi-stoney-fails.txt
@@ -66,10 +66,6 @@ spec at egl_ext_protected_content@conformance,Fail
spec at egl_khr_surfaceless_context@viewport,Fail
spec at egl_mesa_configless_context@basic,Fail
spec at ext_framebuffer_blit@fbo-blit-check-limits,Fail
-spec at ext_framebuffer_multisample@turn-on-off 2,Fail
-spec at ext_framebuffer_multisample@turn-on-off 4,Fail
-spec at ext_framebuffer_multisample@turn-on-off 6,Fail
-spec at ext_framebuffer_multisample@turn-on-off 8,Fail
spec at ext_image_dma_buf_import@ext_image_dma_buf_import-sample_y216,Fail
spec at ext_image_dma_buf_import@ext_image_dma_buf_import-sample_yuyv,Fail
spec at ext_image_dma_buf_import@ext_image_dma_buf_import-sample_uyvy,Fail
diff --git a/src/gallium/drivers/radeonsi/ci/raven-piglit-quick-fail.csv b/src/gallium/drivers/radeonsi/ci/raven-piglit-quick-fail.csv
index 4cb33831ae3..c5a8aeb466a 100644
--- a/src/gallium/drivers/radeonsi/ci/raven-piglit-quick-fail.csv
+++ b/src/gallium/drivers/radeonsi/ci/raven-piglit-quick-fail.csv
@@ -195,10 +195,6 @@ spec at egl_chromium_sync_control@conformance,Fail
spec at egl_chromium_sync_control@conformance at eglGetSyncValuesCHROMIUM_msc_and_sbc_test,Fail
spec at egl_ext_protected_content@conformance,Fail
spec at ext_framebuffer_blit@fbo-blit-check-limits,Fail
-spec at ext_framebuffer_multisample@turn-on-off 2,Fail
-spec at ext_framebuffer_multisample@turn-on-off 4,Fail
-spec at ext_framebuffer_multisample@turn-on-off 6,Fail
-spec at ext_framebuffer_multisample@turn-on-off 8,Fail
spec at ext_image_dma_buf_import@ext_image_dma_buf_import-sample_uyvy,Fail
spec at ext_image_dma_buf_import@ext_image_dma_buf_import-sample_yuyv,Fail
spec at ext_texture_snorm@multisample-formats 2 gl_ext_texture_snorm,Fail
diff --git a/src/gallium/drivers/radeonsi/ci/sienna_cichlid-piglit-quick-fail.csv b/src/gallium/drivers/radeonsi/ci/sienna_cichlid-piglit-quick-fail.csv
index 0e670f49de0..f76031ad501 100644
--- a/src/gallium/drivers/radeonsi/ci/sienna_cichlid-piglit-quick-fail.csv
+++ b/src/gallium/drivers/radeonsi/ci/sienna_cichlid-piglit-quick-fail.csv
@@ -110,10 +110,6 @@ spec at egl_chromium_sync_control@conformance,Fail
spec at egl_chromium_sync_control@conformance at eglGetSyncValuesCHROMIUM_msc_and_sbc_test,Fail
spec at egl_ext_protected_content@conformance,Fail
spec at ext_framebuffer_blit@fbo-blit-check-limits,Fail
-spec at ext_framebuffer_multisample@turn-on-off 2,Fail
-spec at ext_framebuffer_multisample@turn-on-off 4,Fail
-spec at ext_framebuffer_multisample@turn-on-off 6,Fail
-spec at ext_framebuffer_multisample@turn-on-off 8,Fail
spec at ext_image_dma_buf_import@ext_image_dma_buf_import-sample_uyvy,Fail
spec at ext_image_dma_buf_import@ext_image_dma_buf_import-sample_yuyv,Fail
spec at glsl-1.20@compiler at invalid-vec4-array-to-vec3-array-conversion.vert,Fail
diff --git a/src/gallium/drivers/radeonsi/ci/traces-radeonsi.yml b/src/gallium/drivers/radeonsi/ci/traces-radeonsi.yml
index 39892161a3d..a3a3772e7bf 100644
--- a/src/gallium/drivers/radeonsi/ci/traces-radeonsi.yml
+++ b/src/gallium/drivers/radeonsi/ci/traces-radeonsi.yml
@@ -5,19 +5,19 @@ traces:
- path: glmark2/desktop:windows=4:effect=blur:blur-radius=5:passes=1:separable=true.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: d8c9bf6295525e65e318adeff02520e2
+ checksum: 740fa8f8e9a9d815cf160b1893370755
- path: glmark2/jellyfish.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: f68bf374e535ad4a43a08786b0d536d8
+ checksum: 5bc7d5c250b7d568313c4afd064082f6
- path: glxgears/glxgears-2.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: eb9b3d497be567f02a6e039fa32f2b13
+ checksum: ef3653f50d4853d3e9cb3244c799565a
- path: 0ad/0ad.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 8fb8cd54f1ff908952fe0b6dd9f28999
+ checksum: 1a089d8584a9e68e7ab08eada954741b
- path: pathfinder/demo.trace
expectations:
- device: gl-radeonsi-stoney
@@ -33,7 +33,7 @@ traces:
- path: gputest/furmark.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 84c499203944cdc59e70450c324bb8df
+ checksum: 4ceea12000bb5995b915228d2d4b49c7
- path: gputest/triangle.trace
expectations:
- device: gl-radeonsi-stoney
@@ -41,47 +41,47 @@ traces:
- path: humus/Portals.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: fc7d00efe380cacbd4e9ef9b231aea2f
+ checksum: 5b96333495b794691e4ed071ae92ff19
- path: glmark2/buffer:update-fraction=0.5:update-dispersion=0.9:columns=200:update-method=map:interleave=false.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 5af6e31cc78320cb3f9db483c7a426e0
+ checksum: 5db05161041946e8971f39f12bbd847c
- path: glmark2/buffer:update-fraction=0.5:update-dispersion=0.9:columns=200:update-method=map:interleave=true.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 6e9dc5a7dc5a9cbb2b396bfce88a2084
+ checksum: e2154c522fcdb4f43b31b31c17adda74
- path: glmark2/buffer:update-fraction=0.5:update-dispersion=0.9:columns=200:update-method=subdata:interleave=false.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 35e384f833f37411f15bf8ef80ca1914
+ checksum: 70298e48479147af2d848a5441fb5f47
- path: glmark2/bump:bump-render=height.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 526cf3805b9b64bb8edea1b7d86b9cae
+ checksum: f5129b06e401a5fefa18a9895b18deec
- path: glmark2/bump:bump-render=high-poly.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: e026d36eaa71ecd957b47c7e6a5a100b
+ checksum: 3fc1adf0caa289b3296a80c2c13834ca
- path: glmark2/bump:bump-render=normals.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 4508a1be8c33a63fbfa695b141edf48b
+ checksum: 596fd7a084d3a7a6b61b322593385f45
- path: glmark2/conditionals:vertex-steps=0:fragment-steps=0.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: d12ecac5894705295e4fa076d77a72ab
+ checksum: fb2eda378ace8ca8b81d73d20cbfbbf7
- path: glmark2/conditionals:vertex-steps=0:fragment-steps=5.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 8999ff7eda7d7cf25440b96ab0efd4ee
+ checksum: b8575de0e043f540b12f13209054d000
- path: glmark2/conditionals:vertex-steps=5:fragment-steps=0.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: e65fdae9fe7bbd95c5cc0fb0c3eb7bf4
+ checksum: f70625a1f9bd9d2c211e330032b86f85
- path: glmark2/desktop:windows=4:effect=shadow.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 2aff87605464dd3f61aefd4e1dc0bffd
+ checksum: 384015de55daf7dd406c9463576018e9
- path: glmark2/effect2d:kernel=0,1,0;1,-4,1;0,1,0;.trace
expectations:
- device: gl-radeonsi-stoney
@@ -93,87 +93,87 @@ traces:
- path: glmark2/function:fragment-steps=5:fragment-complexity=low.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 13149880306d2183703a5c327f4d750a
+ checksum: 9efd8bb5df15f9483a18a00f9650caa9
- path: glmark2/function:fragment-steps=5:fragment-complexity=medium.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: c6983ffb3a74e234f84e5d817f876f54
+ checksum: 9bdd506c0404cb11a7148cb08b429d1b
- path: glmark2/build:use-vbo=false.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 203a0205580b2c39ed8dcbed57b18f3c
+ checksum: 506b1910317b04e5d32aacf2bd70bd0d
- path: glmark2/build:use-vbo=true.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 437034f264c469757683e51e3e25beca
+ checksum: 793dc29115ae442b279276adb89d0999
- path: glmark2/ideas:speed=10000.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 381d973b00b46fcc15f72808eabb6237
+ checksum: 1ae057093620f868aad846167f04c6e0
- path: glmark2/loop:vertex-steps=5:fragment-steps=5:fragment-loop=false.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 5a32f7917c130581fae23e58b71fd740
+ checksum: 6fb2f9bce414879e3751bb51d1a8d481
- path: glmark2/loop:vertex-steps=5:fragment-steps=5:fragment-uniform=false.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 20586c936a7051ce63503df6f9785d01
+ checksum: 27fabda45ca2a989c21b4ec386a2e8f6
- path: glmark2/loop:vertex-steps=5:fragment-steps=5:fragment-uniform=true.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 255e412701afdc4a7b62654e93b92cc9
+ checksum: 05ac8be6e2e0c03ea1caec85f037cddd
- path: glmark2/pulsar:quads=5:texture=false:light=false.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 42f913c6119a685da4450ea116060614
+ checksum: 0b62b9c04e4c00f44eba64b366c47783
- path: glmark2/refract.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 9d0a2d2fce0b80a265fbcee5107c9e82
+ checksum: c711f3a07f6aa9e0f19c544c6d7c2000
- path: glmark2/shading:shading=blinn-phong-inf.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 4a2cf8a13b248f470e58f785d0a9207d
+ checksum: 429c6bbdf99d573cc4eaaee3c0471257
- path: glmark2/shading:shading=cel.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 8325ce4073135c03eec241087e51a584
+ checksum: 330d9a8375970e42ba5ddc3142dc6477
- path: glmark2/shading:shading=gouraud.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: fbe5b7d038866f6cd4fc801b062e4ce5
+ checksum: e22908309d41af8c9753c5c7cae73b29
- path: glmark2/shading:shading=phong.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 65f9468a37d683b4c1f38d34f09a97db
+ checksum: 2ac9f1b6ba39f8924b374d18181edeeb
- path: glmark2/shadow.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: abd705b0ae76cf6f19905bfea1d3db76
+ checksum: 9215a1525dfe5b12999652b3a3ba05d8
- path: glmark2/terrain.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 59690f1438a1e44fc655d16ce8bb348b
+ checksum: 80a1bba6ff969c9a82c68de0306f2b61
- path: glmark2/texture:texture-filter=linear.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 54bf32b499f3ebfe0e727e5716e54b1a
+ checksum: 928479421abda4823a673393cd59ff81
- path: glmark2/texture:texture-filter=mipmap.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 3a3abce164eef2be10f58604b22583f2
+ checksum: cb94bca58ed8f41c5f6f6dda3fb15600
- path: glmark2/texture:texture-filter=nearest.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 311c394eb4afd48bd8330d0e69470b87
+ checksum: 63762536d93048c0dadb19b22381cb90
- path: godot/Material Testers.x86_64_2020.04.08_13.38_frame799.rdc
expectations:
- device: gl-radeonsi-stoney
- checksum: 02f654ad77c0c1106e1b31e1c86c93bb
+ checksum: 0e7441a7f2c86c75eeac72269786e7a1
- path: gputest/gimark.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 4442dbd44a9704c499da4817fffce306
+ checksum: 3e021c1c5f2900e53c3375e0366eeda9
- path: gputest/pixmark-julia-fp32.trace
expectations:
- device: gl-radeonsi-stoney
@@ -189,56 +189,56 @@ traces:
- path: gputest/plot3d.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 667078b0f51ac8e0469ef9a20326c616
+ checksum: 2832d2190df232bcbde71a4b720303e1
- path: gputest/tessmark.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: cdcf984e81f2cda9ca7f56e3a4addaa7
+ checksum: b4bc7998ff804bf25aac9f9609b8acbf
- path: humus/AmbientAperture.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 664ea58a62b27737b7d0ae9e86ab85c0
+ checksum: 679dc88e5f0d3b3608e0490b4b39fbea
- path: humus/CelShading.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 7d30dbe3470873c327464ea7ebe6bcf0
+ checksum: ebd3f460386b9461aab6adef04dc0547
- path: humus/DynamicBranching3.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 5a5addfb1c7acc22104c0ecb29f45174
+ checksum: 9f932808ccce10a2bf7568ddafe1a1cf
- path: humus/HDR.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: b44ce4cb377321f6423a5a62a9083794
+ checksum: fddf642f7a4e3946f1975e33cf773220
- path: humus/RaytracedShadows.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 2e2e58e0b32329d6cf90ab0ec71108fd
+ checksum: 743ce9c218b87404cfe615ed7951d5b8
- path: paraview/pv-manyspheres.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 7e405460c58557573bd9cc54035bb62a
+ checksum: 7728082892aa81ee74484b1ed239db74
- path: paraview/pv-waveletcontour.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 961fb30a1a3756678662d1a9a45b1400
+ checksum: c49b01d08b9716cd3408173d2dd1c9ef
- path: paraview/pv-waveletvolume.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: f1093f472683a16dec0d5da4b599356f
+ checksum: 491d0572ab0bcbcbc9827521597b3787
- path: humus/VolumetricFogging2.trace
expectations:
- device: gl-radeonsi-stoney
- checksum: 1a9b3aa90d2c4a0d3937ecacca2da052
+ checksum: e3eb8d6934c23cbef7133ba501821698
- path: supertuxkart/supertuxkart-antediluvian-abyss.rdc
expectations:
- device: gl-radeonsi-stoney
- checksum: fd717f0214c2e5d08538d2de30bfe180
+ checksum: 112f203971d0f1ae46bc8268683480a2
- path: supertuxkart/supertuxkart-menu.rdc
expectations:
- device: gl-radeonsi-stoney
- checksum: bc851bb8808b311c2df9c36a6c289e6e
+ checksum: e17f3125683d8b2acac8bc0b4b8e1b44
- path: supertuxkart/supertuxkart-ravenbridge-mansion.rdc
expectations:
- device: gl-radeonsi-stoney
- checksum: f2e730f7378af65abbf82cc30e4ba64d
+ checksum: 60f74020451e9beaf586b4551541b763
diff --git a/src/gallium/drivers/radeonsi/ci/vega20-piglit-quick-fail.csv b/src/gallium/drivers/radeonsi/ci/vega20-piglit-quick-fail.csv
index 77c5e9d53d2..68ae8978516 100644
--- a/src/gallium/drivers/radeonsi/ci/vega20-piglit-quick-fail.csv
+++ b/src/gallium/drivers/radeonsi/ci/vega20-piglit-quick-fail.csv
@@ -97,10 +97,6 @@ spec at arb_sparse_buffer@buffer-data,Fail
spec at arb_sparse_buffer@commit,Fail
spec at egl_ext_protected_content@conformance,Fail
spec at ext_framebuffer_blit@fbo-blit-check-limits,Fail
-spec at ext_framebuffer_multisample@turn-on-off 2,Fail
-spec at ext_framebuffer_multisample@turn-on-off 4,Fail
-spec at ext_framebuffer_multisample@turn-on-off 6,Fail
-spec at ext_framebuffer_multisample@turn-on-off 8,Fail
spec at ext_image_dma_buf_import@ext_image_dma_buf_import-sample_uyvy,Fail
spec at ext_image_dma_buf_import@ext_image_dma_buf_import-sample_y210,Fail
spec at ext_image_dma_buf_import@ext_image_dma_buf_import-sample_y212,Fail
diff --git a/src/gallium/drivers/radeonsi/si_state_viewport.c b/src/gallium/drivers/radeonsi/si_state_viewport.c
index 8894732a7a0..d1ee6fd3a2c 100644
--- a/src/gallium/drivers/radeonsi/si_state_viewport.c
+++ b/src/gallium/drivers/radeonsi/si_state_viewport.c
@@ -387,7 +387,7 @@ static void si_emit_guardband(struct si_context *ctx)
S_028234_HW_SCREEN_OFFSET_Y(hw_screen_offset_y >> 4));
radeon_opt_set_context_reg(
ctx, R_028BE4_PA_SU_VTX_CNTL, SI_TRACKED_PA_SU_VTX_CNTL,
- S_028BE4_PIX_CENTER(rs->half_pixel_center) |
+ S_028BE4_PIX_CENTER(rs->half_pixel_center) | S_028BE4_ROUND_MODE(V_028BE4_X_ROUND_TO_EVEN) |
S_028BE4_QUANT_MODE(V_028BE4_X_16_8_FIXED_POINT_1_256TH + vp_as_scissor.quant_mode));
radeon_end_update_context_roll(ctx);
}
More information about the mesa-commit
mailing list