<div dir="ltr">Hi Matt,<div><br></div><div>I posted results in my cover letter email: [PATCH 0/2v2] i965: Extend fast texture upload</div><div><br></div><div>Please let me know if you want more details.</div><div><br></div>
<div>Courtney</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Nov 7, 2013 at 2:28 PM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I saw the perf demo patches and expected to see results from them in<br>
this patch. Any results to share?<br>
<div><div class="h5"><br>
On Thu, Nov 7, 2013 at 1:22 PM, Courtney Goeltzenleuchter<br>
<<a href="mailto:courtney@lunarg.com">courtney@lunarg.com</a>> wrote:<br>
> Support all levels of a supported texture format.<br>
><br>
> Signed-off-by: Courtney Goeltzenleuchter <courtney@LunarG.com><br>
> ---<br>
>  src/mesa/drivers/dri/i965/intel_tex_subimage.c | 10 +++++++---<br>
>  1 file changed, 7 insertions(+), 3 deletions(-)<br>
><br>
> diff --git a/src/mesa/drivers/dri/i965/intel_tex_subimage.c b/src/mesa/drivers/dri/i965/intel_tex_subimage.c<br>
> index b1826fa..b32af3d 100644<br>
> --- a/src/mesa/drivers/dri/i965/intel_tex_subimage.c<br>
> +++ b/src/mesa/drivers/dri/i965/intel_tex_subimage.c<br>
> @@ -543,7 +543,7 @@ intel_texsubimage_tiled_memcpy(struct gl_context * ctx,<br>
>     uint32_t cpp;<br>
>     mem_copy_fn mem_copy = NULL;<br>
><br>
> -   /* This fastpath is restricted to specific texture types: level 0 of<br>
> +   /* This fastpath is restricted to specific texture types:<br>
>      * a 2D BGRA, RGBA, L8 or A8 texture. It could be generalized to support<br>
>      * more types.<br>
>      *<br>
> @@ -555,7 +555,6 @@ intel_texsubimage_tiled_memcpy(struct gl_context * ctx,<br>
>     if (!brw->has_llc ||<br>
>         type != GL_UNSIGNED_BYTE ||<br>
>         texImage->TexObject->Target != GL_TEXTURE_2D ||<br>
> -       texImage->Level != 0 ||<br>
>         pixels == NULL ||<br>
>         _mesa_is_bufferobj(packing->BufferObj) ||<br>
>         packing->Alignment > 4 ||<br>
> @@ -572,7 +571,7 @@ intel_texsubimage_tiled_memcpy(struct gl_context * ctx,<br>
>        cpp = 1;<br>
>        mem_copy = memcpy;<br>
>     } else if ((texImage->TexFormat == MESA_FORMAT_ARGB8888)<br>
> -         || (texImage->TexFormat == MESA_FORMAT_XRGB8888)) {<br>
> +        || (texImage->TexFormat == MESA_FORMAT_XRGB8888)) {<br>
<br>
</div></div>Extraneous change?<br>
<div class="HOEnZb"><div class="h5"><br>
>        cpp = 4;<br>
>        if (format == GL_BGRA) {<br>
>           mem_copy = memcpy;<br>
> @@ -631,6 +630,11 @@ intel_texsubimage_tiled_memcpy(struct gl_context * ctx,<br>
>         packing->Alignment, packing->RowLength, packing->SkipPixels,<br>
>         packing->SkipRows, for_glTexImage);<br>
><br>
> +   /* Adjust x and y offset based on miplevel<br>
> +    */<br>
> +   xoffset += image->mt->level[texImage->Level].level_x;<br>
> +   yoffset += image->mt->level[texImage->Level].level_y;<br>
> +<br>
>     linear_to_tiled(<br>
>        xoffset * cpp, (xoffset + width) * cpp,<br>
>        yoffset, yoffset + height,<br>
> --<br>
> 1.8.1.2<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Courtney Goeltzenleuchter<br><div>LunarG</div><div><img src="http://media.lunarg.com/wp-content/themes/LunarG/images/logo.png" width="96" height="65"><br>
</div></div>
</div>