isl: Add a function for filling out a surface state
authorJason Ekstrand <jason.ekstrand@intel.com>
Sat, 20 Feb 2016 17:32:43 +0000 (09:32 -0800)
committerJason Ekstrand <jason.ekstrand@intel.com>
Sat, 27 Feb 2016 18:26:13 +0000 (10:26 -0800)
src/intel/isl/Makefile.am
src/intel/isl/isl.c
src/intel/isl/isl.h
src/intel/isl/isl_priv.h
src/intel/isl/isl_surface_state.c [new file with mode: 0644]

index 74e8ab04be5213e8221ee2b3f91f065995444245..b4af3c86f3d2183dba4c2d3e0ff9a42c487b3a70 100644 (file)
@@ -24,6 +24,7 @@ SUBDIRS = .
 
 ISL_GEN_LIBS =                                           \
        libisl-gen7.la                                   \
+       libisl-gen75.la                                  \
        libisl-gen8.la                                   \
        libisl-gen9.la                                   \
        $(NULL)
@@ -39,13 +40,15 @@ AM_CPPFLAGS = \
        $(DEFINES) \
        -I$(top_srcdir)/include \
        -I$(top_srcdir)/src \
+       -I$(top_srcdir)/src/intel \
        -I$(top_srcdir)/src/mapi \
        -I$(top_srcdir)/src/mesa \
        -I$(top_srcdir)/src/mesa/drivers/dri/common \
        -I$(top_srcdir)/src/mesa/drivers/dri/i965 \
        -I$(top_srcdir)/src/gallium/auxiliary \
        -I$(top_srcdir)/src/gallium/include \
-       -I$(top_builddir)/src
+       -I$(top_builddir)/src \
+       -I$(top_builddir)/src/intel
 
 libisl_la_CFLAGS = $(CFLAGS) -Wno-override-init
 
@@ -66,20 +69,28 @@ libisl_la_SOURCES =                                     \
 libisl_gen7_la_SOURCES =                                \
        isl_gen7.c                                      \
        isl_gen7.h                                      \
+        isl_surface_state.c                             \
        $(NULL)
-libisl_gen7_la_CFLAGS = $(libisl_la_CFLAGS)
+libisl_gen7_la_CFLAGS = $(libisl_la_CFLAGS) -DGEN_VERSIONx10=70
+
+libisl_gen75_la_SOURCES =                               \
+        isl_surface_state.c                             \
+       $(NULL)
+libisl_gen75_la_CFLAGS = $(libisl_la_CFLAGS) -DGEN_VERSIONx10=75
 
 libisl_gen8_la_SOURCES =                                \
        isl_gen8.c                                      \
        isl_gen8.h                                      \
+        isl_surface_state.c                             \
        $(NULL)
-libisl_gen8_la_CFLAGS = $(libisl_la_CFLAGS)
+libisl_gen8_la_CFLAGS = $(libisl_la_CFLAGS) -DGEN_VERSIONx10=80
 
 libisl_gen9_la_SOURCES =                                \
        isl_gen9.c                                      \
        isl_gen9.h                                      \
+        isl_surface_state.c                             \
        $(NULL)
-libisl_gen9_la_CFLAGS = $(libisl_la_CFLAGS)
+libisl_gen9_la_CFLAGS = $(libisl_la_CFLAGS) -DGEN_VERSIONx10=90
 
 BUILT_SOURCES =                                         \
        isl_format_layout.c
index 3b6bee10081ddceaa6f0d576604675b2af9aa743..f7f276f16df64f9eb23ebba29f60009222e25d32 100644 (file)
@@ -1161,6 +1161,48 @@ isl_surf_get_tile_info(const struct isl_device *dev,
    isl_tiling_get_info(dev, surf->tiling, fmtl->bs, tile_info);
 }
 
+void
+isl_surf_fill_state_s(const struct isl_device *dev, void *state,
+                      const struct isl_surf_fill_state_info *restrict info)
+{
+#ifndef NDEBUG
+   isl_surf_usage_flags_t _base_usage =
+      info->view->usage & (ISL_SURF_USAGE_RENDER_TARGET_BIT |
+                           ISL_SURF_USAGE_TEXTURE_BIT |
+                           ISL_SURF_USAGE_STORAGE_BIT);
+   /* They may only specify one of the above bits at a time */
+   assert(__builtin_popcount(_base_usage) == 1);
+   /* The only other allowed bit is ISL_SURF_USAGE_CUBE_BIT */
+   assert((info->view->usage & ~ISL_SURF_USAGE_CUBE_BIT) == _base_usage);
+#endif
+
+   if (info->surf->dim == ISL_SURF_DIM_3D) {
+      assert(info->view->base_array_layer + info->view->array_len <=
+             info->surf->logical_level0_px.depth);
+   } else {
+      assert(info->view->base_array_layer + info->view->array_len <=
+             info->surf->logical_level0_px.array_len);
+   }
+
+   switch (ISL_DEV_GEN(dev)) {
+   case 7:
+      if (ISL_DEV_IS_HASWELL(dev)) {
+         isl_gen75_surf_fill_state_s(dev, state, info);
+      } else {
+         isl_gen7_surf_fill_state_s(dev, state, info);
+      }
+      break;
+   case 8:
+      isl_gen8_surf_fill_state_s(dev, state, info);
+      break;
+   case 9:
+      isl_gen9_surf_fill_state_s(dev, state, info);
+      break;
+   default:
+      assert(!"Cannot fill surface state for this gen");
+   }
+}
+
 /**
  * A variant of isl_surf_get_image_offset_sa() specific to
  * ISL_DIM_LAYOUT_GEN4_2D.
index 3e0ff935948359362d2001b992de1cbefe3c8699..dbd480c51cebde31807d010aac7394ad557240fc 100644 (file)
@@ -60,6 +60,16 @@ struct brw_image_param;
 #define ISL_DEV_GEN(__dev) ((__dev)->info->gen)
 #endif
 
+#ifndef ISL_DEV_IS_HASWELL
+/**
+ * @brief Get the hardware generation of isl_device.
+ *
+ * You can define this as a compile-time constant in the CFLAGS. For example,
+ * `gcc -DISL_DEV_GEN(dev)=9 ...`.
+ */
+#define ISL_DEV_IS_HASWELL(__dev) ((__dev)->info->is_haswell)
+#endif
+
 #ifndef ISL_DEV_USE_SEPARATE_STENCIL
 /**
  * You can define this as a compile-time constant in the CFLAGS. For example,
@@ -455,8 +465,21 @@ typedef uint64_t isl_surf_usage_flags_t;
 #define ISL_SURF_USAGE_DISPLAY_ROTATE_270_BIT  (1u << 9)
 #define ISL_SURF_USAGE_DISPLAY_FLIP_X_BIT      (1u << 10)
 #define ISL_SURF_USAGE_DISPLAY_FLIP_Y_BIT      (1u << 11)
+#define ISL_SURF_USAGE_STORAGE_BIT             (1u << 12)
 /** @} */
 
+/**
+ * @brief A channel select (also known as texture swizzle) value
+ */
+enum isl_channel_select {
+   ISL_CHANNEL_SELECT_ZERO = 0,
+   ISL_CHANNEL_SELECT_ONE = 1,
+   ISL_CHANNEL_SELECT_RED = 4,
+   ISL_CHANNEL_SELECT_GREEN = 5,
+   ISL_CHANNEL_SELECT_BLUE = 6,
+   ISL_CHANNEL_SELECT_ALPHA = 7,
+};
+
 /**
  * Identical to VkSampleCountFlagBits.
  */
@@ -695,6 +718,77 @@ struct isl_surf {
    isl_surf_usage_flags_t usage;
 };
 
+struct isl_view {
+   /**
+    * Indicates the usage of the particular view
+    *
+    * Normally, this is one bit.  However, for a cube map texture, it
+    * should be ISL_SURF_USAGE_TEXTURE_BIT | ISL_SURF_USAGE_CUBE_BIT.
+    */
+   isl_surf_usage_flags_t usage;
+
+   /**
+    * The format to use in the view
+    *
+    * This may differ from the format of the actual isl_surf but must have
+    * the same block size.
+    */
+   enum isl_format format;
+
+   uint32_t base_level;
+   uint32_t levels;
+
+   /**
+    * Base array layer
+    *
+    * For cube maps, both base_array_layer and array_len should be
+    * specified in terms of 2-D layers and must be a multiple of 6.
+    */
+   uint32_t base_array_layer;
+   uint32_t array_len;
+
+   enum isl_channel_select channel_select[4];
+};
+
+union isl_color_value {
+   float f32[4];
+   uint32_t u32[4];
+   int32_t i32[4];
+};
+
+struct isl_surf_fill_state_info {
+   const struct isl_surf *surf;
+   const struct isl_view *view;
+
+   /**
+    * The address of the surface in GPU memory.
+    */
+   uint64_t address;
+
+   /**
+    * The Memory Object Control state for the filled surface state.
+    *
+    * The exact format of this value depends on hardware generation.
+    */
+   uint32_t mocs;
+
+   /**
+    * This allows the caller to over-ride the dimensions of the surface.
+    * This is used at the moment for compressed surfaces to let us hack
+    * around the fact that we can't actually render to them.
+    *
+    * FIXME: We really need to get rid of this.  It's a lie.
+    */
+   struct isl_extent4d level0_extent_px;
+
+   /**
+    * The clear color for this surface
+    *
+    * Valid values depend on hardware generation.
+    */
+   union isl_color_value clear_color;
+};
+
 extern const struct isl_format_layout isl_format_layouts[];
 
 void
@@ -889,6 +983,14 @@ isl_surf_get_tile_info(const struct isl_device *dev,
                        const struct isl_surf *surf,
                        struct isl_tile_info *tile_info);
 
+#define isl_surf_fill_state(dev, state, ...) \
+   isl_surf_fill_state_s((dev), (state), \
+                         &(struct isl_surf_fill_state_info) {  __VA_ARGS__ });
+
+void
+isl_surf_fill_state_s(const struct isl_device *dev, void *state,
+                      const struct isl_surf_fill_state_info *restrict info);
+
 /**
  * Alignment of the upper-left sample of each subimage, in units of surface
  * elements.
index bca8503b4fd8dce420b5a33c6c248b2c1186e3c9..d3f08094df9b048abc7992fe9f4394752d04b651 100644 (file)
@@ -138,3 +138,17 @@ isl_extent3d_el_to_sa(enum isl_format fmt, struct isl_extent3d extent_el)
       .d = extent_el.d * fmtl->bd,
    };
 }
+
+void
+isl_gen7_surf_fill_state_s(const struct isl_device *dev, void *state,
+                           const struct isl_surf_fill_state_info *restrict info);
+
+void
+isl_gen75_surf_fill_state_s(const struct isl_device *dev, void *state,
+                            const struct isl_surf_fill_state_info *restrict info);
+void
+isl_gen8_surf_fill_state_s(const struct isl_device *dev, void *state,
+                           const struct isl_surf_fill_state_info *restrict info);
+void
+isl_gen9_surf_fill_state_s(const struct isl_device *dev, void *state,
+                           const struct isl_surf_fill_state_info *restrict info);
diff --git a/src/intel/isl/isl_surface_state.c b/src/intel/isl/isl_surface_state.c
new file mode 100644 (file)
index 0000000..fe3c083
--- /dev/null
@@ -0,0 +1,394 @@
+/*
+ * Copyright 2016 Intel Corporation
+ *
+ *  Permission is hereby granted, free of charge, to any person obtaining a
+ *  copy of this software and associated documentation files (the "Software"),
+ *  to deal in the Software without restriction, including without limitation
+ *  the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ *  and/or sell copies of the Software, and to permit persons to whom the
+ *  Software is furnished to do so, subject to the following conditions:
+ *
+ *  The above copyright notice and this permission notice (including the next
+ *  paragraph) shall be included in all copies or substantial portions of the
+ *  Software.
+ *
+ *  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ *  IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ *  FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ *  THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ *  LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ *  FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ *  IN THE SOFTWARE.
+ */
+
+#include <stdint.h>
+
+#define __gen_address_type uint64_t
+#define __gen_user_data void
+
+static inline uint64_t
+__gen_combine_address(void *data, void *loc, uint64_t addr, uint32_t delta)
+{
+   return addr + delta;
+}
+
+#include "genxml/gen_macros.h"
+#include "genxml/genX_pack.h"
+
+#include "isl_priv.h"
+
+#define __PASTE2(x, y) x ## y
+#define __PASTE(x, y) __PASTE2(x, y)
+#define isl_genX(x) __PASTE(isl_, genX(x))
+
+#if GEN_GEN >= 8
+static const uint8_t isl_to_gen_halign[] = {
+    [4] = HALIGN4,
+    [8] = HALIGN8,
+    [16] = HALIGN16,
+};
+
+static const uint8_t isl_to_gen_valign[] = {
+    [4] = VALIGN4,
+    [8] = VALIGN8,
+    [16] = VALIGN16,
+};
+#else
+static const uint8_t isl_to_gen_halign[] = {
+    [4] = HALIGN_4,
+    [8] = HALIGN_8,
+};
+
+static const uint8_t isl_to_gen_valign[] = {
+    [2] = VALIGN_2,
+    [4] = VALIGN_4,
+};
+#endif
+
+#if GEN_GEN >= 8
+static const uint8_t isl_to_gen_tiling[] = {
+   [ISL_TILING_LINEAR]  = LINEAR,
+   [ISL_TILING_X]       = XMAJOR,
+   [ISL_TILING_Y0]      = YMAJOR,
+   [ISL_TILING_Yf]      = YMAJOR,
+   [ISL_TILING_Ys]      = YMAJOR,
+   [ISL_TILING_W]       = WMAJOR,
+};
+#endif
+
+#if GEN_GEN >= 8
+static const uint32_t isl_to_gen_multisample_layout[] = {
+   [ISL_MSAA_LAYOUT_NONE]           = MSS,
+   [ISL_MSAA_LAYOUT_INTERLEAVED]    = DEPTH_STENCIL,
+   [ISL_MSAA_LAYOUT_ARRAY]          = MSS,
+};
+#else
+static const uint32_t isl_to_gen_multisample_layout[] = {
+   [ISL_MSAA_LAYOUT_NONE]           = MSFMT_MSS,
+   [ISL_MSAA_LAYOUT_INTERLEAVED]    = MSFMT_DEPTH_STENCIL,
+   [ISL_MSAA_LAYOUT_ARRAY]          = MSFMT_MSS,
+};
+#endif
+
+static const uint8_t
+get_surftype(enum isl_surf_dim dim, isl_surf_usage_flags_t usage)
+{
+   switch (dim) {
+   default:
+      unreachable("bad isl_surf_dim");
+   case ISL_SURF_DIM_1D:
+      assert(!(usage & ISL_SURF_USAGE_CUBE_BIT));
+      return SURFTYPE_1D;
+   case ISL_SURF_DIM_2D:
+      if (usage & ISL_SURF_USAGE_STORAGE_BIT) {
+         /* Storage images are always plain 2-D, not cube */
+         return SURFTYPE_2D;
+      } else if (usage & ISL_SURF_USAGE_CUBE_BIT) {
+         return SURFTYPE_CUBE;
+      } else {
+         return SURFTYPE_2D;
+      }
+   case ISL_SURF_DIM_3D:
+      assert(!(usage & ISL_SURF_USAGE_CUBE_BIT));
+      return SURFTYPE_3D;
+   }
+}
+
+/**
+ * Get the values to pack into RENDER_SUFFACE_STATE.SurfaceHorizontalAlignment
+ * and SurfaceVerticalAlignment.
+ */
+static void
+get_halign_valign(const struct isl_surf *surf,
+                  uint32_t *halign, uint32_t *valign)
+{
+   if (GEN_GEN >= 9) {
+      if (isl_tiling_is_std_y(surf->tiling) ||
+          surf->dim_layout == ISL_DIM_LAYOUT_GEN9_1D) {
+         /* The hardware ignores the alignment values. Anyway, the surface's
+          * true alignment is likely outside the enum range of HALIGN* and
+          * VALIGN*.
+          */
+         *halign = 0;
+         *valign = 0;
+      } else {
+         /* In Skylake, RENDER_SUFFACE_STATE.SurfaceVerticalAlignment is in units
+          * of surface elements (not pixels nor samples). For compressed formats,
+          * a "surface element" is defined as a compression block.  For example,
+          * if SurfaceVerticalAlignment is VALIGN_4 and SurfaceFormat is an ETC2
+          * format (ETC2 has a block height of 4), then the vertical alignment is
+          * 4 compression blocks or, equivalently, 16 pixels.
+          */
+         struct isl_extent3d image_align_el
+            = isl_surf_get_image_alignment_el(surf);
+
+         *halign = isl_to_gen_halign[image_align_el.width];
+         *valign = isl_to_gen_valign[image_align_el.height];
+      }
+   } else {
+      /* Pre-Skylake, RENDER_SUFFACE_STATE.SurfaceVerticalAlignment is in
+       * units of surface samples.  For example, if SurfaceVerticalAlignment
+       * is VALIGN_4 and the surface is singlesampled, then for any surface
+       * format (compressed or not) the vertical alignment is
+       * 4 pixels.
+       */
+      struct isl_extent3d image_align_sa
+         = isl_surf_get_image_alignment_sa(surf);
+
+      *halign = isl_to_gen_halign[image_align_sa.width];
+      *valign = isl_to_gen_valign[image_align_sa.height];
+   }
+}
+
+#if GEN_GEN >= 8
+static uint32_t
+get_qpitch(const struct isl_surf *surf)
+{
+   switch (surf->dim) {
+   default:
+      assert(!"Bad isl_surf_dim");
+   case ISL_SURF_DIM_1D:
+      if (GEN_GEN >= 9) {
+         /* QPitch is usually expressed as rows of surface elements (where
+          * a surface element is an compression block or a single surface
+          * sample). Skylake 1D is an outlier.
+          *
+          * From the Skylake BSpec >> Memory Views >> Common Surface
+          * Formats >> Surface Layout and Tiling >> 1D Surfaces:
+          *
+          *    Surface QPitch specifies the distance in pixels between array
+          *    slices.
+          */
+         return isl_surf_get_array_pitch_el(surf);
+      } else {
+         return isl_surf_get_array_pitch_el_rows(surf);
+      }
+   case ISL_SURF_DIM_2D:
+   case ISL_SURF_DIM_3D:
+      if (GEN_GEN >= 9) {
+         return isl_surf_get_array_pitch_el_rows(surf);
+      } else {
+         /* From the Broadwell PRM for RENDER_SURFACE_STATE.QPitch
+          *
+          *    "This field must be set to an integer multiple of the Surface
+          *    Vertical Alignment. For compressed textures (BC*, FXT1,
+          *    ETC*, and EAC* Surface Formats), this field is in units of
+          *    rows in the uncompressed surface, and must be set to an
+          *    integer multiple of the vertical alignment parameter "j"
+          *    defined in the Common Surface Formats section."
+          */
+         return isl_surf_get_array_pitch_sa_rows(surf);
+      }
+   }
+}
+#endif /* GEN_GEN >= 8 */
+
+void
+isl_genX(surf_fill_state_s)(const struct isl_device *dev, void *state,
+                            const struct isl_surf_fill_state_info *restrict info)
+{
+   uint32_t halign, valign;
+   get_halign_valign(info->surf, &halign, &valign);
+
+   struct GENX(RENDER_SURFACE_STATE) s = {
+      .SurfaceType = get_surftype(info->surf->dim, info->view->usage),
+      .SurfaceArray = info->surf->phys_level0_sa.array_len > 1,
+      .SurfaceVerticalAlignment = valign,
+      .SurfaceHorizontalAlignment = halign,
+
+#if GEN_GEN >= 8
+      .TileMode = isl_to_gen_tiling[info->surf->tiling],
+#else
+      .TiledSurface = info->surf->tiling != ISL_TILING_LINEAR,
+      .TileWalk = info->surf->tiling == ISL_TILING_X ? TILEWALK_XMAJOR :
+                                                       TILEWALK_YMAJOR,
+#endif
+
+      .VerticalLineStride = 0,
+      .VerticalLineStrideOffset = 0,
+
+#if GEN_GEN >= 8
+      .SamplerL2BypassModeDisable = true,
+#endif
+
+#if GEN_GEN >= 8
+      .RenderCacheReadWriteMode = WriteOnlyCache,
+#else
+      .RenderCacheReadWriteMode = 0,
+#endif
+
+#if GEN_GEN >= 8
+      .CubeFaceEnablePositiveZ = 1,
+      .CubeFaceEnableNegativeZ = 1,
+      .CubeFaceEnablePositiveY = 1,
+      .CubeFaceEnableNegativeY = 1,
+      .CubeFaceEnablePositiveX = 1,
+      .CubeFaceEnableNegativeX = 1,
+#else
+      .CubeFaceEnables = 0x3f,
+#endif
+
+#if GEN_GEN >= 8
+      .SurfaceQPitch = get_qpitch(info->surf) >> 2,
+#endif
+
+      .Width = info->level0_extent_px.width - 1,
+      .Height = info->level0_extent_px.height - 1,
+      .Depth = 0, /* TEMPLATE */
+
+      .SurfacePitch = info->surf->row_pitch - 1,
+      .RenderTargetViewExtent = 0, /* TEMPLATE */
+      .MinimumArrayElement = 0, /* TEMPLATE */
+
+      .MultisampledSurfaceStorageFormat =
+         isl_to_gen_multisample_layout[info->surf->msaa_layout],
+      .NumberofMultisamples = ffs(info->surf->samples) - 1,
+      .MultisamplePositionPaletteIndex = 0, /* UNUSED */
+
+      .XOffset = 0,
+      .YOffset = 0,
+
+      .ResourceMinLOD = 0.0,
+
+      .MIPCountLOD = 0, /* TEMPLATE */
+      .SurfaceMinLOD = 0, /* TEMPLATE */
+
+#if (GEN_GEN >= 8 || GEN_IS_HASWELL)
+      .ShaderChannelSelectRed = info->view->channel_select[0],
+      .ShaderChannelSelectGreen = info->view->channel_select[1],
+      .ShaderChannelSelectBlue = info->view->channel_select[2],
+      .ShaderChannelSelectAlpha = info->view->channel_select[3],
+#endif
+
+      .SurfaceBaseAddress = info->address,
+      .MOCS = info->mocs,
+
+#if GEN_GEN >= 8
+      .AuxiliarySurfaceMode = AUX_NONE,
+#else
+      .MCSEnable = false,
+#endif
+   };
+
+   if (info->view->usage & ISL_SURF_USAGE_STORAGE_BIT) {
+      s.SurfaceFormat = isl_lower_storage_image_format(dev, info->view->format);
+   } else {
+      s.SurfaceFormat = info->view->format;
+   }
+
+   switch (s.SurfaceType) {
+   case SURFTYPE_1D:
+   case SURFTYPE_2D:
+      s.MinimumArrayElement = info->view->base_array_layer;
+
+      /* From the Broadwell PRM >> RENDER_SURFACE_STATE::Depth:
+       *
+       *    For SURFTYPE_1D, 2D, and CUBE: The range of this field is reduced
+       *    by one for each increase from zero of Minimum Array Element. For
+       *    example, if Minimum Array Element is set to 1024 on a 2D surface,
+       *    the range of this field is reduced to [0,1023].
+       *
+       * In other words, 'Depth' is the number of array layers.
+       */
+      s.Depth = info->view->array_len - 1;
+
+      /* From the Broadwell PRM >> RENDER_SURFACE_STATE::RenderTargetViewExtent:
+       *
+       *    For Render Target and Typed Dataport 1D and 2D Surfaces:
+       *    This field must be set to the same value as the Depth field.
+       */
+      s.RenderTargetViewExtent = s.Depth;
+      break;
+   case SURFTYPE_CUBE:
+      s.MinimumArrayElement = info->view->base_array_layer;
+      /* Same as SURFTYPE_2D, but divided by 6 */
+      s.Depth = info->view->array_len / 6 - 1;
+      s.RenderTargetViewExtent = s.Depth;
+      break;
+   case SURFTYPE_3D:
+      s.MinimumArrayElement = info->view->base_array_layer;
+
+      /* From the Broadwell PRM >> RENDER_SURFACE_STATE::Depth:
+       *
+       *    If the volume texture is MIP-mapped, this field specifies the
+       *    depth of the base MIP level.
+       */
+      s.Depth = info->level0_extent_px.depth - 1;
+
+      /* From the Broadwell PRM >> RENDER_SURFACE_STATE::RenderTargetViewExtent:
+       *
+       *    For Render Target and Typed Dataport 3D Surfaces: This field
+       *    indicates the extent of the accessible 'R' coordinates minus 1 on
+       *    the LOD currently being rendered to.
+       */
+      s.RenderTargetViewExtent = info->level0_extent_px.depth - 1;
+      break;
+   default:
+      unreachable(!"bad SurfaceType");
+   }
+
+   if (info->view->usage & ISL_SURF_USAGE_RENDER_TARGET_BIT) {
+      /* For render target surfaces, the hardware interprets field
+       * MIPCount/LOD as LOD. The Broadwell PRM says:
+       *
+       *    MIPCountLOD defines the LOD that will be rendered into.
+       *    SurfaceMinLOD is ignored.
+       */
+      s.MIPCountLOD = info->view->base_level;
+      s.SurfaceMinLOD = 0;
+   } else {
+      /* For non render target surfaces, the hardware interprets field
+       * MIPCount/LOD as MIPCount.  The range of levels accessible by the
+       * sampler engine is [SurfaceMinLOD, SurfaceMinLOD + MIPCountLOD].
+       */
+      s.SurfaceMinLOD = info->view->base_level;
+      s.MIPCountLOD = MAX(info->view->levels, 1) - 1;
+   }
+
+#if 0
+   if (GEN_GEN == 8) {
+      if (isl_format_is_integer(info->view->format)) {
+         for (unsigned i = 0; i < 4; i++) {
+            assert(info->clear_color.u32[i] == 0 ||
+                   info->clear_color.u32[i] == 1);
+         }
+      } else {
+         for (unsigned i = 0; i < 4; i++) {
+            assert(info->clear_color.f32[i] == 0.0f ||
+                   info->clear_color.f32[i] == 1.0f);
+         }
+      }
+      s.RedClearColor = info->clear_color.u32[0] != 0,
+      s.GreenClearColor = info->clear_color.u32[1] != 0,
+      s.BlueClearColor = info->clear_color.u32[2] != 0,
+      s.AlphaClearColor = info->clear_color.u32[3] != 0,
+   } else {
+      .RedClearColor = info->clear_color.u32[0],
+      .GreenClearColor = info->clear_color.u32[1],
+      .BlueClearColor = info->clear_color.u32[2],
+      .AlphaClearColor = info->clear_color.u32[3],
+   }
+#endif
+
+   GENX(RENDER_SURFACE_STATE_pack)(NULL, state, &s);
+}