<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 2, 2013 at 4:32 PM, Alex Deucher <span dir="ltr"><<a href="mailto:alexdeucher@gmail.com" target="_blank">alexdeucher@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="im">On Tue, Apr 2, 2013 at 2:13 PM, Jerome Glisse <<a href="mailto:j.glisse@gmail.com">j.glisse@gmail.com</a>> wrote:<br>

</div><div><div class="h5">> So i am facing a dilema regarding tiling on radeonsi. Given that we now have<br>
> a fixed table of tiling mode this put more pressure on the kernel userspace<br>
> api. I see either 2 solutions.<br>
><br>
> Enforce kernel to set at fixed index in the table best tiling mode for given<br>
> gpu for given format, such as DEPTH32_2D_4AA at index 4, or COLOR_SCANOUT_2D<br>
> at index 13 ... that way kernel can still adapt the tile mode array value.<br>
> Note that this match the design behind the tile mode index being that there<br>
> is a limited number of useful tile mode combination and for each surface<br>
> format  (depth/color/macro tile/micro/tile) there is a best one.<br>
><br>
> Second solution is to add an ioctl to compute mipmap information in kernel<br>
> (pitch alignment slice size ...) based on format, size of the surface.<br>
><br>
> Some might argue that we could just export the table content to userspace,<br>
> but that would loose information and possibly froze the tile mode table<br>
> forever as API. The information we loose is what index match to prefered<br>
> surface format/type combination. And the tile mode might be considered API<br>
> as if kernel ever change what userspace expect then we might break some<br>
> userspace.<br>
><br>
><br>
> Thought, idea ?<br>
<br>
</div></div>If it helps any, the tiling information for the GB_TILE_MODE register<br>
settings and index locations come from a table provided by the<br>
hardware team during bring up and barring any typos in transcribing<br>
the information, never really changes.<br>
<span class=""><font color="#888888"><br>
Alex<br>
</font></span></blockquote></div><br></div><div class="gmail_extra">Still sounds like you likely want to update those given gap there is inside them.<br><br></div><div class="gmail_extra">Patch set for solution 1:<br><a href="http://people.freedesktop.org/~glisse/si2d-sol1/">http://people.freedesktop.org/~glisse/si2d-sol1/</a><br>
<br></div><div class="gmail_extra">Cheers,<br>Jerome<br></div></div>