<div dir="ltr"><div>FYI, this series is open for reviews again and planned to be pushed soon.<br><br></div>Marek<br><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 26, 2018 at 10:13 PM, Marek Olšák <span dir="ltr"><<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Please ignore this series.<span class="m_1191117992904295377HOEnZb"><font color="#888888"><br><br></font></span></div><span class="m_1191117992904295377HOEnZb"><font color="#888888">Marek<br></font></span></div><div class="m_1191117992904295377HOEnZb"><div class="m_1191117992904295377h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 25, 2018 at 5:16 PM, Marek Olšák <span dir="ltr"><<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
This feature is for gralloc, which requires drivers to be able to map<br>
an image with a stride of its own choosing, which is usually the same<br>
as the image stride. This is a very silly feature that probably comes<br>
from designing around mobile GPUs, and must be emulated on everything<br>
else.<br>
<br>
GCN is pretty limited here. A 16x16 tiled image can have a stride of 16<br>
because it's tiled, but the stride of a linear mapping of that image<br>
will be 64 on <= Polaris, or 256 on >= Vega.<br>
<br>
The hardware doesn't have the capability to give you a stride that is<br>
not a multiple of 64 or 256, so gralloc will have to deal with it.<br>
<br>
Nicolai wrote the first 6 commits. I added the last 4.<br>
(so I'll put my Rb on the first 6)<br>
<br>
Please review,<br>
<br>
Thanks,<br>
Marek<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div></div></div></div>