SetPushConstants support ?

[hzqst]

would be useful on small amount of data that need to be changed on every draw

[TheMostDiligent]

The main question here is what to do on other backends

[hzqst]

as for OpenGL / GLES, what about fallback to ordinary uniform buffer ?

idk if : register(b1, space0) { sucks when being converted to glsl via HLSL2GLSLConverter. I am using spirv-cross to emit glsl instead of HLSL2GLSLConverter though. it won't be a problem to me.

[TheMostDiligent]

The main problem is how to make the data available to the shader. In Vulkan/D3D12 it is set directly in the command buffer, but in other backends it will require some shenanigans with buffers. Given that generally there may be multiple push constants, that is quite non-trivial.

[hzqst]

I'm trying to add SetPushConstants to DiligentCore:

master...hzqst:DiligentCore:pushconstants

the main problems are:

1. How to get correct RootParameterIndex for SetGraphicsRoot32BitConstants?
2. We should probably leave SetPushConstants in D3D11/GL UNSUPPORTED("SetPushConstants is not supported in DirectX 11/OpenGL"); ?

Wicked has something like this and NVRHI has this however I don't see anything similar in Diligent

[TheMostDiligent]

Some comments:

• SetPushConstants method should be part of the IShaderResourceBinding as all resources are set through SRB
• Pipeline resource signatures need to support push constants as they are part of the pipeline layout
  • SHADER_RESOURCE_TYPE_32_BIT_CONSTANTS is added
  • Push constants are reflected from the shaders and added to the signature
  • They can also be added manually to the signature
• In D3D11/GL/WebGPU/Metal they should be somehow emulated through buffers. It will be really inconvenient if the feature is only supported in 2 of 6 backends.

[hzqst]

Some comments:

• SetPushConstants method should be part of the IShaderResourceBinding as all resources are set through SRB

• Pipeline resource signatures need to support push constants as they are part of the pipeline layout

  • SHADER_RESOURCE_TYPE_32_BIT_CONSTANTS is added
  • Push constants are reflected from the shaders and added to the signature
  • They can also be added manually to the signature

• In D3D11/GL/WebGPU/Metal they should be somehow emulated through buffers. It will be really inconvenient if the feature is only supported in 2 of 6 backends.

Some more questions need to be concerned carefully in D3D12:

Storage location for SetPushConstants data

1. Store in PipelineResourceSignatureD3D12Impl (similar to m_RootParams)
2. Store in ShaderResourceBindingD3D12Impl
3. Store in ShaderResourceCacheD3D12

Root parameter allocation for Push Constants

1. Allocate as a separate D3D12_ROOT_PARAMETER_TYPE_32BIT_CONSTANTS root parameter
2. Do not occupy root parameter slots (since 32-bit constants are inlined directly in the root signature)

SetPushConstants call timing/behavior

1. Immediately update and commit to command list each time it's called
2. Cache the data and submit during CommitRootTables
3. Independent submission that can be called before or after CommitRootTables

[hzqst]

SetPushConstants for D3D12 added (pretty buggy though): master...hzqst:DiligentCore:D3D12_SetPushConstants

further investigation needed:

D3D12 ERROR: ID3D12Device::CreateGraphicsPipelineState: Root Signature doesn't match Vertex Shader: Shader CBV descriptor range (BaseShaderRegister=52685, NumDescriptors=1, RegisterSpace=205) is not fully bound in root signature
 [ STATE_CREATION ERROR #688: CREATEGRAPHICSPIPELINESTATE_VS_ROOT_SIGNATURE_MISMATCH]

sample PSO creation:

    PipelineResourceDesc PipelineResources[1];
    PipelineResources[0].Name = "Constants";
    PipelineResources[0].ShaderStages = SHADER_TYPE_VERTEX;
    PipelineResources[0].ArraySize = sizeof(float4x4) / 4;
    PipelineResources[0].ResourceType = SHADER_RESOURCE_TYPE_32_BIT_CONSTANTS;
    PipelineResources[0].VarType      = SHADER_RESOURCE_VARIABLE_TYPE_STATIC;
    Uint32 NumPipelineResources       = 1;

   Diligent::PipelineResourceSignatureDesc PRSDesc;
    PRSDesc.UseCombinedTextureSamplers = true;
    PRSDesc.Resources                  = PipelineResources;
    PRSDesc.NumResources               = NumPipelineResources;
    //PRSDesc.ImmutableSamplers          = ImtblSamplers;
    //PRSDesc.NumImmutableSamplers       = NumImtblSamplers;

    m_pDevice->CreatePipelineResourceSignature(PRSDesc, &m_pPRS);

    IPipelineResourceSignature* ppSignatures[]{m_pPRS};

    PSOCreateInfo.ppResourceSignatures    = ppSignatures;
    PSOCreateInfo.ResourceSignaturesCount = _countof(ppSignatures);

    m_pDevice->CreateGraphicsPipelineState(PSOCreateInfo, &m_pPSO);

    // Since we did not explicitly specify the type for 'Constants' variable, default
    // type (SHADER_RESOURCE_VARIABLE_TYPE_STATIC) will be used. Static variables never
    // change and are bound directly through the pipeline state object.
    //m_pPSO->GetStaticVariableByName(SHADER_TYPE_VERTEX, "Constants")->Set(m_VSConstants);

    // Create a shader resource binding object and bind all static resources in it
    m_pPRS->CreateShaderResourceBinding(&m_pSRB, true);