[Mesa-dev] [PATCH 16/18] intel/genxml: Fix ksp for INTERFACE_DESCRIPTOR_DATA

Kristian H. Kristensen hoegsberg at gmail.com
Tue Nov 29 20:48:16 UTC 2016


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>
---
 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 17234f6..6769077 100644
--- a/src/intel/genxml/gen8.xml
+++ b/src/intel/genxml/gen8.xml
@@ -595,8 +595,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 c9fb770..3b8ffeb 100644
--- a/src/intel/genxml/gen9.xml
+++ b/src/intel/genxml/gen9.xml
@@ -605,8 +605,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"/>
-- 
2.9.3



More information about the mesa-dev mailing list