Mesa (master): intel/genxml: Fix ksp for INTERFACE_DESCRIPTOR_DATA

Kristian Høgsberg krh at kemper.freedesktop.org
Wed Nov 30 06:04:25 UTC 2016


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

Author: Kristian H. Kristensen <hoegsberg at gmail.com>
Date:   Mon Nov 28 21:00:34 2016 -0800

intel/genxml: Fix ksp for INTERFACE_DESCRIPTOR_DATA

This one was split across two dwords as "Kernel Start Pointer" and
"Kernel Start Pointer High", which looks like it works when the driver
only accesses "Kernel Start Pointer". This breaks, of course, with BO
offsets > 4G.

Signed-off-by: Kristian H. Kristensen <hoegsberg at gmail.com>
Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>

---

 src/intel/genxml/gen8.xml | 3 +--
 src/intel/genxml/gen9.xml | 3 +--
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/intel/genxml/gen8.xml b/src/intel/genxml/gen8.xml
index 7df479c..ebaf73a 100644
--- a/src/intel/genxml/gen8.xml
+++ b/src/intel/genxml/gen8.xml
@@ -588,8 +588,7 @@
   </struct>
 
   <struct name="INTERFACE_DESCRIPTOR_DATA" length="8">
-    <field name="Kernel Start Pointer" start="6" end="31" type="offset"/>
-    <field name="Kernel Start Pointer High" start="32" end="47" type="offset"/>
+    <field name="Kernel Start Pointer" start="6" end="47" type="offset"/>
     <field name="Denorm Mode" start="83" end="83" type="uint">
       <value name="Ftz" value="0"/>
       <value name="SetByKernel" value="1"/>
diff --git a/src/intel/genxml/gen9.xml b/src/intel/genxml/gen9.xml
index ec85494..211dd53 100644
--- a/src/intel/genxml/gen9.xml
+++ b/src/intel/genxml/gen9.xml
@@ -598,8 +598,7 @@
   </struct>
 
   <struct name="INTERFACE_DESCRIPTOR_DATA" length="8">
-    <field name="Kernel Start Pointer" start="6" end="31" type="offset"/>
-    <field name="Kernel Start Pointer High" start="32" end="47" type="offset"/>
+    <field name="Kernel Start Pointer" start="6" end="47" type="offset"/>
     <field name="Denorm Mode" start="83" end="83" type="uint">
       <value name="Ftz" value="0"/>
       <value name="SetByKernel" value="1"/>




More information about the mesa-commit mailing list