On Mon, Sep 24, 2012 at 5:22 PM, Brian Paul <span dir="ltr"><<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On 09/23/2012 05:44 AM, John Kåre Alsaker wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
---<br>
  src/gallium/drivers/svga/svga_resource_texture.c | 8 --------<br>
  1 file changed, 8 deletions(-)<br>
<br>
diff --git a/src/gallium/drivers/svga/svga_resource_texture.c b/src/gallium/drivers/svga/svga_resource_texture.c<br>
index 9830e79..97ec7ee 100644<br>
--- a/src/gallium/drivers/svga/svga_resource_texture.c<br>
+++ b/src/gallium/drivers/svga/svga_resource_texture.c<br>
@@ -540,14 +540,6 @@ svga_texture_from_handle(struct pipe_screen *screen,<br>
     pipe_reference_init(&tex->b.b.reference, 1);<br>
     tex->b.b.screen = screen;<br>
<br>
-   if (format == SVGA3D_X8R8G8B8)<br>
-      tex->b.b.format = PIPE_FORMAT_B8G8R8X8_UNORM;<br>
-   else if (format == SVGA3D_A8R8G8B8)<br>
-      tex->b.b.format = PIPE_FORMAT_B8G8R8A8_UNORM;<br>
-   else {<br>
-      /* ?? */<br>
-   }<br>
-<br>
     SVGA_DBG(DEBUG_DMA, "wrap surface sid %p\n", srf);<br>
<br>
     tex->key.cachable = 0;<br>
</blockquote>
<br></div></div>
Yeah, I don't know what that's all about either.  Have you done a piglit run with this change to check for regressions?</blockquote><div>Piglit doesn't pass the sanity tests with Mesa master. Are there any tests for shared handles in there?</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888"><br>
<br>
-Brian<br>
</font></span></blockquote><br>