Mesa (master): i965/fs: Set the header_size on LOAD_PAYLOAD in opt_sampler_eot

Neil Roberts nroberts at kemper.freedesktop.org
Fri May 8 11:44:36 UTC 2015


Module: Mesa
Branch: master
Commit: 1c5de556c5972c3020b4095c586a9b439b20cf69
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=1c5de556c5972c3020b4095c586a9b439b20cf69

Author: Neil Roberts <neil at linux.intel.com>
Date:   Thu May  7 18:55:55 2015 +0100

i965/fs: Set the header_size on LOAD_PAYLOAD in opt_sampler_eot

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.

Reviewed-by: Jason Ekstrand <jason.ekstrand at intel.com>
Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>

---

 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);




More information about the mesa-commit mailing list