From 327c200991620cb6df41f7c2a05336639e577389 Mon Sep 17 00:00:00 2001 From: Chuck Walbourn Date: Thu, 29 Aug 2024 14:13:48 -0700 Subject: [PATCH] Support loading legacy DDS files using mixed channel formats (#501) --- DirectXTex/DirectXTexDDS.cpp | 223 +++++++++++++++++++++++++++++------ 1 file changed, 186 insertions(+), 37 deletions(-) diff --git a/DirectXTex/DirectXTexDDS.cpp b/DirectXTex/DirectXTexDDS.cpp index 0daf9423..de5deb08 100644 --- a/DirectXTex/DirectXTexDDS.cpp +++ b/DirectXTex/DirectXTexDDS.cpp @@ -46,6 +46,9 @@ namespace CONV_FLAGS_L8 = 0x40000, // Source is a 8 luminance format CONV_FLAGS_L16 = 0x80000, // Source is a 16 luminance format CONV_FLAGS_A8L8 = 0x100000, // Source is a 8:8 luminance format + CONV_FLAGS_L6V5U5 = 0x200000, // Source is a 6:5:5 bumpluminance format + CONV_FLAGS_L8U8V8 = 0x400000, // Source is a X:8:8:8 bumpluminance format + CONV_FLAGS_WUV10 = 0x800000, // Source is a 2:10:10:10 bump format }; struct LegacyDDS @@ -141,7 +144,7 @@ namespace { DXGI_FORMAT_R8G8B8A8_UNORM, CONV_FLAGS_EXPAND | CONV_FLAGS_PAL8 - | CONV_FLAGS_A8P8, { sizeof(DDS_PIXELFORMAT), DDS_PAL8A, 0, 16, 0, 0, 0, 0 } }, // D3DFMT_A8P8 + | CONV_FLAGS_A8P8, { sizeof(DDS_PIXELFORMAT), DDS_PAL8A, 0, 16, 0, 0, 0, 0xff00 } }, // D3DFMT_A8P8 { DXGI_FORMAT_R8G8B8A8_UNORM, CONV_FLAGS_EXPAND | CONV_FLAGS_PAL8, { sizeof(DDS_PIXELFORMAT), DDS_PAL8, 0, 8, 0, 0, 0, 0 } }, // D3DFMT_P8 @@ -157,6 +160,11 @@ namespace { DXGI_FORMAT_R8G8_SNORM, CONV_FLAGS_NONE, DDSPF_V8U8 }, // D3DFMT_V8U8 { DXGI_FORMAT_R8G8B8A8_SNORM, CONV_FLAGS_NONE, DDSPF_Q8W8V8U8 }, // D3DFMT_Q8W8V8U8 { DXGI_FORMAT_R16G16_SNORM, CONV_FLAGS_NONE, DDSPF_V16U16 }, // D3DFMT_V16U16 + + { DXGI_FORMAT_R8G8B8A8_UNORM, CONV_FLAGS_L6V5U5 + | CONV_FLAGS_EXPAND, DDSPF_L6V5U5 }, // D3DFMT_L6V5U5 + { DXGI_FORMAT_R8G8B8A8_UNORM, CONV_FLAGS_L8U8V8, DDSPF_X8L8V8U8 }, // D3DFMT_X8L8V8U8 + { DXGI_FORMAT_R10G10B10A2_UNORM, CONV_FLAGS_WUV10, DDSPF_A2W10V10U10 }, // D3DFMT_A2W10V10U10 }; // Note that many common DDS reader/writers (including D3DX) swap the @@ -166,15 +174,14 @@ namespace // header extension and specify the DXGI_FORMAT_R10G10B10A2_UNORM format directly // We do not support the following legacy Direct3D 9 formats: - // BumpDuDv D3DFMT_A2W10V10U10 - // BumpLuminance D3DFMT_L6V5U5, D3DFMT_X8L8V8U8 - // FourCC 117 D3DFMT_CxV8U8 - // ZBuffer D3DFMT_D16_LOCKABLE + // D3DFMT_D16_LOCKABLE (DDPF_ZBUFFER: 0x00000400) // FourCC 82 D3DFMT_D32F_LOCKABLE + // FourCC 117 D3DFMT_CxV8U8 // We do not support the following known FourCC codes: // FourCC CTX1 (Xbox 360 only) // FourCC EAR, EARG, ET2, ET2A (Ericsson Texture Compression) + // FourCC MET1 (a.k.a. D3DFMT_MULTI2_ARGB8; rarely supported by any hardware) DXGI_FORMAT GetDXGIFormat(const DDS_HEADER& hdr, const DDS_PIXELFORMAT& ddpf, DDS_FLAGS flags, @@ -215,17 +222,16 @@ namespace if (ddpf.fourCC == entry->ddpf.fourCC) break; } - else if (ddpfFlags == entry->ddpf.flags) + else if ((ddpfFlags == entry->ddpf.flags) && (ddpf.RGBBitCount == entry->ddpf.RGBBitCount)) { if (entry->ddpf.flags & DDS_PAL8) { - if (ddpf.RGBBitCount == entry->ddpf.RGBBitCount) - break; + // PAL8 / PAL8A + break; } else if (entry->ddpf.flags & DDS_ALPHA) { - if (ddpf.RGBBitCount == entry->ddpf.RGBBitCount - && ddpf.ABitMask == entry->ddpf.ABitMask) + if (ddpf.ABitMask == entry->ddpf.ABitMask) break; } else if (entry->ddpf.flags & DDS_LUMINANCE) @@ -233,48 +239,53 @@ namespace if (entry->ddpf.flags & DDS_ALPHAPIXELS) { // LUMINANCEA - if (ddpf.RGBBitCount == entry->ddpf.RGBBitCount - && ddpf.RBitMask == entry->ddpf.RBitMask + if (ddpf.RBitMask == entry->ddpf.RBitMask && ddpf.ABitMask == entry->ddpf.ABitMask) break; } else { // LUMINANCE - if (ddpf.RGBBitCount == entry->ddpf.RGBBitCount - && ddpf.RBitMask == entry->ddpf.RBitMask) + if (ddpf.RBitMask == entry->ddpf.RBitMask) break; } } else if (entry->ddpf.flags & DDS_BUMPDUDV) - { - if (ddpf.RGBBitCount == entry->ddpf.RGBBitCount - && ddpf.RBitMask == entry->ddpf.RBitMask - && ddpf.GBitMask == entry->ddpf.GBitMask - && ddpf.BBitMask == entry->ddpf.BBitMask - && ddpf.ABitMask == entry->ddpf.ABitMask) - break; - } - else if (ddpf.RGBBitCount == entry->ddpf.RGBBitCount) { if (entry->ddpf.flags & DDS_ALPHAPIXELS) { - // RGBA + // BUMPDUDVA if (ddpf.RBitMask == entry->ddpf.RBitMask - && ddpf.GBitMask == entry->ddpf.GBitMask - && ddpf.BBitMask == entry->ddpf.BBitMask && ddpf.ABitMask == entry->ddpf.ABitMask) + { + flags &= ~DDS_FLAGS_NO_R10B10G10A2_FIXUP; break; + } } else { - // RGB - if (ddpf.RBitMask == entry->ddpf.RBitMask - && ddpf.GBitMask == entry->ddpf.GBitMask - && ddpf.BBitMask == entry->ddpf.BBitMask) + // BUMPDUDV + if (ddpf.RBitMask == entry->ddpf.RBitMask) break; } } + else if (entry->ddpf.flags & DDS_ALPHAPIXELS) + { + // RGBA + if (ddpf.RBitMask == entry->ddpf.RBitMask + && ddpf.GBitMask == entry->ddpf.GBitMask + && ddpf.BBitMask == entry->ddpf.BBitMask + && ddpf.ABitMask == entry->ddpf.ABitMask) + break; + } + else + { + // RGB + if (ddpf.RBitMask == entry->ddpf.RBitMask + && ddpf.GBitMask == entry->ddpf.GBitMask + && ddpf.BBitMask == entry->ddpf.BBitMask) + break; + } } } } @@ -996,7 +1007,10 @@ namespace TEXP_LEGACY_B4G4R4A4, TEXP_LEGACY_L8, TEXP_LEGACY_L16, - TEXP_LEGACY_A8L8 + TEXP_LEGACY_A8L8, + TEXP_LEGACY_L6V5U5, + TEXP_LEGACY_X8L8V8U8, + TEXP_LEGACY_A2W10V10U10 }; constexpr TEXP_LEGACY_FORMAT FindLegacyFormat(uint32_t flags) noexcept @@ -1023,6 +1037,12 @@ namespace lformat = TEXP_LEGACY_L16; else if (flags & CONV_FLAGS_A8L8) lformat = TEXP_LEGACY_A8L8; + else if (flags & CONV_FLAGS_L6V5U5) + lformat = TEXP_LEGACY_L6V5U5; + else if (flags & CONV_FLAGS_L8U8V8) + lformat = TEXP_LEGACY_X8L8V8U8; + else if (flags & CONV_FLAGS_WUV10) + lformat = TEXP_LEGACY_A2W10V10U10; return lformat; } @@ -1328,11 +1348,120 @@ namespace } return false; + case TEXP_LEGACY_L6V5U5: + if (outFormat != DXGI_FORMAT_R8G8B8A8_UNORM) + return false; + + // D3DFMT_L6V5U5 -> DXGI_FORMAT_R8G8B8A8_UNORM (LUVA) + if (inSize >= 2 && outSize >= 4) + { + const uint16_t* __restrict sPtr = static_cast(pSource); + uint32_t * __restrict dPtr = static_cast(pDestination); + + for (size_t ocount = 0, icount = 0; ((icount < (inSize - 1)) && (ocount < (outSize - 3))); icount += 2, ocount += 4) + { + const uint16_t t = *(sPtr++); + + // Converts unsigned 6-bit/signed 5-bit/signed 5-bit bump luminance to 8:8:8:8 unsigned + uint32_t t1 = ((t & 0xFC00) >> 8) | ((t & 0xC000) >> 14); + + constexpr int m = 1U << 4; + int8_t v = ((((t >> 5) & 0x1f) ^ m) - m) + 16; + int8_t u = (((t & 0x1f) ^ m) - m) + 16; + + uint32_t t2 = u << 3 | u >> 2; + uint32_t t3 = v << 3 | v >> 2; + + *(dPtr++) = t1 | (t2 << 8) | (t3 << 16) | 0xff000000; + } + return true; + } + return false; + default: return false; } } + _Success_(return) + bool LegacyConvertScanline( + _Out_writes_bytes_(outSize) void* pDestination, + size_t outSize, + _In_ DXGI_FORMAT outFormat, + _In_reads_bytes_(inSize) const void* pSource, + size_t inSize, + _In_ TEXP_LEGACY_FORMAT inFormat, + uint32_t tflags) noexcept + { + assert(pDestination && outSize > 0); + assert(pSource && inSize > 0); + + switch (inFormat) + { + case TEXP_LEGACY_X8L8V8U8: + if (outFormat != DXGI_FORMAT_R8G8B8A8_UNORM) + return false; + + // D3DFMT_X8L8V8U8 -> DXGI_FORMAT_R8G8B8A8_UNORM (LUVA) + if (inSize >= 4 && outSize >= 4) + { + auto sPtr = static_cast(pSource); + auto dPtr = static_cast(pDestination); + + for (size_t ocount = 0, icount = 0; ((icount < (inSize - 3)) && (ocount < (outSize - 3))); icount += 4, ocount += 4) + { + const uint32_t t = *(sPtr++); + + // Converts 8-bit unsigned / 8-bit signed / 8-bit signed to 8:8:8:8 unsigned + uint32_t t1 = (t >> 16) & 0xff; + constexpr int m = 1U << 7; + uint32_t v = ((((t >> 8) & 0xff) ^ m) - m) + 128; + uint32_t u = (((t & 0xff) ^ m) - m) + 128; + + uint32_t t2 = u << 8; + uint32_t t3 = v << 16; + + *(dPtr++) = t1 | t2 | t3 | 0xff000000; + } + return true; + } + return false; + + case TEXP_LEGACY_A2W10V10U10: + if (outFormat != DXGI_FORMAT_R10G10B10A2_UNORM) + return false; + + // D3DFMT_A2W10V10U10 -> DXGI_FORMAT_R10G10B10A2_UNORM (UVWA) + if (inSize >= 4 && outSize >= 4) + { + auto sPtr = static_cast(pSource); + auto dPtr = static_cast(pDestination); + + for (size_t ocount = 0, icount = 0; ((icount < (inSize - 3)) && (ocount < (outSize - 3))); icount += 4, ocount += 4) + { + const uint32_t t = *(sPtr++); + + // Converts 2-bit unsigned / 10-bit signed / 10-bit signed / 10-bit signed to 2:10:10:10 unsigned + constexpr int m = 1U << 9; + uint32_t w = ((((t >> 20) & 0x3ff) ^ m) - m) + 512; + uint32_t v = ((((t >> 10) & 0x3ff) ^ m) - m) + 512; + uint32_t u = (((t & 0x3ff) ^ m) - m) + 512; + + uint32_t t1 = u; + uint32_t t2 = v << 10; + uint32_t t3 = w << 20; + uint32_t ta = (tflags & TEXP_SCANLINE_SETALPHA) ? 0xC0000000 : (t & 0xC0000000); + + *(dPtr++) = t1 | t2 | t3 | ta; + } + return true; + } + return false; + + default: + return false; + } + } //------------------------------------------------------------------------------------- // Converts or copies image data from pPixels into scratch image data @@ -1356,7 +1485,7 @@ namespace { if (convFlags & CONV_FLAGS_888) cpFlags |= CP_FLAGS_24BPP; - else if (convFlags & (CONV_FLAGS_565 | CONV_FLAGS_5551 | CONV_FLAGS_4444 | CONV_FLAGS_8332 | CONV_FLAGS_A8P8 | CONV_FLAGS_L16 | CONV_FLAGS_A8L8)) + else if (convFlags & (CONV_FLAGS_565 | CONV_FLAGS_5551 | CONV_FLAGS_4444 | CONV_FLAGS_8332 | CONV_FLAGS_A8P8 | CONV_FLAGS_L16 | CONV_FLAGS_A8L8 | CONV_FLAGS_L6V5U5)) cpFlags |= CP_FLAGS_16BPP; else if (convFlags & (CONV_FLAGS_44 | CONV_FLAGS_332 | CONV_FLAGS_PAL8 | CONV_FLAGS_L8)) cpFlags |= CP_FLAGS_8BPP; @@ -1502,13 +1631,18 @@ namespace } else if (convFlags & CONV_FLAGS_SWIZZLE) { - SwizzleScanline(pDest, dpitch, pSrc, spitch, - metadata.format, tflags); + SwizzleScanline(pDest, dpitch, pSrc, spitch, metadata.format, tflags); + } + else if (convFlags & (CONV_FLAGS_L8U8V8 | CONV_FLAGS_WUV10)) + { + const TEXP_LEGACY_FORMAT lformat = FindLegacyFormat(convFlags); + if (!LegacyConvertScanline(pDest, dpitch, metadata.format, + pSrc, spitch, lformat, tflags)) + return E_FAIL; } else { - CopyScanline(pDest, dpitch, pSrc, spitch, - metadata.format, tflags); + CopyScanline(pDest, dpitch, pSrc, spitch, metadata.format, tflags); } pSrc += spitch; @@ -1605,6 +1739,13 @@ namespace { SwizzleScanline(pDest, dpitch, pSrc, spitch, metadata.format, tflags); } + else if (convFlags & (CONV_FLAGS_L8U8V8 | CONV_FLAGS_WUV10)) + { + const TEXP_LEGACY_FORMAT lformat = FindLegacyFormat(convFlags); + if (!LegacyConvertScanline(pDest, dpitch, metadata.format, + pSrc, spitch, lformat, tflags)) + return E_FAIL; + } else { CopyScanline(pDest, dpitch, pSrc, spitch, metadata.format, tflags); @@ -1662,6 +1803,14 @@ namespace { SwizzleScanline(pPixels, rowPitch, pPixels, rowPitch, metadata.format, tflags); } + else if (convFlags & (CONV_FLAGS_L8U8V8 | CONV_FLAGS_WUV10)) + { + const TEXP_LEGACY_FORMAT lformat = FindLegacyFormat(convFlags); + if (!LegacyConvertScanline(pPixels, rowPitch, metadata.format, pPixels, rowPitch, lformat, tflags)) + { + return E_UNEXPECTED; + } + } else { CopyScanline(pPixels, rowPitch, pPixels, rowPitch, metadata.format, tflags); @@ -2133,7 +2282,7 @@ HRESULT DirectX::LoadFromDDSFileEx( } #endif - if (convFlags & (CONV_FLAGS_SWIZZLE | CONV_FLAGS_NOALPHA)) + if (convFlags & (CONV_FLAGS_SWIZZLE | CONV_FLAGS_NOALPHA | CONV_FLAGS_L8U8V8 | CONV_FLAGS_WUV10)) { // Swizzle/copy image in place hr = CopyImageInPlace(convFlags, image);