<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jun 2, 2017 at 12:57 PM, Nanley Chery <span dir="ltr"><<a href="mailto:nanleychery@gmail.com" target="_blank">nanleychery@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, May 26, 2017 at 04:30:18PM -0700, Jason Ekstrand wrote:<br>
</span><span class="">> This enum describes all of the states that a auxiliary compressed<br>
</span>                                             ^<br>
                                             an<br>
<div><div class="h5">> surface can have.  All of the states as well as normative language for<br>
> referring to each of the compression operations is provided in the<br>
> truly colossal comment for the new isl_aux_state enum.  There is also<br>
> a diagram showing how surfaces move between the different states.<br>
> ---<br>
>  src/intel/isl/isl.h | 142 ++++++++++++++++++++++++++++++<wbr>++++++++++++++++++++++<br>
>  1 file changed, 142 insertions(+)<br>
><br>
> diff --git a/src/intel/isl/isl.h b/src/intel/isl/isl.h<br>
> index b9d8fa8..df6d3e3 100644<br>
> --- a/src/intel/isl/isl.h<br>
> +++ b/src/intel/isl/isl.h<br>
> @@ -560,6 +560,148 @@ enum isl_aux_usage {<br>
>     ISL_AUX_USAGE_CCS_E,<br>
>  };<br>
><br>
> +/**<br>
> + * Enum for keeping track of the state an auxiliary compressed surface.<br>
> + *<br>
> + * For any given auxiliary surface compression format (HiZ, CCS, or MCS), any<br>
> + * given slice (lod + array layer) can be in one of the six states described<br>
> + * by this enum.  Draw and resolve operations may cause the slice to change<br>
> + * from one state to another.  The six valid states are:<br>
> + *<br>
> + *    1) Clear:  In this state, each block in the auxiliary surface contains a<br>
> + *       magic value that indicates that the block is in the clear state.  If<br>
> + *       a block is in the clear state, it's values in the primary surface are<br>
> + *       ignored and the color of the samples in the block is taken either the<br>
> + *       RENDER_SURFACE_STATE packet for color or 3DSTATE_CLEAR_PARAMS for<br>
> + *       depth.  Since neither the primary surface nor the auxiliary surface<br>
> + *       contains the clear value, the surface can be cleared to a different<br>
> + *       color by simply changing the clear color without modifying either<br>
> + *       surface.<br>
> + *<br>
> + *    2) Compressed w/ Clear:  In this state, neither the auxiliary surface<br>
> + *       nor the primary surface has a complete representation of the data.<br>
> + *       Instead, both surfaces must be used together or else rendering<br>
> + *       corruption may occur.  Depending on the auxiliary compression format<br>
> + *       and the data, any given block in the primary surface may contain all,<br>
> + *       some, or none of the data required to reconstruct the actual sample<br>
> + *       values.  Blocks may also be in the clear state (see Clear) and have<br>
> + *       their value taken from outside the surface.<br>
> + *<br>
> + *    3) Compressed w/o Clear:  This state is identical to the state above<br>
> + *       except that no blocks are in the clear state.  In this state, all of<br>
> + *       the data required to reconstruct the final sample values is contained<br>
> + *       in the auxiliary and primary surface and the clear value is not<br>
> + *       considered.<br>
> + *<br>
> + *    4) Resolved:  In this state, the primary surface contains 100% of the<br>
> + *       data.  The auxiliary surface is also valid so the surface can be<br>
> + *       validly used with or without aux enabled.  The auxiliary surface may,<br>
> + *       however, contain non-trivial data and any update to the primary<br>
> + *       surface with aux disabled will cause the two to get out of sync.<br>
> + *<br>
> + *    5) Pass-through:  In this state, the primary surface contains 100% of the<br>
> + *       data and every block in the auxiliary surface contains a magic value<br>
> + *       which indicates that the auxiliary surface should be ignored and the<br>
> + *       only the primary surface should be considered.  Updating the primary<br>
> + *       surface without aux works fine and can be done repeatedly in this<br>
> + *       mode.  Writing to a surface in pass-through mode with aux enabled may<br>
> + *       cause the auxiliary buffer to contain non-trivial data and no longer<br>
> + *       be in the pass-through state.<br>
> + *<br>
> + *    5) Aux Invalid:  In this state, the primary surface contains 100% of the<br>
</div></div>         ^<br>
         6<br>
<span class="">> + *       data and the auxiliary surface is completely bogus.  Any attempt to<br>
> + *       use the auxiliary surface is liable to result in rendering<br>
> + *       corruption.  The only thing that one can do to re-enable aux once<br>
> + *       this state is reached is to use an ambiguate pass to transition into<br>
> + *       the pass-through state.<br>
> + *<br>
> + * Drawing with or without aux enabled may implicitly cause the surface to<br>
> + * transition between these states.  There are also four types of "resolve"<br>
> + * operations which cause an explicit transition:<br>
> + *<br>
<br>
</span>I'm not sure why Fast Clear is described as a "resolve" operation.  To<br>
me, "resolve" operations are those which make data in one buffer match<br>
the data in another buffer. Perhaps you can clarify this?<br><div><div class="h5"></div></div></blockquote><div><br></div><div>How about "auxiliary compression operation"?  Yeah, resolve is a bit of a bad name.<br><br></div><div>--Jason<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
> + *    1) Fast Clear:  This operation writes the magic "clear" value to the<br>
> + *       auxiliary surface.  This operation will safely transition any slice<br>
> + *       of a surface from any state to the clear state so long as the entire<br>
> + *       slice is fast cleared at once.<br>
> + *<br>
> + *    2) Full Resolve:  This operation combines the auxiliary surface data<br>
> + *       with the primary surface data and writes the result to the primary.<br>
> + *       For CCS resolves, this operation is destructive in the sense that it<br>
> + *       also sets the auxiliary surface to the pass-through mode.  For HiZ,<br>
> + *       it is not destructive.<br>
> + *<br>
> + *    3) Partial Resolve:  This operation considers blocks which are in the<br>
> + *       "clear" state and writes the clear value directly into the primary or<br>
> + *       auxiliary surface.  Once this operation completes, the surface is<br>
> + *       still compressed but no longer references the clear color.  This<br>
> + *       operation is only available for CCS.<br>
> + *<br>
> + *    4) Ambiguate:  This operation throws away the current auxiliary data and<br>
> + *       replaces it with the magic pass-through value.  If an ambiguate<br>
> + *       operation is performed when the primary surface does not contain 100%<br>
> + *       of the data, data will be lost.  This operation is only implemented<br>
> + *       in hardware for depth where it is called a HiZ resolve.<br>
> + *<br>
> + * Not all operations are valid or useful in all states.  The diagram below<br>
> + * contains a complete description of the states and all valid and useful<br>
> + * transitions except clear.<br>
> + *<br>
> + *   Draw w/ Aux<br>
> + *   +----------+<br>
> + *   |          |<br>
> + *   |       +-------------+     Draw w/ Aux      +-------------+<br>
> + *   +------>| Compressed  |<---------------------|    Clear    |<br>
> + *           |  w/ Clear   |                      |             |<br>
> + *           +-------------+                      +-------------+<br>
> + *                  |   |                                |<br>
> + *          Partial |   |                                |<br>
> + *          Resolve |   |        Full Resolve            |<br>
> + *                  |   +----------------------------+   |  Full<br>
> + *                  |                                |   | Resolve<br>
> + *   Draw w/ aux    |                                |   |<br>
> + *   +----------+   |                                |   |<br>
> + *   |          |  \|/                              \|/ \|/<br>
> + *   |       +-------------+     Full Resolve     +-------------+<br>
> + *   +------>| Compressed  |--------------------->|  Resolved   |<br>
> + *           |  w/o Clear  |<---------------------|             |<br>
> + *           +-------------+     Draw w/ Aux      +-------------+<br>
> + *                 /|\                               |   |<br>
> + *                  |  Draw                          |   |  Draw<br>
> + *                  | w/ Aux                         |   | w/o Aux<br>
> + *                  |             Ambiguate          |   |<br>
> + *                  |   +----------------------------+   |<br>
> + *   Draw w/o Aux   |   |                                |   Draw w/o Aux<br>
> + *   +----------+   |   |                                |   +----------+<br>
> + *   |          |   |  \|/                              \|/  |          |<br>
> + *   |       +-------------+      Ambiguate       +-------------+       |<br>
> + *   +------>|    Pass-    |<---------------------|     Aux     |<------+<br>
> + *           |   through   |                      |   Invalid   |<br>
> + *           +-------------+                      +-------------+<br>
> + *<br>
> + *<br>
> + * As referenced in the description of the different operations above, not all<br>
> + * auxiliary surface formats actually support all of the above modes.  With<br>
> + * HiZ, for instance, does not have a partial resolve operation so the two<br>
> + * "compressed" modes are the same.  With CCS, the resolve operation is<br>
> + * destructive and takes you directly to passthrough so the "resolved" state<br>
> + * doesn't really exist.  However, if you consider the CCS resolve operation<br>
> + * as doing a resolve and then an ambiguate, the diagram is still accurate.<br>
> + */<br>
<br>
</div></div>This comment is very helpful.<br>
</blockquote></div><br></div></div>