mirror of
https://github.com/Ryujinx/Ryujinx.git
synced 2024-09-10 09:48:51 +02:00
Improve Buffer Textures and flush Image Stores (#2088)
* Improve Buffer Textures and flush Image Stores Fixes a number of issues with buffer textures: - Reworked Buffer Textures to create their buffers in the TextureManager, then bind them with the BufferManager later. - Fixes an issue where a buffer texture's buffer could be invalidated after it is bound, but before use. - Fixed width unpacking for large buffer textures. The width is now 32-bit rather than 16. - Force buffer textures to be rebound whenever any buffer is created, as using the handle id wasn't reliable, and the cost of binding isn't too high. Fixes vertex explosions and flickering animations in UE4 games. * Set ImageStore flag... for ImageStore. * Check the offset and size.
This commit is contained in:
parent
da283ff3c3
commit
1623ab524f
12 changed files with 154 additions and 30 deletions
|
@ -141,8 +141,8 @@ namespace Ryujinx.Graphics.Gpu.Engine
|
||||||
|
|
||||||
TextureManager.SetComputeImages(imageBindings);
|
TextureManager.SetComputeImages(imageBindings);
|
||||||
|
|
||||||
BufferManager.CommitComputeBindings();
|
|
||||||
TextureManager.CommitComputeBindings();
|
TextureManager.CommitComputeBindings();
|
||||||
|
BufferManager.CommitComputeBindings();
|
||||||
|
|
||||||
_context.Renderer.Pipeline.DispatchCompute(
|
_context.Renderer.Pipeline.DispatchCompute(
|
||||||
qmd.CtaRasterWidth,
|
qmd.CtaRasterWidth,
|
||||||
|
|
|
@ -304,8 +304,8 @@ namespace Ryujinx.Graphics.Gpu.Engine
|
||||||
{
|
{
|
||||||
UpdateStorageBuffers();
|
UpdateStorageBuffers();
|
||||||
|
|
||||||
BufferManager.CommitGraphicsBindings();
|
|
||||||
TextureManager.CommitGraphicsBindings();
|
TextureManager.CommitGraphicsBindings();
|
||||||
|
BufferManager.CommitGraphicsBindings();
|
||||||
}
|
}
|
||||||
|
|
||||||
/// <summary>
|
/// <summary>
|
||||||
|
|
|
@ -303,7 +303,7 @@ namespace Ryujinx.Graphics.Gpu.Image
|
||||||
// Ensure that the buffer texture is using the correct buffer as storage.
|
// Ensure that the buffer texture is using the correct buffer as storage.
|
||||||
// Buffers are frequently re-created to accomodate larger data, so we need to re-bind
|
// Buffers are frequently re-created to accomodate larger data, so we need to re-bind
|
||||||
// to ensure we're not using a old buffer that was already deleted.
|
// to ensure we're not using a old buffer that was already deleted.
|
||||||
_context.Methods.BufferManager.SetBufferTextureStorage(hostTexture, texture.Range.GetSubRange(0).Address, texture.Size, _isCompute);
|
_context.Methods.BufferManager.SetBufferTextureStorage(hostTexture, texture.Range.GetSubRange(0).Address, texture.Size, bindingInfo, bindingInfo.Format, false);
|
||||||
}
|
}
|
||||||
|
|
||||||
Sampler sampler = _samplerPool.Get(samplerId);
|
Sampler sampler = _samplerPool.Get(samplerId);
|
||||||
|
@ -349,12 +349,26 @@ namespace Ryujinx.Graphics.Gpu.Image
|
||||||
|
|
||||||
ITexture hostTexture = texture?.GetTargetTexture(bindingInfo.Target);
|
ITexture hostTexture = texture?.GetTargetTexture(bindingInfo.Target);
|
||||||
|
|
||||||
|
bool isStore = bindingInfo.Flags.HasFlag(TextureUsageFlags.ImageStore);
|
||||||
|
|
||||||
if (hostTexture != null && texture.Target == Target.TextureBuffer)
|
if (hostTexture != null && texture.Target == Target.TextureBuffer)
|
||||||
{
|
{
|
||||||
// Ensure that the buffer texture is using the correct buffer as storage.
|
// Ensure that the buffer texture is using the correct buffer as storage.
|
||||||
// Buffers are frequently re-created to accomodate larger data, so we need to re-bind
|
// Buffers are frequently re-created to accomodate larger data, so we need to re-bind
|
||||||
// to ensure we're not using a old buffer that was already deleted.
|
// to ensure we're not using a old buffer that was already deleted.
|
||||||
_context.Methods.BufferManager.SetBufferTextureStorage(hostTexture, texture.Range.GetSubRange(0).Address, texture.Size, _isCompute);
|
|
||||||
|
Format format = bindingInfo.Format;
|
||||||
|
|
||||||
|
if (format == 0 && texture != null)
|
||||||
|
{
|
||||||
|
format = texture.Format;
|
||||||
|
}
|
||||||
|
|
||||||
|
_context.Methods.BufferManager.SetBufferTextureStorage(hostTexture, texture.Range.GetSubRange(0).Address, texture.Size, bindingInfo, format, true);
|
||||||
|
}
|
||||||
|
else if (isStore)
|
||||||
|
{
|
||||||
|
texture?.SignalModified();
|
||||||
}
|
}
|
||||||
|
|
||||||
if (_imageState[stageIndex][index].Texture != hostTexture || _rebind)
|
if (_imageState[stageIndex][index].Texture != hostTexture || _rebind)
|
||||||
|
|
|
@ -153,6 +153,15 @@ namespace Ryujinx.Graphics.Gpu.Image
|
||||||
return (int)(Word4 & 0xffff) + 1;
|
return (int)(Word4 & 0xffff) + 1;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// <summary>
|
||||||
|
/// Unpack the width of a buffer texture.
|
||||||
|
/// </summary>
|
||||||
|
/// <returns>The texture width</returns>
|
||||||
|
public int UnpackBufferTextureWidth()
|
||||||
|
{
|
||||||
|
return (int)((Word4 & 0xffff) | (Word3 << 16)) + 1;
|
||||||
|
}
|
||||||
|
|
||||||
/// <summary>
|
/// <summary>
|
||||||
/// Unpacks the texture sRGB format flag.
|
/// Unpacks the texture sRGB format flag.
|
||||||
/// </summary>
|
/// </summary>
|
||||||
|
|
|
@ -172,8 +172,6 @@ namespace Ryujinx.Graphics.Gpu.Image
|
||||||
/// <returns>The texture information</returns>
|
/// <returns>The texture information</returns>
|
||||||
private TextureInfo GetInfo(TextureDescriptor descriptor, out int layerSize)
|
private TextureInfo GetInfo(TextureDescriptor descriptor, out int layerSize)
|
||||||
{
|
{
|
||||||
int width = descriptor.UnpackWidth();
|
|
||||||
int height = descriptor.UnpackHeight();
|
|
||||||
int depthOrLayers = descriptor.UnpackDepth();
|
int depthOrLayers = descriptor.UnpackDepth();
|
||||||
int levels = descriptor.UnpackLevels();
|
int levels = descriptor.UnpackLevels();
|
||||||
|
|
||||||
|
@ -190,6 +188,9 @@ namespace Ryujinx.Graphics.Gpu.Image
|
||||||
|
|
||||||
Target target = descriptor.UnpackTextureTarget().Convert((samplesInX | samplesInY) != 1);
|
Target target = descriptor.UnpackTextureTarget().Convert((samplesInX | samplesInY) != 1);
|
||||||
|
|
||||||
|
int width = target == Target.TextureBuffer ? descriptor.UnpackBufferTextureWidth() : descriptor.UnpackWidth();
|
||||||
|
int height = descriptor.UnpackHeight();
|
||||||
|
|
||||||
// We use 2D targets for 1D textures as that makes texture cache
|
// We use 2D targets for 1D textures as that makes texture cache
|
||||||
// management easier. We don't know the target for render target
|
// management easier. We don't know the target for render target
|
||||||
// and copies, so those would normally use 2D targets, which are
|
// and copies, so those would normally use 2D targets, which are
|
||||||
|
|
|
@ -1,9 +1,11 @@
|
||||||
using Ryujinx.Common;
|
using Ryujinx.Common;
|
||||||
using Ryujinx.Graphics.GAL;
|
using Ryujinx.Graphics.GAL;
|
||||||
|
using Ryujinx.Graphics.Gpu.Image;
|
||||||
using Ryujinx.Graphics.Gpu.State;
|
using Ryujinx.Graphics.Gpu.State;
|
||||||
using Ryujinx.Graphics.Shader;
|
using Ryujinx.Graphics.Shader;
|
||||||
using Ryujinx.Memory.Range;
|
using Ryujinx.Memory.Range;
|
||||||
using System;
|
using System;
|
||||||
|
using System.Collections.Generic;
|
||||||
using System.Collections.ObjectModel;
|
using System.Collections.ObjectModel;
|
||||||
using System.Linq;
|
using System.Linq;
|
||||||
|
|
||||||
|
@ -31,6 +33,7 @@ namespace Ryujinx.Graphics.Gpu.Memory
|
||||||
private IndexBuffer _indexBuffer;
|
private IndexBuffer _indexBuffer;
|
||||||
private VertexBuffer[] _vertexBuffers;
|
private VertexBuffer[] _vertexBuffers;
|
||||||
private BufferBounds[] _transformFeedbackBuffers;
|
private BufferBounds[] _transformFeedbackBuffers;
|
||||||
|
private List<BufferTextureBinding> _bufferTextures;
|
||||||
|
|
||||||
/// <summary>
|
/// <summary>
|
||||||
/// Holds shader stage buffer state and binding information.
|
/// Holds shader stage buffer state and binding information.
|
||||||
|
@ -138,6 +141,8 @@ namespace Ryujinx.Graphics.Gpu.Memory
|
||||||
_gpStorageBuffers[index] = new BuffersPerStage(Constants.TotalGpStorageBuffers);
|
_gpStorageBuffers[index] = new BuffersPerStage(Constants.TotalGpStorageBuffers);
|
||||||
_gpUniformBuffers[index] = new BuffersPerStage(Constants.TotalGpUniformBuffers);
|
_gpUniformBuffers[index] = new BuffersPerStage(Constants.TotalGpUniformBuffers);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
_bufferTextures = new List<BufferTextureBinding>();
|
||||||
}
|
}
|
||||||
|
|
||||||
/// <summary>
|
/// <summary>
|
||||||
|
@ -620,10 +625,39 @@ namespace Ryujinx.Graphics.Gpu.Memory
|
||||||
|
|
||||||
_context.Renderer.Pipeline.SetUniformBuffers(uRanges);
|
_context.Renderer.Pipeline.SetUniformBuffers(uRanges);
|
||||||
|
|
||||||
|
CommitBufferTextureBindings();
|
||||||
|
|
||||||
// Force rebind after doing compute work.
|
// Force rebind after doing compute work.
|
||||||
_rebind = true;
|
_rebind = true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// <summary>
|
||||||
|
/// Commit any queued buffer texture bindings.
|
||||||
|
/// </summary>
|
||||||
|
private void CommitBufferTextureBindings()
|
||||||
|
{
|
||||||
|
if (_bufferTextures.Count > 0)
|
||||||
|
{
|
||||||
|
foreach (var binding in _bufferTextures)
|
||||||
|
{
|
||||||
|
binding.Texture.SetStorage(GetBufferRange(binding.Address, binding.Size, binding.BindingInfo.Flags.HasFlag(TextureUsageFlags.ImageStore)));
|
||||||
|
|
||||||
|
// The texture must be rebound to use the new storage if it was updated.
|
||||||
|
|
||||||
|
if (binding.IsImage)
|
||||||
|
{
|
||||||
|
_context.Renderer.Pipeline.SetImage(binding.BindingInfo.Binding, binding.Texture, binding.Format);
|
||||||
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
_context.Renderer.Pipeline.SetTexture(binding.BindingInfo.Binding, binding.Texture);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
_bufferTextures.Clear();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
/// <summary>
|
/// <summary>
|
||||||
/// Ensures that the graphics engine bindings are visible to the host GPU.
|
/// Ensures that the graphics engine bindings are visible to the host GPU.
|
||||||
/// Note: this actually performs the binding using the host graphics API.
|
/// Note: this actually performs the binding using the host graphics API.
|
||||||
|
@ -743,6 +777,8 @@ namespace Ryujinx.Graphics.Gpu.Memory
|
||||||
UpdateBuffers(_gpUniformBuffers);
|
UpdateBuffers(_gpUniformBuffers);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
CommitBufferTextureBindings();
|
||||||
|
|
||||||
_rebind = false;
|
_rebind = false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -813,31 +849,19 @@ namespace Ryujinx.Graphics.Gpu.Memory
|
||||||
}
|
}
|
||||||
|
|
||||||
/// <summary>
|
/// <summary>
|
||||||
/// Sets the buffer storage of a buffer texture.
|
/// Sets the buffer storage of a buffer texture. This will be bound when the buffer manager commits bindings.
|
||||||
/// </summary>
|
/// </summary>
|
||||||
/// <param name="texture">Buffer texture</param>
|
/// <param name="texture">Buffer texture</param>
|
||||||
/// <param name="address">Address of the buffer in memory</param>
|
/// <param name="address">Address of the buffer in memory</param>
|
||||||
/// <param name="size">Size of the buffer in bytes</param>
|
/// <param name="size">Size of the buffer in bytes</param>
|
||||||
/// <param name="compute">Indicates if the buffer texture belongs to the compute or graphics pipeline</param>
|
/// <param name="bindingInfo">Binding info for the buffer texture</param>
|
||||||
public void SetBufferTextureStorage(ITexture texture, ulong address, ulong size, bool compute)
|
/// <param name="format">Format of the buffer texture</param>
|
||||||
|
/// <param name="isImage">Whether the binding is for an image or a sampler</param>
|
||||||
|
public void SetBufferTextureStorage(ITexture texture, ulong address, ulong size, TextureBindingInfo bindingInfo, Format format, bool isImage)
|
||||||
{
|
{
|
||||||
CreateBuffer(address, size);
|
CreateBuffer(address, size);
|
||||||
|
|
||||||
if (_rebind)
|
_bufferTextures.Add(new BufferTextureBinding(texture, address, size, bindingInfo, format, isImage));
|
||||||
{
|
|
||||||
// We probably had to modify existing buffers to create the texture buffer,
|
|
||||||
// so rebind everything to ensure we're using the new buffers for all bound resources.
|
|
||||||
if (compute)
|
|
||||||
{
|
|
||||||
CommitComputeBindings();
|
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
CommitGraphicsBindings();
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
texture.SetStorage(GetBufferRange(address, size));
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/// <summary>
|
/// <summary>
|
||||||
|
|
60
Ryujinx.Graphics.Gpu/Memory/BufferTextureBinding.cs
Normal file
60
Ryujinx.Graphics.Gpu/Memory/BufferTextureBinding.cs
Normal file
|
@ -0,0 +1,60 @@
|
||||||
|
using Ryujinx.Graphics.GAL;
|
||||||
|
using Ryujinx.Graphics.Gpu.Image;
|
||||||
|
|
||||||
|
namespace Ryujinx.Graphics.Gpu.Memory
|
||||||
|
{
|
||||||
|
/// <summary>
|
||||||
|
/// A buffer binding to apply to a buffer texture.
|
||||||
|
/// </summary>
|
||||||
|
struct BufferTextureBinding
|
||||||
|
{
|
||||||
|
/// <summary>
|
||||||
|
/// The buffer texture.
|
||||||
|
/// </summary>
|
||||||
|
public ITexture Texture { get; }
|
||||||
|
|
||||||
|
/// <summary>
|
||||||
|
/// The base address of the buffer binding.
|
||||||
|
/// </summary>
|
||||||
|
public ulong Address { get; }
|
||||||
|
|
||||||
|
/// <summary>
|
||||||
|
/// The size of the buffer binding in bytes.
|
||||||
|
/// </summary>
|
||||||
|
public ulong Size { get; }
|
||||||
|
|
||||||
|
/// <summary>
|
||||||
|
/// The image or sampler binding info for the buffer texture.
|
||||||
|
/// </summary>
|
||||||
|
public TextureBindingInfo BindingInfo { get; }
|
||||||
|
|
||||||
|
/// <summary>
|
||||||
|
/// The image format for the binding.
|
||||||
|
/// </summary>
|
||||||
|
public Format Format { get; }
|
||||||
|
|
||||||
|
/// <summary>
|
||||||
|
/// Whether the binding is for an image or a sampler.
|
||||||
|
/// </summary>
|
||||||
|
public bool IsImage { get; }
|
||||||
|
|
||||||
|
/// <summary>
|
||||||
|
/// Create a new buffer texture binding.
|
||||||
|
/// </summary>
|
||||||
|
/// <param name="texture">Buffer texture</param>
|
||||||
|
/// <param name="address">Base address</param>
|
||||||
|
/// <param name="size">Size in bytes</param>
|
||||||
|
/// <param name="bindingInfo">Binding info</param>
|
||||||
|
/// <param name="format">Binding format</param>
|
||||||
|
/// <param name="isImage">Whether the binding is for an image or a sampler</param>
|
||||||
|
public BufferTextureBinding(ITexture texture, ulong address, ulong size, TextureBindingInfo bindingInfo, Format format, bool isImage)
|
||||||
|
{
|
||||||
|
Texture = texture;
|
||||||
|
Address = address;
|
||||||
|
Size = size;
|
||||||
|
BindingInfo = bindingInfo;
|
||||||
|
Format = format;
|
||||||
|
IsImage = isImage;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
|
@ -35,7 +35,7 @@ namespace Ryujinx.Graphics.Gpu.Shader
|
||||||
/// <summary>
|
/// <summary>
|
||||||
/// Version of the codegen (to be changed when codegen or guest format change).
|
/// Version of the codegen (to be changed when codegen or guest format change).
|
||||||
/// </summary>
|
/// </summary>
|
||||||
private const ulong ShaderCodeGenVersion = 1961;
|
private const ulong ShaderCodeGenVersion = 2088;
|
||||||
|
|
||||||
// Progress reporting helpers
|
// Progress reporting helpers
|
||||||
private int _shaderCount;
|
private int _shaderCount;
|
||||||
|
|
|
@ -6,12 +6,17 @@ namespace Ryujinx.Graphics.OpenGL.Image
|
||||||
{
|
{
|
||||||
class TextureBuffer : TextureBase, ITexture
|
class TextureBuffer : TextureBase, ITexture
|
||||||
{
|
{
|
||||||
|
private Renderer _renderer;
|
||||||
private int _bufferOffset;
|
private int _bufferOffset;
|
||||||
private int _bufferSize;
|
private int _bufferSize;
|
||||||
|
private int _bufferCount;
|
||||||
|
|
||||||
private BufferHandle _buffer;
|
private BufferHandle _buffer;
|
||||||
|
|
||||||
public TextureBuffer(TextureCreateInfo info) : base(info) {}
|
public TextureBuffer(Renderer renderer, TextureCreateInfo info) : base(info)
|
||||||
|
{
|
||||||
|
_renderer = renderer;
|
||||||
|
}
|
||||||
|
|
||||||
public void CopyTo(ITexture destination, int firstLayer, int firstLevel)
|
public void CopyTo(ITexture destination, int firstLayer, int firstLevel)
|
||||||
{
|
{
|
||||||
|
@ -50,16 +55,19 @@ namespace Ryujinx.Graphics.OpenGL.Image
|
||||||
|
|
||||||
public void SetStorage(BufferRange buffer)
|
public void SetStorage(BufferRange buffer)
|
||||||
{
|
{
|
||||||
if (buffer.Handle == _buffer &&
|
if (_buffer != BufferHandle.Null &&
|
||||||
buffer.Offset == _bufferOffset &&
|
buffer.Offset == _bufferOffset &&
|
||||||
buffer.Size == _bufferSize)
|
buffer.Size == _bufferSize &&
|
||||||
|
_renderer.BufferCount == _bufferCount)
|
||||||
{
|
{
|
||||||
|
// Only rebind the buffer when more have been created.
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
_buffer = buffer.Handle;
|
_buffer = buffer.Handle;
|
||||||
_bufferOffset = buffer.Offset;
|
_bufferOffset = buffer.Offset;
|
||||||
_bufferSize = buffer.Size;
|
_bufferSize = buffer.Size;
|
||||||
|
_bufferCount = _renderer.BufferCount;
|
||||||
|
|
||||||
Bind(0);
|
Bind(0);
|
||||||
|
|
||||||
|
|
|
@ -30,6 +30,8 @@ namespace Ryujinx.Graphics.OpenGL
|
||||||
|
|
||||||
internal ResourcePool ResourcePool { get; }
|
internal ResourcePool ResourcePool { get; }
|
||||||
|
|
||||||
|
internal int BufferCount { get; private set; }
|
||||||
|
|
||||||
public string GpuVendor { get; private set; }
|
public string GpuVendor { get; private set; }
|
||||||
public string GpuRenderer { get; private set; }
|
public string GpuRenderer { get; private set; }
|
||||||
public string GpuVersion { get; private set; }
|
public string GpuVersion { get; private set; }
|
||||||
|
@ -52,6 +54,8 @@ namespace Ryujinx.Graphics.OpenGL
|
||||||
|
|
||||||
public BufferHandle CreateBuffer(int size)
|
public BufferHandle CreateBuffer(int size)
|
||||||
{
|
{
|
||||||
|
BufferCount++;
|
||||||
|
|
||||||
return Buffer.Create(size);
|
return Buffer.Create(size);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -69,7 +73,7 @@ namespace Ryujinx.Graphics.OpenGL
|
||||||
{
|
{
|
||||||
if (info.Target == Target.TextureBuffer)
|
if (info.Target == Target.TextureBuffer)
|
||||||
{
|
{
|
||||||
return new TextureBuffer(info);
|
return new TextureBuffer(this, info);
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
|
|
|
@ -111,6 +111,9 @@ namespace Ryujinx.Graphics.Shader.CodeGen.Glsl.Instructions
|
||||||
|
|
||||||
if (texOp.Inst == Instruction.ImageStore)
|
if (texOp.Inst == Instruction.ImageStore)
|
||||||
{
|
{
|
||||||
|
int texIndex = context.FindImageDescriptorIndex(texOp);
|
||||||
|
context.ImageDescriptors[texIndex] = context.ImageDescriptors[texIndex].SetFlag(TextureUsageFlags.ImageStore);
|
||||||
|
|
||||||
VariableType type = texOp.Format.GetComponentType();
|
VariableType type = texOp.Format.GetComponentType();
|
||||||
|
|
||||||
string[] cElems = new string[4];
|
string[] cElems = new string[4];
|
||||||
|
|
|
@ -12,6 +12,7 @@ namespace Ryujinx.Graphics.Shader
|
||||||
|
|
||||||
// Integer sampled textures must be noted for resolution scaling.
|
// Integer sampled textures must be noted for resolution scaling.
|
||||||
ResScaleUnsupported = 1 << 0,
|
ResScaleUnsupported = 1 << 0,
|
||||||
NeedsScaleValue = 1 << 1
|
NeedsScaleValue = 1 << 1,
|
||||||
|
ImageStore = 1 << 2
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue