<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 3, 2019 at 7:58 PM David Airlie <<a href="mailto:airlied@redhat.com">airlied@redhat.com</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 Wed, Apr 3, 2019 at 5:23 PM Gerd Hoffmann <<a href="mailto:kraxel@redhat.com" target="_blank">kraxel@redhat.com</a>> wrote:<br>
><br>
> Time to kill some bad sample code people are copying from ;)<br>
><br>
> This is a complete rewrite of the cirrus driver.  The cirrus_mode_set()<br>
> function is pretty much the only function which is carried over largely<br>
> unmodified.  Everything else is upside down.<br>
><br>
> It is a single monster patch.  But given that it does some pretty<br>
> fundamental changes to the drivers workflow and also reduces the code<br>
> size by roughly 70% I think it'll still be alot easier to review than a<br>
> longish baby-step patch series.<br>
><br>
> Changes summary:<br>
>  - Given the small amout of video memory (4 MB) the cirrus device has<br>
>    the rewritten driver doesn't try to manage buffers there.  Instead<br>
>    it will blit (memcpy) the active framebuffer to video memory.<br>
<br>
Does it get any slower, with TTM I just wrote it to migrate just the<br>
frontbuffer in/out of VRAM on modeset, won't we end up with more<br>
copies now?<br>
<br>
>  - All gem objects are stored in main memory and are manged using the<br>
>    new shmem helpers.  ttm is out.<br>
>  - Only DRM_FORMAT_RGB565 (depth 16) is supported.  The old driver does<br>
>    that too by default.  There was a module parameter which enables 24/32<br>
>    bpp support and disables higher resolutions (due to cirrus hardware<br>
>    constrains).  That parameter wasn't reimplemented.<br>
This might be the big sticking point, this is a userspace regression<br>
for a feature that was explicitly added a few years ago, can we really<br>
get away without it?<br></blockquote><div><br></div><div>Chrome OS testing in VMs was one of the consumers of 32bpp on cirrus, and we have gotten rid of cirrus in favor of virtio gpu, so we'd be fine. Of course I can't speak for other consumers :)</div><div><br></div><div>Stéphane</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
The rest looks good though!<br>
Dave.<br>
<br>
>  - The simple display pipeline is used.<br>
>  - The generic fbdev emulation is used.<br>
>  - It's a atomic driver now.<br>
><br>
_______________________________________________<br>
dri-devel mailing list<br>
<a href="mailto:dri-devel@lists.freedesktop.org" target="_blank">dri-devel@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/dri-devel" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/dri-devel</a></blockquote></div></div>