From: Ziqing Hui Subject: [PATCH 3/3] d3dx10: Support block compressed format for D3DX10CreateTextureFromMemory. Message-Id: <58baa2da-b22d-604d-103b-796e387992bd@codeweavers.com> Date: Fri, 18 Jun 2021 14:29:59 +0800 Signed-off-by: Ziqing Hui --- dlls/d3dx10_43/tests/d3dx10.c | 3 +- dlls/d3dx10_43/texture.c | 92 +++++++++++++++++++++++++---------- 2 files changed, 69 insertions(+), 26 deletions(-) diff --git a/dlls/d3dx10_43/tests/d3dx10.c b/dlls/d3dx10_43/tests/d3dx10.c index 184a061ff0e..3419b298bfa 100644 --- a/dlls/d3dx10_43/tests/d3dx10.c +++ b/dlls/d3dx10_43/tests/d3dx10.c @@ -1147,6 +1147,8 @@ static void check_resource_data(ID3D10Resource *resource, const struct test_imag { line_match = !memcmp(image->expected_data + stride * i, (BYTE *)map.pData + map.RowPitch * i, stride); + todo_wine_if(is_block_compressed(image->expected_info.Format) + && (image->expected_info.Width % 4 != 0 || image->expected_info.Height % 4 != 0)) ok_(__FILE__, line)(line_match, "Data mismatch for line %u.\n", i); if (!line_match) break; @@ -1986,7 +1988,6 @@ static void test_create_texture(void) winetest_push_context("Test %u", i); hr = D3DX10CreateTextureFromMemory(device, test_image[i].data, test_image[i].size, NULL, NULL, &resource, NULL); - todo_wine_if(is_block_compressed(test_image[i].expected_info.Format)) ok(hr == S_OK, "Got unexpected hr %#x.\n", hr); if (hr == S_OK) { diff --git a/dlls/d3dx10_43/texture.c b/dlls/d3dx10_43/texture.c index 84164bf54a5..94db6bc769e 100644 --- a/dlls/d3dx10_43/texture.c +++ b/dlls/d3dx10_43/texture.c @@ -71,6 +71,17 @@ wic_pixel_formats[] = { &GUID_WICPixelFormat128bppRGBAFloat, DXGI_FORMAT_R32G32B32A32_FLOAT } }; +static DXGI_FORMAT block_compressed_formats[] = +{ + DXGI_FORMAT_BC1_TYPELESS, DXGI_FORMAT_BC1_UNORM, DXGI_FORMAT_BC1_UNORM_SRGB, + DXGI_FORMAT_BC2_TYPELESS, DXGI_FORMAT_BC2_UNORM, DXGI_FORMAT_BC2_UNORM_SRGB, + DXGI_FORMAT_BC3_TYPELESS, DXGI_FORMAT_BC3_UNORM, DXGI_FORMAT_BC3_UNORM_SRGB, + DXGI_FORMAT_BC4_TYPELESS, DXGI_FORMAT_BC4_UNORM, DXGI_FORMAT_BC4_SNORM, + DXGI_FORMAT_BC5_TYPELESS, DXGI_FORMAT_BC5_UNORM, DXGI_FORMAT_BC5_SNORM, + DXGI_FORMAT_BC6H_TYPELESS, DXGI_FORMAT_BC6H_UF16, DXGI_FORMAT_BC6H_SF16, + DXGI_FORMAT_BC7_TYPELESS, DXGI_FORMAT_BC7_UNORM, DXGI_FORMAT_BC7_UNORM_SRGB +}; + static const DXGI_FORMAT to_be_converted_format[] = { DXGI_FORMAT_UNKNOWN, @@ -124,6 +135,17 @@ static D3D10_RESOURCE_DIMENSION wic_dimension_to_d3dx10_dimension(WICDdsDimensio } } +static BOOL is_block_compressed(DXGI_FORMAT format) +{ + unsigned int i; + + for (i = 0; i < ARRAY_SIZE(block_compressed_formats); ++i) + if (format == block_compressed_formats[i]) + return TRUE; + + return FALSE; +} + static unsigned int get_bpp_from_format(DXGI_FORMAT format) { switch (format) @@ -556,6 +578,7 @@ HRESULT WINAPI D3DX10CreateTextureFromMemory(ID3D10Device *device, const void *s { unsigned int frame_count, width, height, stride, frame_size; IWICFormatConverter *converter = NULL; + IWICDdsFrameDecode *dds_frame = NULL; D3D10_TEXTURE2D_DESC texture_2d_desc; IWICBitmapFrameDecode *frame = NULL; IWICImagingFactory *factory = NULL; @@ -602,6 +625,11 @@ HRESULT WINAPI D3DX10CreateTextureFromMemory(ID3D10Device *device, const void *s width = img_info.Width; height = img_info.Height; + if (is_block_compressed(img_info.Format)) + { + width = (width + 3) & ~3; + height = (height + 3) & ~3; + } stride = (width * get_bpp_from_format(img_info.Format) + 7) / 8; frame_size = stride * height; @@ -612,41 +640,53 @@ HRESULT WINAPI D3DX10CreateTextureFromMemory(ID3D10Device *device, const void *s goto end; } - dst_format = (GUID *)dxgi_format_to_wic_guid(img_info.Format); - if (!dst_format) + if (is_block_compressed(img_info.Format)) { - FIXME("Unsupported DXGI format %#x.\n", img_info.Format); - hr = E_FAIL; - goto end; - } - if (IsEqualGUID(&src_format, dst_format)) - { - hr = IWICBitmapFrameDecode_CopyPixels(frame, NULL, stride, frame_size, buffer); + hr = IWICBitmapFrameDecode_QueryInterface(frame, &IID_IWICDdsFrameDecode, (void **)&dds_frame); + if (FAILED(hr)) + goto end; + hr = IWICDdsFrameDecode_CopyBlocks(dds_frame, NULL, stride * 4, frame_size, buffer); if (FAILED(hr)) goto end; } else { - hr = IWICImagingFactory_CreateFormatConverter(factory, &converter); - if (FAILED(hr)) - goto end; - hr = IWICFormatConverter_CanConvert(converter, &src_format, dst_format, &can_convert); - if (FAILED(hr)) - goto end; - if (!can_convert) + dst_format = (GUID *)dxgi_format_to_wic_guid(img_info.Format); + if (!dst_format) { - WARN("Format converting %s to %s is not supported by WIC.\n", - debugstr_guid(&src_format), debugstr_guid(dst_format)); + FIXME("Unsupported DXGI format %#x.\n", img_info.Format); hr = E_FAIL; goto end; } - hr = IWICFormatConverter_Initialize(converter, (IWICBitmapSource *)frame, dst_format, - WICBitmapDitherTypeErrorDiffusion, 0, 0, WICBitmapPaletteTypeCustom); - if (FAILED(hr)) - goto end; - hr = IWICFormatConverter_CopyPixels(converter, NULL, stride, frame_size, buffer); - if (FAILED(hr)) - goto end; + if (IsEqualGUID(&src_format, dst_format)) + { + hr = IWICBitmapFrameDecode_CopyPixels(frame, NULL, stride, frame_size, buffer); + if (FAILED(hr)) + goto end; + } + else + { + hr = IWICImagingFactory_CreateFormatConverter(factory, &converter); + if (FAILED(hr)) + goto end; + hr = IWICFormatConverter_CanConvert(converter, &src_format, dst_format, &can_convert); + if (FAILED(hr)) + goto end; + if (!can_convert) + { + WARN("Format converting %s to %s is not supported by WIC.\n", + debugstr_guid(&src_format), debugstr_guid(dst_format)); + hr = E_FAIL; + goto end; + } + hr = IWICFormatConverter_Initialize(converter, (IWICBitmapSource *)frame, dst_format, + WICBitmapDitherTypeErrorDiffusion, 0, 0, WICBitmapPaletteTypeCustom); + if (FAILED(hr)) + goto end; + hr = IWICFormatConverter_CopyPixels(converter, NULL, stride, frame_size, buffer); + if (FAILED(hr)) + goto end; + } } memset(&texture_2d_desc, 0, sizeof(texture_2d_desc)); @@ -670,6 +710,8 @@ HRESULT WINAPI D3DX10CreateTextureFromMemory(ID3D10Device *device, const void *s end: if (converter) IWICFormatConverter_Release(converter); + if (dds_frame) + IWICDdsFrameDecode_Release(dds_frame); if (buffer) heap_free(buffer); if (frame)