Mesa (master): i965: return PASSTHRU surface types also on gen7

Juan Antonio Suárez Romero jasuarez at kemper.freedesktop.org
Thu Jan 12 12:04:20 UTC 2017


Module: Mesa
Branch: master
Commit: 717f99b34a329d21d7b65e3193c6bf2162ad7964
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=717f99b34a329d21d7b65e3193c6bf2162ad7964

Author: Alejandro Piñeiro <apinheiro at igalia.com>
Date:   Fri Apr 15 19:08:04 2016 +0200

i965: return PASSTHRU surface types also on gen7

Although gen7 doesn't include surface types as a valid conversion format,
we return it, as it reflects what we want to achieve, even if we need
to workaround it on gen < 8.

Reviewed-by: Jordan Justen <jordan.l.justen at intel.com>

---

 src/mesa/drivers/dri/i965/brw_draw_upload.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_draw_upload.c b/src/mesa/drivers/dri/i965/brw_draw_upload.c
index b7527f2..79eb634 100644
--- a/src/mesa/drivers/dri/i965/brw_draw_upload.c
+++ b/src/mesa/drivers/dri/i965/brw_draw_upload.c
@@ -231,9 +231,13 @@ double_types(struct brw_context *brw,
     * 64-bit components are stored in the URB without any conversion."
     * Also included on BDW PRM, Volume 7, page 470, table "Source Element
     * Formats Supported in VF Unit"
-    * Previous PRMs don't include those references.
+    *
+    * Previous PRMs don't include those references, so for gen7 we can't use
+    * PASSTHRU formats directly. But in any case, we prefer to return passthru
+    * even in that case, because that reflects what we want to achieve, even
+    * if we would need to workaround on gen < 8.
     */
-   return (brw->gen >= 8 && doubles
+   return (doubles
            ? double_types_passthru[size]
            : double_types_float[size]);
 }




More information about the mesa-commit mailing list