<div dir="ltr"><div dir="ltr">On Fri, Sep 6, 2024 at 9:10 AM Jocelyn Falempe <<a href="mailto:jfalempe@redhat.com">jfalempe@redhat.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 06/09/2024 14:53, Ilia Mirkin wrote:<br>
> On Fri, Sep 6, 2024 at 6:05 AM Jocelyn Falempe <<a href="mailto:jfalempe@redhat.com" target="_blank">jfalempe@redhat.com</a> <br>
> <mailto:<a href="mailto:jfalempe@redhat.com" target="_blank">jfalempe@redhat.com</a>>> wrote:<br>
> <br>
>     Add drm_panic support, for nv50+ cards.<br>
>     It's enough to get the panic screen while running Gnome/Wayland on a<br>
>     GTX 1650.<br>
>     It doesn't support multi-plane or compressed format.<br>
>     Support for other formats and older cards will come later.<br>
>     Tiling is only tested on GTX1650, and might be wrong for other cards.<br>
> <br>
> <br>
> I'm moderately sure that nv50 and nvc0 tile differently (the general <br>
> algo is the same, but height is different):<br>
> <br>
> <a href="https://envytools.readthedocs.io/en/latest/hw/memory/g80-surface.html?highlight=tiling#blocklinear-surfaces" rel="noreferrer" target="_blank">https://envytools.readthedocs.io/en/latest/hw/memory/g80-surface.html?highlight=tiling#blocklinear-surfaces</a> <<a href="https://envytools.readthedocs.io/en/latest/hw/memory/g80-surface.html?highlight=tiling#blocklinear-surfaces" rel="noreferrer" target="_blank">https://envytools.readthedocs.io/en/latest/hw/memory/g80-surface.html?highlight=tiling#blocklinear-surfaces</a>><br>
<br>
Thanks, it looks like it needs a small adjustment, as in the Doc, GF100 <br>
uses a default height of 4, and GF100+ default to 8 (and I've hardcoded <br>
it to NV_TILE_BLK_BASE_HEIGHT 8).<br>
GF100 is still nv50, so it should use this code.<br></blockquote><div><br></div><div>Just as a point of clarification -- nv50 = G80. nvc0 = GF100. (There are many other chips, but those are the big generational changes as far as this code is concerned. <a href="https://nouveau.freedesktop.org/CodeNames.html">https://nouveau.freedesktop.org/CodeNames.html</a> for a fuller list.)</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> <br>
> That said, I don't know that nv50 supports scanout of tiled surfaces <br>
> (nor was I aware that nvc0+ did, perhaps it's a recent feature, or <br>
> perhaps I'm just forgetful).<br>
<br>
What I know is that when using Gnome/Wayland, the framebuffer is tiled, <br>
and without this tiling code, the panic screen is unreadable.<br>
When using the VT console, the framebuffer is linear, and it's easier.<br></blockquote><div><br></div><div>OK. Then clearly it's supported :) I'm sure Ben or Lyude will know offhand when this support came in.</div><div><br></div><div>Cheers,</div><div><br></div><div>  -ilia</div></div></div>