<feed xmlns='http://www.w3.org/2005/Atom'>
<title>external_mesa3d/src/intel/isl/isl.h, branch replicant-6.0</title>
<subtitle>external/mesa3d
</subtitle>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant/external_mesa3d/'/>
<entry>
<title>isl: use ifndef header guards</title>
<updated>2016-10-14T10:53:32+00:00</updated>
<author>
<name>Emil Velikov</name>
<email>emil.velikov@collabora.com</email>
</author>
<published>2016-10-11T17:26:20+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant/external_mesa3d/commit/?id=3a98bffa5924d5e7d078268cfe5a42c9f58f0470'/>
<id>3a98bffa5924d5e7d078268cfe5a42c9f58f0470</id>
<content type='text'>
Signed-off-by: Emil Velikov &lt;emil.velikov@collabora.com&gt;
Reviewed-by: Chad Versace &lt;chadversary@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Emil Velikov &lt;emil.velikov@collabora.com&gt;
Reviewed-by: Chad Versace &lt;chadversary@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>isl: make locally used functions static</title>
<updated>2016-10-14T10:53:30+00:00</updated>
<author>
<name>Emil Velikov</name>
<email>emil.velikov@collabora.com</email>
</author>
<published>2016-10-11T17:26:19+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant/external_mesa3d/commit/?id=4c1c9d62a9503d912f0f4c22e0e5a162c7f2e0d3'/>
<id>4c1c9d62a9503d912f0f4c22e0e5a162c7f2e0d3</id>
<content type='text'>
Signed-off-by: Emil Velikov &lt;emil.velikov@collabra.com&gt;
Reviewed-by: Chad Versace &lt;chadversary@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Emil Velikov &lt;emil.velikov@collabra.com&gt;
Reviewed-by: Chad Versace &lt;chadversary@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>isl: Correct a comment in the isl_format enum</title>
<updated>2016-10-07T19:54:17+00:00</updated>
<author>
<name>Nanley Chery</name>
<email>nanley.g.chery@intel.com</email>
</author>
<published>2016-10-07T19:07:22+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant/external_mesa3d/commit/?id=814fa1237935884c3c0ba47a31e3599b975aec1a'/>
<id>814fa1237935884c3c0ba47a31e3599b975aec1a</id>
<content type='text'>
HiZ is not a color surface, but an auxiliary depth surface.

Signed-off-by: Nanley Chery &lt;nanley.g.chery@intel.com&gt;
Reviewed-by: Chad Versace &lt;chadversary@chromium.org&gt;
Reviewed-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
HiZ is not a color surface, but an auxiliary depth surface.

Signed-off-by: Nanley Chery &lt;nanley.g.chery@intel.com&gt;
Reviewed-by: Chad Versace &lt;chadversary@chromium.org&gt;
Reviewed-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>intel/isl: Add a format_supports_multisampling helper</title>
<updated>2016-10-03T21:53:01+00:00</updated>
<author>
<name>Jason Ekstrand</name>
<email>jason.ekstrand@intel.com</email>
</author>
<published>2016-09-02T01:57:01+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant/external_mesa3d/commit/?id=5637f3f1202b46ff02c80e743dd84752a040685a'/>
<id>5637f3f1202b46ff02c80e743dd84752a040685a</id>
<content type='text'>
Signed-off-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
Reviewed-by: Chad Versace &lt;chadversary@chromium.org&gt;
Reviewed-by: Nanley Chery &lt;nanley.g.chery@intel.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
Reviewed-by: Chad Versace &lt;chadversary@chromium.org&gt;
Reviewed-by: Nanley Chery &lt;nanley.g.chery@intel.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>intel/isl: Add support for RGB formats in X and Y-tiled memory</title>
<updated>2016-09-13T02:44:05+00:00</updated>
<author>
<name>Jason Ekstrand</name>
<email>jason.ekstrand@intel.com</email>
</author>
<published>2016-09-09T05:18:47+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant/external_mesa3d/commit/?id=d038adca0eae173be0d2082f9b6d7e6d7c8aadf1'/>
<id>d038adca0eae173be0d2082f9b6d7e6d7c8aadf1</id>
<content type='text'>
Normally, using a non-linear tiling format helps improve cache locality by
ensuring that neighboring pixels are usually close-by in memory.  For RGB
formats, this still sort-of holds, but it can also lead to rather terrible
memory access patterns where a single RGB pixel value crosses a tile
boundary and gets split into two pieces in different 4K pages.  It also
makes for some rather awkward calculations because your tile size is no
longer an even multiple of surface element size.  For these reasons, we
chose to simply never create tiled RGB images in the Vulkan driver.

The GL driver, however, is not so kind so we need to support it somehow.  I
briefly toyed with a couple of different schemes but this is the best one I
could come up with.  The fundamental problem is that a tile no longer
contains an integer number of surface elements.  I briefly considered a
couple other options but found them wanting:

 1) Using floats for the logical tile size.  This leads to potential
    rounding error problems.

 2) When presented with a RGB format, just make the tile 3-times as wide.
    This isn't so nice because now our tiles are no longer power-of-two
    size.  Also, it can force the row_pitch to be larger than needed which,
    while not strictly a problem for ISL, causes incompatibility problems
    with the way the GL driver chooses surface pitches.

The chosen method requires that you pay attention and not just assume that
your tile_info is in the units you think it is.  However, it's nice because
it provides a nice "these are the units" declaration in isl_tile_info
itself.  Previously, the tile_info wasn't usable as a stand-alone structure
because you had to also know the format.  It also forces figuring out how
to deal with inconsistencies between tiling and format back to the caller
which is good because the two different consumers of isl_tile_info really
want to deal with it differently:  Computation of the surface size wants
the fewest number of horizontal tiles possible while get_intratile_offset
is far more concerned with things aligning nicely.

Signed-off-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
Acked-by: Chad Versace &lt;chadversary@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Normally, using a non-linear tiling format helps improve cache locality by
ensuring that neighboring pixels are usually close-by in memory.  For RGB
formats, this still sort-of holds, but it can also lead to rather terrible
memory access patterns where a single RGB pixel value crosses a tile
boundary and gets split into two pieces in different 4K pages.  It also
makes for some rather awkward calculations because your tile size is no
longer an even multiple of surface element size.  For these reasons, we
chose to simply never create tiled RGB images in the Vulkan driver.

The GL driver, however, is not so kind so we need to support it somehow.  I
briefly toyed with a couple of different schemes but this is the best one I
could come up with.  The fundamental problem is that a tile no longer
contains an integer number of surface elements.  I briefly considered a
couple other options but found them wanting:

 1) Using floats for the logical tile size.  This leads to potential
    rounding error problems.

 2) When presented with a RGB format, just make the tile 3-times as wide.
    This isn't so nice because now our tiles are no longer power-of-two
    size.  Also, it can force the row_pitch to be larger than needed which,
    while not strictly a problem for ISL, causes incompatibility problems
    with the way the GL driver chooses surface pitches.

The chosen method requires that you pay attention and not just assume that
your tile_info is in the units you think it is.  However, it's nice because
it provides a nice "these are the units" declaration in isl_tile_info
itself.  Previously, the tile_info wasn't usable as a stand-alone structure
because you had to also know the format.  It also forces figuring out how
to deal with inconsistencies between tiling and format back to the caller
which is good because the two different consumers of isl_tile_info really
want to deal with it differently:  Computation of the surface size wants
the fewest number of horizontal tiles possible while get_intratile_offset
is far more concerned with things aligning nicely.

Signed-off-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
Acked-by: Chad Versace &lt;chadversary@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>intel/isl: Add a helper for getting the size of an interleaved pixel</title>
<updated>2016-09-13T02:42:57+00:00</updated>
<author>
<name>Jason Ekstrand</name>
<email>jason.ekstrand@intel.com</email>
</author>
<published>2016-08-29T23:34:06+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant/external_mesa3d/commit/?id=8ac99eabb6570f0f3c5f7d7da1332a99ce636362'/>
<id>8ac99eabb6570f0f3c5f7d7da1332a99ce636362</id>
<content type='text'>
Signed-off-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
Reviewed-by: Topi Pohjolainen &lt;topi.pohjolainen@intel.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
Reviewed-by: Topi Pohjolainen &lt;topi.pohjolainen@intel.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>intel/isl: Fix an assert in get_intratile_offset_sa</title>
<updated>2016-09-13T02:42:57+00:00</updated>
<author>
<name>Jason Ekstrand</name>
<email>jason.ekstrand@intel.com</email>
</author>
<published>2016-08-29T16:16:06+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant/external_mesa3d/commit/?id=43d25edf78c3723ec76eb0704ed99f5345ef8d1e'/>
<id>43d25edf78c3723ec76eb0704ed99f5345ef8d1e</id>
<content type='text'>
Signed-off-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
Reviewed-by: Topi Pohjolainen &lt;topi.pohjolainen@intel.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
Reviewed-by: Topi Pohjolainen &lt;topi.pohjolainen@intel.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>intel/isl: Add an isl_swizzle structure and use it for isl_view swizzles</title>
<updated>2016-09-13T02:42:57+00:00</updated>
<author>
<name>Jason Ekstrand</name>
<email>jason.ekstrand@intel.com</email>
</author>
<published>2016-08-28T04:39:16+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant/external_mesa3d/commit/?id=7ddb21708c800dfbe3cea2df6aaaf7aed852761b'/>
<id>7ddb21708c800dfbe3cea2df6aaaf7aed852761b</id>
<content type='text'>
This should be more compact than the enum isl_channel_select[4] that we
were using before.  It's also very convenient because we already had such a
structure in the Vulkan driver we just needed to pull it over.

Signed-off-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
Reviewed-by: Topi Pohjolainen &lt;topi.pohjolainen@intel.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This should be more compact than the enum isl_channel_select[4] that we
were using before.  It's also very convenient because we already had such a
structure in the Vulkan driver we just needed to pull it over.

Signed-off-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
Reviewed-by: Topi Pohjolainen &lt;topi.pohjolainen@intel.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>intel/isl: Treat 3-D textures as 2-D arrays for rendering</title>
<updated>2016-09-12T23:48:56+00:00</updated>
<author>
<name>Jason Ekstrand</name>
<email>jason.ekstrand@intel.com</email>
</author>
<published>2016-09-12T15:13:43+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant/external_mesa3d/commit/?id=48f195d7c6483ed00a6d02526c6f35bb527fcaba'/>
<id>48f195d7c6483ed00a6d02526c6f35bb527fcaba</id>
<content type='text'>
In particular, this means that isl_view::base_array_layer and
isl_view::array_len get applied to 3-D textures but only when rendering.
We were already applying isl_view::base_array_layer for rendering into 3-D
textures so this isn't a huge deviation.

Signed-off-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
Reviewed-by: Kenneth Graunke &lt;kenneth@whitecape.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
In particular, this means that isl_view::base_array_layer and
isl_view::array_len get applied to 3-D textures but only when rendering.
We were already applying isl_view::base_array_layer for rendering into 3-D
textures so this isn't a huge deviation.

Signed-off-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
Reviewed-by: Kenneth Graunke &lt;kenneth@whitecape.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>intel: s/brw_device_info/gen_device_info/</title>
<updated>2016-09-03T15:23:06+00:00</updated>
<author>
<name>Jason Ekstrand</name>
<email>jason.ekstrand@intel.com</email>
</author>
<published>2016-08-22T22:01:08+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant/external_mesa3d/commit/?id=527f37199929932300acc1688d8160e1f3b1d753'/>
<id>527f37199929932300acc1688d8160e1f3b1d753</id>
<content type='text'>
Generated by:

sed -i -e 's/brw_device_info/gen_device_info/g' src/intel/**/*.c
sed -i -e 's/brw_device_info/gen_device_info/g' src/intel/**/*.h
sed -i -e 's/brw_device_info/gen_device_info/g' **/i965/*.c
sed -i -e 's/brw_device_info/gen_device_info/g' **/i965/*.cpp
sed -i -e 's/brw_device_info/gen_device_info/g' **/i965/*.h

Signed-off-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
Reviewed-by: Jordan Justen &lt;jordan.l.justen@intel.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Generated by:

sed -i -e 's/brw_device_info/gen_device_info/g' src/intel/**/*.c
sed -i -e 's/brw_device_info/gen_device_info/g' src/intel/**/*.h
sed -i -e 's/brw_device_info/gen_device_info/g' **/i965/*.c
sed -i -e 's/brw_device_info/gen_device_info/g' **/i965/*.cpp
sed -i -e 's/brw_device_info/gen_device_info/g' **/i965/*.h

Signed-off-by: Jason Ekstrand &lt;jason@jlekstrand.net&gt;
Reviewed-by: Jordan Justen &lt;jordan.l.justen@intel.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
