<div dir="ltr"><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Mon, Jan 20, 2025 at 1:41 PM Simona Vetter <<a href="mailto:simona.vetter@ffwll.ch">simona.vetter@ffwll.ch</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Jan 20, 2025 at 08:58:20AM +0100, Thomas Zimmermann wrote:<br>
> Hi<br>
> <br>
> <br>
> Am 18.01.25 um 03:37 schrieb Marek Olšák:<br>
> [...]<br>
> > <br>
> > 3) Implementing DRM_FORMAT_MOD_LINEAR as having 256B pitch and offset<br>
> > alignment. This is what we do today. Even if Intel and some AMD chips<br>
> > can do 64B or 128B alignment, they overalign to 256B. With so many<br>
> > AMD+NV laptops out there, NV is probably next, unless they already do<br>
> > this in the closed source driver.<br>
<br>
I don't think this works, or at least not any better than the current<br>
linear modifier. There's way too many users of that thing out there that I<br>
think you can realistically redefine it.<br></blockquote><div><br></div>DRM_FORMAT_MOD_LINEAR was redefined on PC a long time ago to mean 256B pitch alignment because of laptops with AMD+Intel. Drivers redefined it because that's what happens when it's under-defined. As you say, DRM_FORMAT_MOD_LINEAR can't be removed, but then it can't work with any other pitch alignment on all PC hw either, so there is no other choice.</div><div class="gmail_quote gmail_quote_container"><br></div><div class="gmail_quote gmail_quote_container">The options for PC are either a new parameterized linear modifier (with properly defined addressing and size equations) or DRM_FORMAT_MOD_LINEAR with 256B pitch alignment. There is no 3rd option. Even if you totally disregard AMD, you won't get it below 128B or 64B on the rest of PC hw anyway, and that's the same problem.<br></div><div class="gmail_quote gmail_quote_container"><br></div><div class="gmail_quote gmail_quote_container">Marek<br></div></div>