<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Nov 28, 2017 at 8:37 PM, Jason Ekstrand <span dir="ltr"><<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This series is one more go at fixing texture_from_pixmap.  The previous<br>
patches I apparently didn't test well enough because it regressed a bunch<br>
of tests when I ran it through Jenkins today.  These are all fixed by the<br>
new patch 1 and the corresponding hunk in patch 2.  I've also re-ordered<br>
things a bit so that "Pull the tex format from the renderbuffer in<br>
intelSetTexBuffer2" before "Reference the renderbuffer miptree in<br>
setTexBuffer2" to address some concerns from Chad<br></blockquote><div><br></div><div>I should probably add that I don't really like the way my solution turned out.  Unfortunately, the way miptrees work is kind-of a mess and I couldn't really come up with a better way to let tex_obj->_Format differ from mt->format other than to make it match the gl_texture_images.</div><div><br></div><div>--Jason<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Cc: Chad Versace <<a href="mailto:chadversary@chromium.org">chadversary@chromium.org</a>><br>
Cc: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
<br>
Jason Ekstrand (6):<br>
  i965/miptree: Add a helper for getting the textureable format<br>
  i965/miptree: Loosen the format check in miptree_match_image<br>
  i965/tex_image: Pull the tex format from the renderbuffer in<br>
    intelSetTexBuffer2<br>
  i965/tex_image: Reference the renderbuffer miptree in setTexBuffer2<br>
  i965: Use finish_external instead of make_shareable in setTexBuffer2<br>
  intel/isl: Improve the documentation on get_default_aux_state<br>
<br>
 src/intel/isl/isl.h                              |  24 ++++-<br>
 src/mesa/drivers/dri/i965/brw_<wbr>draw.c             |   3 +-<br>
 src/mesa/drivers/dri/i965/brw_<wbr>wm_surface_state.c |  10 ++-<br>
 src/mesa/drivers/dri/i965/<wbr>intel_mipmap_tree.c    |  57 +++++++++++-<br>
 src/mesa/drivers/dri/i965/<wbr>intel_mipmap_tree.h    |   9 ++<br>
 src/mesa/drivers/dri/i965/<wbr>intel_screen.c         |   2 +-<br>
 src/mesa/drivers/dri/i965/<wbr>intel_tex.c            |   2 +-<br>
 src/mesa/drivers/dri/i965/<wbr>intel_tex.h            |   2 +<br>
 src/mesa/drivers/dri/i965/<wbr>intel_tex_image.c      | 107 ++++++++++++++++++-----<br>
 src/mesa/drivers/dri/i965/<wbr>intel_tex_obj.h        |   4 +-<br>
 src/mesa/drivers/dri/i965/<wbr>intel_tex_validate.c   |   2 +-<br>
 11 files changed, 185 insertions(+), 37 deletions(-)<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
2.5.0.400.gff86faf<br>
<br>
</font></span></blockquote></div><br></div></div>