Add functions to support Standard Swizzle textures - #515
Conversation
|
@microsoft-github-policy-service agree company="Microsoft" |
|
Note for testing, I'll add coverage to the test suite which will likely just do a swizzle and deswizzle and make sure the bits match. As for determining if the swizzle is correct, I probably need to make a test app that renders using WARP which supports standard swizzle. |
Chuck Walbourn (walbourn)
changed the title
13baea9 to
09ff9e8
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Chuck Walbourn (walbourn)
changed the title
- needs to be tested. where to get dx12 boilerplate? test just that texute still looks correct, or also test performance improves when roteated? - maybe merge functions? would then pass a boolean/enum to determine rowToSwizzle versus swizzleToRow Added functions that converts pixel order from row major to standard swizzle and from standard swizzle to row major. Functions for both 1 Image, and an array of Images. Folwoing standard DirectXTex pattern: Uses Image and TexMetadata as input. Outputs/initializes a ScratchImage.
Added new file to .vcxproj, other.vcxproj, and Cmake Merged to/from functions togeather memcpy src is const added non-AVX2 deposit_bits
TODO what flags/threshold to use to re-Compress?
Code review
Linux filesystem is case sensitive
09ff9e8 to
68b9cc7
Compare
Updated with DLL annotation for new function
|
I'm going to wrap up the March 2025 release, and then finish this PR out. That way I have some time to iterate on the new method. The current implementation only works for 'tiled sized' textures. |
| for (size_t x = 0; x < width; ++x) | ||
| { | ||
| const uint32_t swizzleIndex = deposit_bits(static_cast<uint32_t>(x), xBytesMask) + deposit_bits(static_cast<uint32_t>(y), ~xBytesMask); | ||
| const size_t swizzleOffset = swizzleIndex * bytesPerPixel; |
There was a problem hiding this comment.
🤔 Given the 16-bit masks above like 0b1010101011001111, would multiplying by bytesPerPixel yield a swizzled byte offset outside the current 64KB per page? From the docs, all formats from 8bpp to 64bpp are swizzled within each 64KB page.
"The standard swizzle formats applies within each page-sized chunk, and pages are laid out in linear order with respect to one another. A 16-bit interleave pattern defines the conversion from pre-swizzled intra-page location to the post-swizzled location." ... "layout of texels within the tile is defined" ... "Tiles are arranged in row-major order."
So for say BC1, it sounds like each page would be 512x256 pixel Morton tile, and that every 512x256 tile is then linearly appended in classic left-to-right then top-to-bottom order.
| Texel block size | Mask | Tile size per 64KB page |
|---|---|---|
| 8bpp 1x1 pixel (e.g. DXGI_FORMAT_R8_UNORM) |
XYXY XYXY YYYY XXXX | 256x256 pixels (2^8 x 2^8) |
| 16bpp 1x1 pixel (e.g. DXGI_FORMAT_B5G5R5A1_UNORM) |
XYXY XYXY XYYY XXX- | 256x128 pixels (2^8 x 2^7) |
| 32bpp 1x1 pixel (e.g. DXGI_FORMAT_B8G8R8A8_UNORM) |
XYXY XYXY XYYY XX-- | 128x128 pixels (2^7 x 2^7) |
| 64bpp 1x1 pixel (e.g. DXGI_FORMAT_R16G16B16A16_UNORM) |
XYXY XYXY XXYY X--- | 128x64 pixels (2^7 x 2^6) |
| 64bpb 4x4 block (e.g. DXGI_FORMAT_BC1_UNORM) |
XYXY XYXY XXYY X--- | 512x256 pixels (2^7*4 x 2^6*4) |
| 128bpp 1x1 pixel (e.g. DXGI_FORMAT_R32G32B32A32_FLOAT) |
XYXY XYXY XXYY ---- | 64x64 pixels (2^6 x 2^6) |
| 128bpb 4x4 block (e.g. DXGI_FORMAT_BC2_UNORM / BC3) |
XYXY XYXY XXYY ---- | 256x256 pixels (2^6*4 x 2^6*4) |
If correct, then a 1024x512 BC1 texture would be laid out like:
| - | left | right |
|---|---|---|
| top | page 0 (512x256) | page 1 (512x256) |
| bottom | page 2 (512x256) | page 3 (512x256) |
That means this process could be in terms of page tiles (x / 512, y / 256) with an inner 1D 64KB loop that doesn't actually care about x,y coordinates anymore - it's just remapping from byte addresses to byte addresses via the bitmasks.
(but I could be all wrong too 😅🤷♂️, without seeing how D3D actually implemented D3D12_TEXTURE_LAYOUT_64KB_STANDARD_SWIZZLE)
| DirectXTex/DirectXTexMipmaps.cpp | ||
| DirectXTex/DirectXTexMisc.cpp | ||
| DirectXTex/DirectXTexNormalMaps.cpp | ||
| DirectXTex/DirectXTexSwizzle.cpp |
There was a problem hiding this comment.
I've reverted all the vcxproj changes for now. I'll do those in a follow-up PR after finishing this one.
4 participants
Implement z-order curve "swizzling" for 2D and 3D textures to support the pixel order used for "Standard Swizzle" as defined by
D3D12_TEXTURE_LAYOUT_64KB_STANDARD_SWIZZLE.