Revert "anv: limit URB reconfigurations when using blorp"
authorLionel Landwerlin <lionel.g.landwerlin@intel.com>
Sat, 27 Apr 2019 03:35:32 +0000 (11:35 +0800)
committerLionel Landwerlin <lionel.g.landwerlin@intel.com>
Mon, 29 Apr 2019 11:41:27 +0000 (11:41 +0000)
commit9628631a380ff24630bf97b982df07b4e7e2a79f
tree3b1bce0d90f8f27fba45fcf92414b8e0d2e4352b
parentcc5b8a938a4064c26bd9cd034ad2cafedb04ee9a
Revert "anv: limit URB reconfigurations when using blorp"

In commit 0d46e404 ("anv: limit URB reconfigurations when using
blorp") we tried to limit the number of URB reconfiguration by
checking if the last allocation is large enough to fit the blorp
dispatch.

We used the last bound pipeline to compare the allocation. The problem
with this is that the pipeline is bound but its commands might not
have been emitted into the command buffer yet.

Let's just revert commit 0d46e404677264bfb12ada15290e39c10a5eb455
since it didn't seem to yield any performance improvement.

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Fixes: 0d46e404 ("anv: limit URB reconfigurations when using blorp")
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110535
Acked-by: Jason Ekstrand <jason@jlekstrand.net>
src/intel/vulkan/anv_private.h
src/intel/vulkan/genX_blorp_exec.c
src/intel/vulkan/genX_pipeline.c