Fix texture blit off-by-one errors (#2335)

This commit is contained in:
gdkchan 2021-06-02 20:30:48 -03:00 committed by GitHub
parent 534a45d6e8
commit b84ba43406
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 27 additions and 4 deletions

View file

@ -24,11 +24,29 @@ namespace Ryujinx.Graphics.Gpu.Engine
var control = state.Get<CopyTextureControl>(MethodOffset.CopyTextureControl); var control = state.Get<CopyTextureControl>(MethodOffset.CopyTextureControl);
int srcX1 = (int)(region.SrcXF >> 32); bool originCorner = control.UnpackOriginCorner();
int srcY1 = (int)(region.SrcYF >> 32);
int srcX2 = (int)((region.SrcXF + region.SrcWidthRF * region.DstWidth) >> 32); long srcX = region.SrcXF;
int srcY2 = (int)((region.SrcYF + region.SrcHeightRF * region.DstHeight) >> 32); long srcY = region.SrcYF;
if (originCorner)
{
// If the origin is corner, it is assumed that the guest API
// is manually centering the origin by adding a offset to the
// source region X/Y coordinates.
// Here we attempt to remove such offset to ensure we have the correct region.
// The offset is calculated as FactorXY / 2.0, where FactorXY = SrcXY / DstXY,
// so we do the same here by dividing the fixed point value by 2, while
// throwing away the fractional part to avoid rounding errors.
srcX -= (region.SrcWidthRF >> 33) << 32;
srcY -= (region.SrcHeightRF >> 33) << 32;
}
int srcX1 = (int)(srcX >> 32);
int srcY1 = (int)(srcY >> 32);
int srcX2 = srcX1 + (int)((region.SrcWidthRF * region.DstWidth + uint.MaxValue) >> 32);
int srcY2 = srcY1 + (int)((region.SrcHeightRF * region.DstHeight + uint.MaxValue) >> 32);
int dstX1 = region.DstX; int dstX1 = region.DstX;
int dstY1 = region.DstY; int dstY1 = region.DstY;

View file

@ -9,6 +9,11 @@ namespace Ryujinx.Graphics.Gpu.State
public uint Packed; public uint Packed;
#pragma warning restore CS0649 #pragma warning restore CS0649
public bool UnpackOriginCorner()
{
return (Packed & 1u) != 0;
}
public bool UnpackLinearFilter() public bool UnpackLinearFilter()
{ {
return (Packed & (1u << 4)) != 0; return (Packed & (1u << 4)) != 0;