diff options
author | Jason Ekstrand <[email protected]> | 2016-11-12 08:34:33 -0800 |
---|---|---|
committer | Jason Ekstrand <[email protected]> | 2016-11-16 10:08:42 -0800 |
commit | 23d1919fe3a5f834d0c522cf8dbb63ac9bb26a11 (patch) | |
tree | 6dc0809e46d4a020fd7ce0c46c70ea05c0ea0f60 /src/intel/vulkan/gen8_pipeline.c | |
parent | bda247d3fd70870da2e71c254821c59aa1751d0f (diff) |
anv/pipeline: Stop claiming to support running without a vertex shader
From the Vulkan spec version 1.0.32 docs for vkCreateGraphicsPipelines:
The stage member of one element of pStages must be
VK_SHADER_STAGE_VERTEX_BIT
Since a vertex shader is always required, this hasn't been used since we
deleted meta. Let's get rid of the complexity.
Reviewed-by: Timothy Arceri <[email protected]>
Reviewed-by: Kristian H. Kristensen <[email protected]>
Diffstat (limited to 'src/intel/vulkan/gen8_pipeline.c')
-rw-r--r-- | src/intel/vulkan/gen8_pipeline.c | 98 |
1 files changed, 44 insertions, 54 deletions
diff --git a/src/intel/vulkan/gen8_pipeline.c b/src/intel/vulkan/gen8_pipeline.c index 20fd4ba0f9b..40cfe3f4a18 100644 --- a/src/intel/vulkan/gen8_pipeline.c +++ b/src/intel/vulkan/gen8_pipeline.c @@ -177,60 +177,50 @@ genX(graphics_pipeline_create)( offset = 1; length = (vs_prog_data->base.vue_map.num_slots + 1) / 2 - offset; - if (!anv_pipeline_has_stage(pipeline, MESA_SHADER_VERTEX)) { - anv_batch_emit(&pipeline->batch, GENX(3DSTATE_VS), vs) { - vs.FunctionEnable = false; - /* Even if VS is disabled, SBE still gets the amount of - * vertex data to read from this field. */ - vs.VertexURBEntryOutputReadOffset = offset; - vs.VertexURBEntryOutputLength = length; - } - } else { - const struct anv_shader_bin *vs_bin = - pipeline->shaders[MESA_SHADER_VERTEX]; - - anv_batch_emit(&pipeline->batch, GENX(3DSTATE_VS), vs) { - vs.KernelStartPointer = vs_bin->kernel.offset; - vs.SingleVertexDispatch = false; - vs.VectorMaskEnable = false; - - vs.SamplerCount = get_sampler_count(vs_bin); - vs.BindingTableEntryCount = get_binding_table_entry_count(vs_bin); - - vs.ThreadDispatchPriority = false; - vs.FloatingPointMode = IEEE754; - vs.IllegalOpcodeExceptionEnable = false; - vs.AccessesUAV = false; - vs.SoftwareExceptionEnable = false; - - vs.ScratchSpaceBasePointer = (struct anv_address) { - .bo = anv_scratch_pool_alloc(device, &device->scratch_pool, - MESA_SHADER_VERTEX, - vs_prog_data->base.base.total_scratch), - .offset = 0, - }; - vs.PerThreadScratchSpace = scratch_space(&vs_prog_data->base.base); - - vs.DispatchGRFStartRegisterForURBData = - vs_prog_data->base.base.dispatch_grf_start_reg; - - vs.VertexURBEntryReadLength = vs_prog_data->base.urb_read_length; - vs.VertexURBEntryReadOffset = 0; - - vs.MaximumNumberofThreads = devinfo->max_vs_threads - 1; - vs.StatisticsEnable = false; - vs.SIMD8DispatchEnable = - vs_prog_data->base.dispatch_mode == DISPATCH_MODE_SIMD8; - vs.VertexCacheDisable = false; - vs.FunctionEnable = true; - - vs.VertexURBEntryOutputReadOffset = offset; - vs.VertexURBEntryOutputLength = length; - - /* TODO */ - vs.UserClipDistanceClipTestEnableBitmask = 0; - vs.UserClipDistanceCullTestEnableBitmask = 0; - } + assert(anv_pipeline_has_stage(pipeline, MESA_SHADER_VERTEX)); + const struct anv_shader_bin *vs_bin = + pipeline->shaders[MESA_SHADER_VERTEX]; + anv_batch_emit(&pipeline->batch, GENX(3DSTATE_VS), vs) { + vs.KernelStartPointer = vs_bin->kernel.offset; + vs.SingleVertexDispatch = false; + vs.VectorMaskEnable = false; + + vs.SamplerCount = get_sampler_count(vs_bin); + vs.BindingTableEntryCount = get_binding_table_entry_count(vs_bin); + + vs.ThreadDispatchPriority = false; + vs.FloatingPointMode = IEEE754; + vs.IllegalOpcodeExceptionEnable = false; + vs.AccessesUAV = false; + vs.SoftwareExceptionEnable = false; + + vs.ScratchSpaceBasePointer = (struct anv_address) { + .bo = anv_scratch_pool_alloc(device, &device->scratch_pool, + MESA_SHADER_VERTEX, + vs_prog_data->base.base.total_scratch), + .offset = 0, + }; + vs.PerThreadScratchSpace = scratch_space(&vs_prog_data->base.base); + + vs.DispatchGRFStartRegisterForURBData = + vs_prog_data->base.base.dispatch_grf_start_reg; + + vs.VertexURBEntryReadLength = vs_prog_data->base.urb_read_length; + vs.VertexURBEntryReadOffset = 0; + + vs.MaximumNumberofThreads = devinfo->max_vs_threads - 1; + vs.StatisticsEnable = false; + vs.SIMD8DispatchEnable = + vs_prog_data->base.dispatch_mode == DISPATCH_MODE_SIMD8; + vs.VertexCacheDisable = false; + vs.FunctionEnable = true; + + vs.VertexURBEntryOutputReadOffset = offset; + vs.VertexURBEntryOutputLength = length; + + /* TODO */ + vs.UserClipDistanceClipTestEnableBitmask = 0; + vs.UserClipDistanceCullTestEnableBitmask = 0; } const int num_thread_bias = GEN_GEN == 8 ? 2 : 1; |