<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Sat, Jul 21, 2018, 05:45 Gert Wollny <<a href="mailto:gert.wollny@collabora.com">gert.wollny@collabora.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Am Freitag, den 20.07.2018, 12:31 -0400 schrieb Ilia Mirkin:<br>
> <br>
> > +   /* Allow 3-comp 32 bit texturs only for TBOs (needed for<br>
> > ARB_tbo_rgb32) */<br>
> > +   if ((format == PIPE_FORMAT_R32G32B32_FLOAT ||<br>
> > +       format == PIPE_FORMAT_R32G32B32_SINT ||<br>
> > +       format == PIPE_FORMAT_R32G32B32_UINT) &&<br>
> > +       target != PIPE_BUFFER)<br>
> > +      return FALSE;<br>
> <br>
> My personal recommendation would be to disallow *all* packed RGB<br>
> formats unless target == PIPE_BUFFER. (And even then -- questionable,<br>
> except for RGB32, which is required.)<br>
> <br>
RGB8 and RGB16 are disabled/replaced by RGBX* on the host side, and<br>
R11G11B10 doesn't seem to make problems. <br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">No other driver supports these. What happens on the host side is hidden from virgl. You're just signing yourself up for weird issues down the line.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
but thanks for the feedback anyway, <br>
Gert <br>
</blockquote></div></div></div>