aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorAnuj Phogat <[email protected]>2020-01-31 09:31:29 -0800
committerAnuj Phogat <[email protected]>2020-01-31 18:14:54 -0800
commit00a84c170a0a495f21008a80557a1d2b8257ea56 (patch)
tree626f119e2aa3570729db02f3d652502414078262 /src
parentc0d8b373adebbf8396b16537bd4d633ab6659900 (diff)
intel/gen12+: Reserve 4KB of URB space per bank for Compute Engine
This patch is required to fix 11K+ vulkan CTS failures we were getting with way_size_per_bank of 4 (see next patch). Thanks to Sagar Ghuge and Jordan Justen for all the hard work of debugging and testing. Signed-off-by: Anuj Phogat <[email protected]> Reviewed-by: Jason Ekstrand <[email protected]> Reviewed-by: Sagar Ghuge<[email protected]>
Diffstat (limited to 'src')
-rw-r--r--src/intel/common/gen_urb_config.c20
1 files changed, 19 insertions, 1 deletions
diff --git a/src/intel/common/gen_urb_config.c b/src/intel/common/gen_urb_config.c
index 5a60ca1ef11..0b819f4c937 100644
--- a/src/intel/common/gen_urb_config.c
+++ b/src/intel/common/gen_urb_config.c
@@ -65,7 +65,25 @@ gen_get_urb_config(const struct gen_device_info *devinfo,
unsigned entries[4], unsigned start[4],
enum gen_urb_deref_block_size *deref_block_size)
{
- const unsigned urb_size_kB = gen_get_l3_config_urb_size(devinfo, l3_cfg);
+ unsigned urb_size_kB = gen_get_l3_config_urb_size(devinfo, l3_cfg);
+
+ /* RCU_MODE register for Gen12+ in BSpec says:
+ *
+ * "HW reserves 4KB of URB space per bank for Compute Engine out of the
+ * total storage available in L3. SW must consider that 4KB of storage
+ * per bank will be reduced from what is programmed for the URB space
+ * in L3 for Render Engine executed workloads.
+ *
+ * Example: When URB space programmed is 64KB (per bank) for Render
+ * Engine, the actual URB space available for operation is only 60KB
+ * (per bank). Similarly when URB space programmed is 128KB (per bank)
+ * for render engine, the actual URB space available for operation is
+ * only 124KB (per bank). More detailed descripton available in "L3
+ * Cache" section of the B-Spec."
+ */
+ if (devinfo->gen >= 12)
+ urb_size_kB -= 4 * devinfo->l3_banks;
+
const unsigned push_constant_kB =
(devinfo->gen >= 8 || (devinfo->is_haswell && devinfo->gt == 3)) ? 32 : 16;