intel/blorp: Inline get_vs_entry_size into emit_urb_config
authorJason Ekstrand <jason.ekstrand@intel.com>
Mon, 29 Aug 2016 15:51:10 +0000 (08:51 -0700)
committerJason Ekstrand <jason.ekstrand@intel.com>
Tue, 30 Aug 2016 16:24:50 +0000 (09:24 -0700)
Topi asked to have the prefix removed because there's nothing gen7 about
it.  However, now that everything is in a single file, there is no good
reason to have it split out into a helper function anyway.  Let's just put
the contents in emit_urb_config and call it a day.

Reviewed-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
src/intel/blorp/blorp_genX_exec.h

index f44076e129f550e04fbabfd0cb792167ba1b71fa..966e4b3da9ceaf83ce54e6d63a1af4dbe9557dab 100644 (file)
@@ -113,28 +113,6 @@ __gen_combine_address(struct blorp_batch *batch, void *location,
       _dw + 1; /* Array starts at dw[1] */               \
    })
 
-/* Once vertex fetcher has written full VUE entries with complete
- * header the space requirement is as follows per vertex (in bytes):
- *
- *     Header    Position    Program constants
- *   +--------+------------+-------------------+
- *   |   16   |     16     |      n x 16       |
- *   +--------+------------+-------------------+
- *
- * where 'n' stands for number of varying inputs expressed as vec4s.
- *
- * The URB size is in turn expressed in 64 bytes (512 bits).
- */
-static inline unsigned
-gen7_blorp_get_vs_entry_size(const struct blorp_params *params)
-{
-    const unsigned num_varyings =
-       params->wm_prog_data ? params->wm_prog_data->num_varying_inputs : 0;
-    const unsigned total_needed = 16 + 16 + num_varyings * 16;
-
-   return DIV_ROUND_UP(total_needed, 64);
-}
-
 /* 3DSTATE_URB
  * 3DSTATE_URB_VS
  * 3DSTATE_URB_HS
@@ -166,7 +144,24 @@ static void
 emit_urb_config(struct blorp_batch *batch,
                 const struct blorp_params *params)
 {
-   blorp_emit_urb_config(batch, gen7_blorp_get_vs_entry_size(params));
+   /* Once vertex fetcher has written full VUE entries with complete
+    * header the space requirement is as follows per vertex (in bytes):
+    *
+    *     Header    Position    Program constants
+    *   +--------+------------+-------------------+
+    *   |   16   |     16     |      n x 16       |
+    *   +--------+------------+-------------------+
+    *
+    * where 'n' stands for number of varying inputs expressed as vec4s.
+    */
+    const unsigned num_varyings =
+       params->wm_prog_data ? params->wm_prog_data->num_varying_inputs : 0;
+    const unsigned total_needed = 16 + 16 + num_varyings * 16;
+
+   /* The URB size is expressed in units of 64 bytes (512 bits) */
+   const unsigned vs_entry_size = DIV_ROUND_UP(total_needed, 64);
+
+   blorp_emit_urb_config(batch, vs_entry_size);
 }
 
 static void