[Mesa-dev] [PATCH] i965/fs: Set the header_size on LOAD_PAYLOAD in opt_sampler_eot

Jason Ekstrand jason at jlekstrand.net
Thu May 7 11:15:46 PDT 2015


Reviewed-by: Jason Ekstrand <jason.ekstrand at intel.com>

On Thu, May 7, 2015 at 11:06 AM, Neil Roberts <neil at linux.intel.com> wrote:
> Commit 94ee908448 added a header size parameter to the function to
> create the LOAD_PAYLOAD instruction. However this broke
> opt_sampler_eot which manually constructs the instruction and so
> wasn't setting the header_size. This ends up making the parameters for
> the send message all have the wrong location and it all falls apart.
> ---
>  src/mesa/drivers/dri/i965/brw_fs.cpp | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp
> index 3bf5866..02a1ad5 100644
> --- a/src/mesa/drivers/dri/i965/brw_fs.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp
> @@ -2701,6 +2701,7 @@ fs_visitor::opt_sampler_eot()
>                                                      load_payload->sources + 1);
>
>     new_load_payload->regs_written = load_payload->regs_written + 1;
> +   new_load_payload->header_size = 1;
>     tex_inst->mlen++;
>     tex_inst->header_size = 1;
>     tex_inst->insert_before(cfg->blocks[cfg->num_blocks - 1], new_load_payload);
> --
> 1.9.3
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list