intel/fs: Stop using brw_dp_read/write_desc in Gen7+ only code
Those helpers exist primarily to sort out some of the weirdness around Gen4-6 dataport access. On Gen5 and earlier, everything was called "dataport" and, instead of the SFID we have today there was a "target cache" parameter in the descriptor. There are also some bits that moved around on various gens depending on read vs. write. Starting with Gen6, most things which target one of the data cache SFIDs should use brw_dp_desc() instead. v2: Drop backward comment (Ken) Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by: Kenneth Graunke <kenneth@whitecape.org> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7455>
This commit is contained in:

committed by
Marge Bot

parent
2e7656ae2f
commit
34c560ae95
@@ -1653,10 +1653,9 @@ fs_generator::generate_uniform_pull_constant_load_gfx7(fs_inst *inst,
|
||||
brw_set_desc(p, send,
|
||||
brw_message_desc(devinfo, 1, DIV_ROUND_UP(inst->size_written,
|
||||
REG_SIZE), true) |
|
||||
brw_dp_read_desc(devinfo, surf_index,
|
||||
BRW_DATAPORT_OWORD_BLOCK_DWORDS(inst->exec_size),
|
||||
GFX7_DATAPORT_DC_OWORD_BLOCK_READ,
|
||||
BRW_DATAPORT_READ_TARGET_DATA_CACHE));
|
||||
brw_dp_desc(devinfo, surf_index,
|
||||
GFX7_DATAPORT_DC_OWORD_BLOCK_READ,
|
||||
BRW_DATAPORT_OWORD_BLOCK_DWORDS(inst->exec_size)));
|
||||
|
||||
} else {
|
||||
const tgl_swsb swsb = brw_get_default_swsb(p);
|
||||
@@ -1681,10 +1680,9 @@ fs_generator::generate_uniform_pull_constant_load_gfx7(fs_inst *inst,
|
||||
retype(payload, BRW_REGISTER_TYPE_UD), addr,
|
||||
brw_message_desc(devinfo, 1,
|
||||
DIV_ROUND_UP(inst->size_written, REG_SIZE), true) |
|
||||
brw_dp_read_desc(devinfo, 0 /* surface */,
|
||||
BRW_DATAPORT_OWORD_BLOCK_DWORDS(inst->exec_size),
|
||||
GFX7_DATAPORT_DC_OWORD_BLOCK_READ,
|
||||
BRW_DATAPORT_READ_TARGET_DATA_CACHE),
|
||||
brw_dp_desc(devinfo, 0 /* surface */,
|
||||
GFX7_DATAPORT_DC_OWORD_BLOCK_READ,
|
||||
BRW_DATAPORT_OWORD_BLOCK_DWORDS(inst->exec_size)),
|
||||
false /* EOT */);
|
||||
|
||||
brw_pop_insn_state(p);
|
||||
|
Reference in New Issue
Block a user