mth/ebiten
1
0
Fork 0
mirror of https://github.com/hajimehoshi/ebiten.git synced 2024-12-26 03:38:55 +01:00
Code Issues Packages Projects Releases Wiki Activity

internal/graphicsdriver/directx: bug fix: wrong offsets

Browse Source 
* Wrong offsets were speicified when creating a constant buffer view and
  a shader resource view.
* Wrong root descriptor tables were speicified. For one draw command,
  one descriptor table for a constant buffer and textures should be used.

Updates #2198
Closes #2201
This commit is contained in:
Hajime Hoshi 2022-07-14 02:41:44 +09:00
parent 42d5d91829
commit b3267a7126
2 changed files with 10 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
internal/graphicsdriver/directx/graphics_windows.go
View File

@ -758,7 +758,7 @@ func (g *Graphics) End(present bool) error {
// Release vertices and indices buffers when too many ones were created. // Release vertices and indices buffers when too many ones were created.
// This is needed espciallly for testings, where present is always false. // This is needed espciallly for testings, where present is always false.
if len(g.vertices[g.frameIndex]) >= 16 { if len(g.vertices[g.frameIndex]) >= numDescriptorsPerFrame {
if err := g.waitForCommandQueue(); err != nil { if err := g.waitForCommandQueue(); err != nil {
return err return err
} }

17
internal/graphicsdriver/directx/pipeline_windows.go
View File

@ -23,7 +23,7 @@ import (
"github.com/hajimehoshi/ebiten/v2/internal/graphicsdriver" "github.com/hajimehoshi/ebiten/v2/internal/graphicsdriver"
) )
const numDescriptorsPerFrame = 256 const numDescriptorsPerFrame = 16
func operationToBlend(c graphicsdriver.Operation, alpha bool) _D3D12_BLEND { func operationToBlend(c graphicsdriver.Operation, alpha bool) _D3D12_BLEND {
switch c { switch c {
@ -402,7 +402,8 @@ func (p *pipelineStates) useGraphicsPipelineState(device *_ID3D12Device, command
if err != nil { if err != nil {
return err return err
} }
h.Offset(int32(frameIndex*numDescriptorsPerFrame+numConstantBufferAndSourceTextures*idx), p.shaderDescriptorSize) offset := int32(numConstantBufferAndSourceTextures * (frameIndex*numDescriptorsPerFrame + idx))
h.Offset(offset, p.shaderDescriptorSize)
device.CreateConstantBufferView(&_D3D12_CONSTANT_BUFFER_VIEW_DESC{ device.CreateConstantBufferView(&_D3D12_CONSTANT_BUFFER_VIEW_DESC{
BufferLocation: cb.GetGPUVirtualAddress(), BufferLocation: cb.GetGPUVirtualAddress(),
SizeInBytes: bufferSize, SizeInBytes: bufferSize,
@ -422,7 +423,8 @@ func (p *pipelineStates) useGraphicsPipelineState(device *_ID3D12Device, command
if err != nil { if err != nil {
return err return err
} }
h.Offset(int32(frameIndex*numDescriptorsPerFrame+numConstantBufferAndSourceTextures*idx), p.shaderDescriptorSize) offset := int32(numConstantBufferAndSourceTextures * (frameIndex*numDescriptorsPerFrame + idx))
h.Offset(offset, p.shaderDescriptorSize)
for _, src := range srcs { for _, src := range srcs {
h.Offset(1, p.shaderDescriptorSize) h.Offset(1, p.shaderDescriptorSize)
if src == nil { if src == nil {
@ -459,9 +461,8 @@ func (p *pipelineStates) useGraphicsPipelineState(device *_ID3D12Device, command
if err != nil { if err != nil {
return err return err
} }
gh.Offset(int32(frameIndex*numDescriptorsPerFrame+numConstantBufferAndSourceTextures*idx), p.shaderDescriptorSize) gh.Offset(offset, p.shaderDescriptorSize)
commandList.SetGraphicsRootDescriptorTable(0, gh) commandList.SetGraphicsRootDescriptorTable(0, gh)
gh.Offset(1, p.shaderDescriptorSize)
commandList.SetGraphicsRootDescriptorTable(1, gh) commandList.SetGraphicsRootDescriptorTable(1, gh)
sh, err := p.samplerDescriptorHeap.GetGPUDescriptorHandleForHeapStart() sh, err := p.samplerDescriptorHeap.GetGPUDescriptorHandleForHeapStart()
if err != nil { if err != nil {
@ -482,21 +483,21 @@ func (p *pipelineStates) ensureRootSignature(device *_ID3D12Device) (rootSignatu
NumDescriptors: 1, NumDescriptors: 1,
BaseShaderRegister: 0, BaseShaderRegister: 0,
RegisterSpace: 0, RegisterSpace: 0,
OffsetInDescriptorsFromTableStart: _D3D12_DESCRIPTOR_RANGE_OFFSET_APPEND, OffsetInDescriptorsFromTableStart: 0,
} }
srv := _D3D12_DESCRIPTOR_RANGE{ srv := _D3D12_DESCRIPTOR_RANGE{
RangeType: _D3D12_DESCRIPTOR_RANGE_TYPE_SRV, // t0 RangeType: _D3D12_DESCRIPTOR_RANGE_TYPE_SRV, // t0
NumDescriptors: graphics.ShaderImageCount, NumDescriptors: graphics.ShaderImageCount,
BaseShaderRegister: 0, BaseShaderRegister: 0,
RegisterSpace: 0, RegisterSpace: 0,
OffsetInDescriptorsFromTableStart: _D3D12_DESCRIPTOR_RANGE_OFFSET_APPEND, OffsetInDescriptorsFromTableStart: 1,
} }
sampler := _D3D12_DESCRIPTOR_RANGE{ sampler := _D3D12_DESCRIPTOR_RANGE{
RangeType: _D3D12_DESCRIPTOR_RANGE_TYPE_SAMPLER, // s0 RangeType: _D3D12_DESCRIPTOR_RANGE_TYPE_SAMPLER, // s0
NumDescriptors: 1, NumDescriptors: 1,
BaseShaderRegister: 0, BaseShaderRegister: 0,
RegisterSpace: 0, RegisterSpace: 0,
OffsetInDescriptorsFromTableStart: _D3D12_DESCRIPTOR_RANGE_OFFSET_APPEND, OffsetInDescriptorsFromTableStart: 0,
} }
rootParams := [...]_D3D12_ROOT_PARAMETER{ rootParams := [...]_D3D12_ROOT_PARAMETER{