external_mesa3d/src/intel/vulkan/anv_cmd_buffer.c, branch replicant-6.0

anv: Handle null in all destructors

2016-11-24T16:34:40+00:00

This fixes a bunch of new CTS tests which look for exactly this.  Even in
the cases where we just call vk_free to free a CPU data structure, we still
handle NULL explicitly.  This way we're less likely to forget to handle
NULL later should we actually do something less trivial.

Cc: "13.0" 
Reviewed-by: Dave Airlie 
(cherry picked from commit 49f08ad77f51cc344e4bfe60ba9f8d9fccfbd753)
[Emil Velikov: color_rt_surface_state is still around]
Signed-off-by: Emil Velikov 

Conflicts:
	src/intel/vulkan/anv_image.c

anv/pipeline: Put actual pointers in anv_shader_bin

2016-11-09T13:44:32+00:00

While we can simply calculate offsets to get to things such as the
prog_data and the key, it's much more user-friendly if there are just
pointers.  Also, it's a bit more fool-proof.

While we're at it, we rework the pipeline cache API to use the
brw_stage_prog_data type directly.

Signed-off-by: Jason Ekstrand 
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98012
Reviewed-by: Topi Pohjolainen 
Reviewed-by: Timothy Arceri 
Cc: "13.0" 
(cherry picked from commit ff3185e3ba85b3635a1f645e8e951954f4022afe)

anv: move to using vk_alloc helpers.

2016-10-18T23:05:26+00:00

This moves all the alloc/free in anv to the generic helpers.

Acked-by: Jason Ekstrand 
Signed-off-by: Dave Airlie

anv: Get rid of anv_cmd_buffer_emit_state_base_address

2016-10-18T00:41:35+00:00

All code that would have once called this can now call the gen-specific
version.  The switching version is no longer needed.

Signed-off-by: Jason Ekstrand 
Reviewed-by: Anuj Phogat

anv/cmd_buffer: Move descriptor flushing into genX_cmd_buffer.c

2016-10-18T00:41:35+00:00

It really should have gone here all along.  We were trying a bit too hard
to make it gen-agnostic just because it didn't have any #if's.

Signed-off-by: Jason Ekstrand 
Reviewed-by: Anuj Phogat

anv/cmd_buffer: Expose ensure_push_constant_*

2016-10-18T00:41:35+00:00

Signed-off-by: Jason Ekstrand 
Reviewed-by: Anuj Phogat

anv/cmd_buffer: Move Begin/End/Execute to genX_cmd_buffer.c

2016-10-18T00:41:35+00:00

vkBeginCommandBuffer and vkCmdExecuteCommands both call into the
gen-specific emit_state_base_address function and vkEndCommandBuffer
belongs with begin.

Signed-off-by: Jason Ekstrand 
Reviewed-by: Anuj Phogat

anv/cmd_buffer: Move state base address re-emit into ExecuteCommands

2016-10-18T00:41:35+00:00

This has two primary advantages.  First, it means that the batch_chain code
knows less about the actual command buffer contents which is good because
improves separation.  Second, it means that it only gets re-emitted once
after all of the secondaries instead of once after each secondary which is
just wasteful.  It also has the advantage of cleaning the code up a bit.

Signed-off-by: Jason Ekstrand 
Reviewed-by: Anuj Phogat

anv/cmd_buffer: Don't call set_subpass in a secondary

2016-10-06T23:52:31+00:00

Initially, we had intended set_subpass to be an interesting function that
did whatever (presumably a lot) setup we needed for a subpass.  In reality,
it just sets a pointer and a dirty bit and then emits depth and stencil
state.  When we call BeginCommandBuffer on a secondary, there's no point in
setting depth and stencil state since it will already be set by the
primary.  Instead, the only thing we need to do at the start of a secondary
is set the subpass pointer and the dirty bit.

Signed-off-by: Jason Ekstrand 
Reviewed-by: Nanley Chery

anv: Rework pipeline caching

2016-08-30T22:08:23+00:00

The original pipeline cache the Kristian wrote was based on a now-false
premise that the shaders can be stored in the pipeline cache.  The Vulkan
1.0 spec explicitly states that the pipeline cache object is transiant and
you are allowed to delete it after using it to create a pipeline with no
ill effects.  As nice as Kristian's design was, it doesn't jive with the
expectation provided by the Vulkan spec.

The new pipeline cache uses reference-counted anv_shader_bin objects that
are backed by a large state pool.  The cache itself is just a hash table
mapping keys hashes to anv_shader_bin objects.  This has the added
advantage of removing one more hand-rolled hash table from mesa.

Signed-off-by: Jason Ekstrand 
Cc: "12.0" 
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97476
Acked-by: Kristian Høgsberg Kristensen