From 094c73a080c18f0fc5d4ed01a136e3298f9b715d Mon Sep 17 00:00:00 2001 From: Orphis Date: Tue, 20 Oct 2009 00:55:07 +0000 Subject: OpenCL: Moved the OpenCL code from the compiled source to a separate file TextureDecoder.cl and added a .rules files to copy the updated version to the build directory. Fixed RGB5A3 decoding with alpha New CMPR decoding, blocks with no alpha are great, still have to figure the problems with transparent blocks. Disabled for now. Added a better error reporting to the base OpenCL functions git-svn-id: https://dolphin-emu.googlecode.com/svn/trunk@4439 8ced0084-cf51-0410-be5f-012b33b47a6e --- .../VideoCommon/Src/OpenCL/OCLTextureDecoder.cpp | 307 +++------------------ 1 file changed, 38 insertions(+), 269 deletions(-) (limited to 'Source/Core/VideoCommon/Src/OpenCL/OCLTextureDecoder.cpp') diff --git a/Source/Core/VideoCommon/Src/OpenCL/OCLTextureDecoder.cpp b/Source/Core/VideoCommon/Src/OpenCL/OCLTextureDecoder.cpp index cf8d79c85b..43179d9821 100644 --- a/Source/Core/VideoCommon/Src/OpenCL/OCLTextureDecoder.cpp +++ b/Source/Core/VideoCommon/Src/OpenCL/OCLTextureDecoder.cpp @@ -18,6 +18,7 @@ #include "OCLTextureDecoder.h" #include "OpenCL.h" +#include "FileUtil.h" #include #include @@ -26,159 +27,16 @@ #include #include #include - +#include + +//#define DEBUG_OPENCL + struct sDecoders { const char name[256]; // kernel name cl_kernel kernel; // compute kernel }; -const char *Kernel = " \n\ -kernel void DecodeI4(global uchar *dst, \n\ - const global uchar *src, int width) \n\ -{ \n\ - int x = get_global_id(0) * 8, y = get_global_id(1) * 8; \n\ - int srcOffset = x + y * width / 8; \n\ - for (int iy = 0; iy < 8; iy++) \n\ - { \n\ - uchar4 val = vload4(srcOffset, src); \n\ - uchar8 res; \n\ - res.even = (val >> 4) & 0x0F; \n\ - res.odd = val & 0x0F; \n\ - res |= res << 4; \n\ - vstore8(res, 0, dst + ((y + iy)*width + x)); \n\ - srcOffset++; \n\ - } \n\ -} \n\ - \n\ -kernel void DecodeI8(global uchar *dst, \n\ - const global uchar *src, int width) \n\ -{ \n\ - int x = get_global_id(0) * 8, y = get_global_id(1) * 4; \n\ - int srcOffset = ((x * 4) + (y * width)) / 8; \n\ - for (int iy = 0; iy < 4; iy++) \n\ - { \n\ - vstore8(vload8(srcOffset, src), \n\ - 0, dst + ((y + iy)*width + x)); \n\ - srcOffset++; \n\ - } \n\ -} \n\ - \n\ -kernel void DecodeIA8(global uchar *dst, \n\ - const global uchar *src, int width) \n\ -{ \n\ - int x = get_global_id(0) * 4, y = get_global_id(1) * 4; \n\ - int srcOffset = ((x * 4) + (y * width)) / 4; \n\ - for (int iy = 0; iy < 4; iy++) \n\ - { \n\ - uchar8 val = vload8(srcOffset, src); \n\ - uchar8 res; \n\ - res.odd = val.even; \n\ - res.even = val.odd; \n\ - vstore8(res, 0, dst + ((y + iy)*width + x) * 2); \n\ - srcOffset++; \n\ - } \n\ -} \n\ - \n\ -kernel void DecodeIA4(global uchar *dst, \n\ - const global uchar *src, int width) \n\ -{ \n\ - int x = get_global_id(0) * 8, y = get_global_id(1) * 4; \n\ - int srcOffset = ((x * 4) + (y * width)) / 8; \n\ - for (int iy = 0; iy < 4; iy++) \n\ - { \n\ - uchar8 val = vload8(srcOffset, src); \n\ - uchar16 res; \n\ - res.odd = (val >> 4) & 0x0F; \n\ - res.even = val & 0x0F; \n\ - res |= res << 4; \n\ - vstore16(res, 0, dst + ((y + iy)*width + x) * 2); \n\ - srcOffset++; \n\ - } \n\ -} \n\ - \n\ -kernel void DecodeRGBA8(global uchar *dst, \n\ - const global uchar *src, int width) \n\ -{ \n\ - int x = get_global_id(0) * 4, y = get_global_id(1) * 4; \n\ - int srcOffset = (x * 2) + (y * width) / 2; \n\ - for (int iy = 0; iy < 4; iy++) \n\ - { \n\ - uchar8 ar = vload8(srcOffset, src); \n\ - uchar8 gb = vload8(srcOffset + 4, src); \n\ - uchar16 res; \n\ - res.even.even = gb.odd; \n\ - res.even.odd = ar.odd; \n\ - res.odd.even = gb.even; \n\ - res.odd.odd = ar.even; \n\ - vstore16(res, 0, dst + ((y + iy)*width + x) * 4); \n\ - srcOffset++; \n\ - } \n\ -} \n\ - \n\ -kernel void DecodeRGB565(global ushort *dst, \n\ - const global ushort *src, int width) \n\ -{ \n\ - int x = get_global_id(0) * 4, y = get_global_id(1) * 4; \n\ - int srcOffset = x + (y * width) / 4; \n\ - for (int iy = 0; iy < 4; iy++) \n\ - { \n\ - ushort4 val = vload4(srcOffset, src); \n\ - val = (val >> 8) | (val << 8); \n\ - vstore4(val, 0, dst + ((y + iy)*width + x)); \n\ - srcOffset++; \n\ - } \n\ -} \n\ - \n\ -kernel void DecodeRGB5A3(global uchar *dst, \n\ - const global uchar *src, int width) \n\ -{ \n\ - int x = get_global_id(0) * 4, y = get_global_id(1) * 4; \n\ - int srcOffset = x + (y * width) / 4; \n\ - for (int iy = 0; iy < 4; iy++) \n\ - { \n\ - ushort8 val = convert_ushort8(vload8(srcOffset, src));\n\ - ushort4 vs = val.odd | (val.even << 8); \n\ - uchar16 resNoAlpha; \n\ - resNoAlpha.odd.odd = (uchar4)(0xFF); \n\ - resNoAlpha.even.odd = convert_uchar4(vs >> 7) & 0xF8; \n\ - resNoAlpha.odd.even = convert_uchar4(vs >> 2) & 0xF8; \n\ - resNoAlpha.even.even = convert_uchar4(vs << 3) & 0xF8;\n\ - // Better but cause color bleeding \n\ - //resNoAlpha |= resNoAlpha >> 5; \n\ - uchar16 resAlpha; \n\ - resAlpha.even.odd = convert_uchar4(vs >> 8) & 0x0F; \n\ - resAlpha.odd.even = convert_uchar4(vs >> 4) & 0x0F; \n\ - resAlpha.even.even = convert_uchar4(vs) & 0x0F; \n\ - resAlpha |= resNoAlpha << 4; \n\ - resAlpha.odd.odd = convert_uchar4(vs >> 7) & 0xE0; \n\ - resAlpha.odd.odd |= (resAlpha.odd.odd >> 3) \n\ - | (resAlpha.odd.odd >> 6); \n\ - uchar16 choice = (uchar16)((uchar4)(vs.s0 >> 8), \n\ - (uchar4)(vs.s1 >> 8), \n\ - (uchar4)(vs.s2 >> 8), \n\ - (uchar4)(vs.s3 >> 8)); \n\ - uchar16 res; \n\ - res = select(resAlpha, resNoAlpha, choice); \n\ - vstore16(res, 0, dst + ((y + iy)*width + x) * 4); \n\ - srcOffset++; \n\ - } \n\ -} \n\ - \n\ -kernel void DecodeDXT(global ulong *dst, \n\ - const global ulong *src, int width) \n\ -{ // TODO: PLEASE NOTE THAT THIS CODE DOES NOT WORK \n\ - int x = get_global_id(0) * 8, y = get_global_id(1) * 8; \n\ - int srcOffset = ((x * 4) + (y * width)) / 8; \n\ - for (int iy = 0; iy < 4; iy++) \n\ - { \n\ - vstore8(vload8(srcOffset, src), \n\ - 0, dst + ((y + iy)*width + x)); \n\ - srcOffset++; \n\ - } \n\ -} \n\ -"; - cl_program g_program; // NULL terminated set of kernels sDecoders Decoders[] = { @@ -189,22 +47,29 @@ sDecoders Decoders[] = { {"DecodeRGBA8", NULL}, {"DecodeRGB565", NULL}, {"DecodeRGB5A3", NULL}, -{"DecodeDXT", NULL}, +{"DecodeCMPR", NULL}, {"", NULL}, }; bool g_Inited = false; cl_mem g_clsrc, g_cldst; // texture buffer memory objects -void TexDecoder_OpenCL_Initialize() { -#if defined(HAVE_OPENCL) && HAVE_OPENCL +void TexDecoder_OpenCL_Initialize() { +#if defined(HAVE_OPENCL) && HAVE_OPENCL if(!g_Inited) { if(!OpenCL::Initialize()) return; + std::string code; + char* filename = "User/OpenCL/TextureDecoder.cl"; + if (!File::ReadFileToString(true, filename, code)) + { + ERROR_LOG(VIDEO, "Failed to load OpenCL code %s - file is missing?", filename); + return; + } - g_program = OpenCL::CompileProgram(Kernel); + g_program = OpenCL::CompileProgram(code.c_str()); int i = 0; while(strlen(Decoders[i].name) > 0) { @@ -213,16 +78,18 @@ void TexDecoder_OpenCL_Initialize() { } // Allocating maximal Wii texture size in advance, so that we don't have to allocate/deallocate per texture +#ifndef DEBUG_OPENCL g_clsrc = clCreateBuffer(OpenCL::GetContext(), CL_MEM_READ_ONLY , 1024 * 1024 * sizeof(u32), NULL, NULL); g_cldst = clCreateBuffer(OpenCL::GetContext(), CL_MEM_WRITE_ONLY, 1024 * 1024 * sizeof(u32), NULL, NULL); +#endif g_Inited = true; - } -#endif -} - + } +#endif +} + void TexDecoder_OpenCL_Shutdown() { -#if defined(HAVE_OPENCL) && HAVE_OPENCL +#if defined(HAVE_OPENCL) && HAVE_OPENCL && !defined(DEBUG_OPENCL) if(g_clsrc) clReleaseMemObject(g_clsrc); @@ -257,7 +124,6 @@ PC_TexFormat TexDecoder_Decode_OpenCL(u8 *dst, const u8 *src, int width, int hei formatResult = PC_TEX_FMT_I8; break; case GX_TF_IA4: - // Maybe a cleaner way is needed kernelToRun = Decoders[2].kernel; sizeOfSrc = sizeof(u8); sizeOfDst = sizeof(u16); @@ -295,9 +161,10 @@ PC_TexFormat TexDecoder_Decode_OpenCL(u8 *dst, const u8 *src, int width, int hei formatResult = PC_TEX_FMT_BGRA32; break; case GX_TF_CMPR: - return PC_TEX_FMT_NONE; // <-- TODO: Fix CMPR + return PC_TEX_FMT_NONE; // Remove to test CMPR kernelToRun = Decoders[7].kernel; - sizeOfSrc = sizeOfDst = sizeof(u32); + sizeOfSrc = sizeof(u8) / 2.0f; + sizeOfDst = sizeof(u32); xSkip = 8; ySkip = 8; formatResult = PC_TEX_FMT_BGRA32; @@ -306,6 +173,11 @@ PC_TexFormat TexDecoder_Decode_OpenCL(u8 *dst, const u8 *src, int width, int hei return PC_TEX_FMT_NONE; } +#ifdef DEBUG_OPENCL + g_clsrc = clCreateBuffer(OpenCL::GetContext(), CL_MEM_READ_ONLY , 1024 * 1024 * sizeof(u32), NULL, NULL); + g_cldst = clCreateBuffer(OpenCL::GetContext(), CL_MEM_WRITE_ONLY, 1024 * 1024 * sizeof(u32), NULL, NULL); +#endif + clEnqueueWriteBuffer(OpenCL::GetCommandQueue(), g_clsrc, CL_TRUE, 0, (size_t)(width * height * sizeOfSrc), src, 0, NULL, NULL); clSetKernelArg(kernelToRun, 0, sizeof(cl_mem), &g_cldst); @@ -324,125 +196,22 @@ PC_TexFormat TexDecoder_Decode_OpenCL(u8 *dst, const u8 *src, int width, int hei err = clEnqueueNDRangeKernel(OpenCL::GetCommandQueue(), kernelToRun, 2, NULL, global, NULL, 0, NULL, NULL); if(err) - PanicAlert("Error queueing kernel"); + OpenCL::HandleCLError(err, "Failed to enqueue kernel"); clFinish(OpenCL::GetCommandQueue()); clEnqueueReadBuffer(OpenCL::GetCommandQueue(), g_cldst, CL_TRUE, 0, (size_t)(width * height * sizeOfDst), dst, 0, NULL, NULL); - + +#ifdef DEBUG_OPENCL + clReleaseMemObject(g_clsrc); + clReleaseMemObject(g_cldst); +#endif + return formatResult; #else return PC_TEX_FMT_NONE; #endif - /* switch (texformat) - { - case GX_TF_C4: - if (tlutfmt == 2) - { - // Special decoding is required for TLUT format 5A3 - for (int y = 0; y < height; y += 8) - for (int x = 0; x < width; x += 8) - for (int iy = 0; iy < 8; iy++, src += 4) - decodebytesC4_5A3_To_BGRA32((u32*)dst + (y + iy) * width + x, src, tlutaddr); - } - else - { - for (int y = 0; y < height; y += 8) - for (int x = 0; x < width; x += 8) - for (int iy = 0; iy < 8; iy++, src += 4) - decodebytesC4_To_Raw16((u16*)dst + (y + iy) * width + x, src, tlutaddr); - } - return GetPCFormatFromTLUTFormat(tlutfmt); - case GX_TF_C8: - if (tlutfmt == 2) - { - // Special decoding is required for TLUT format 5A3 - for (int y = 0; y < height; y += 4) - for (int x = 0; x < width; x += 8) - for (int iy = 0; iy < 4; iy++, src += 8) - decodebytesC8_5A3_To_BGRA32((u32*)dst + (y + iy) * width + x, src, tlutaddr); - } - else - { - for (int y = 0; y < height; y += 4) - for (int x = 0; x < width; x += 8) - for (int iy = 0; iy < 4; iy++, src += 8) - decodebytesC8_To_Raw16((u16*)dst + (y + iy) * width + x, src, tlutaddr); - } - return GetPCFormatFromTLUTFormat(tlutfmt); - case GX_TF_C14X2: - if (tlutfmt == 2) - { - // Special decoding is required for TLUT format 5A3 - for (int y = 0; y < height; y += 4) - for (int x = 0; x < width; x += 4) - for (int iy = 0; iy < 4; iy++, src += 8) - decodebytesC14X2_5A3_To_BGRA32((u32*)dst + (y + iy) * width + x, (u16*)src, tlutaddr); - } - else - { - for (int y = 0; y < height; y += 4) - for (int x = 0; x < width; x += 4) - for (int iy = 0; iy < 4; iy++, src += 8) - decodebytesC14X2_To_Raw16((u16*)dst + (y + iy) * width + x, (u16*)src, tlutaddr); - } - return GetPCFormatFromTLUTFormat(tlutfmt); - case GX_TF_RGB565: - { - for (int y = 0; y < height; y += 4) - for (int x = 0; x < width; x += 4) - for (int iy = 0; iy < 4; iy++, src += 8) - { - u16 *ptr = (u16 *)dst + (y + iy) * width + x; - u16 *s = (u16 *)src; - for(int j = 0; j < 4; j++) - *ptr++ = Common::swap16(*s++); - } - } - return PC_TEX_FMT_RGB565; - case GX_TF_RGB5A3: - { - for (int y = 0; y < height; y += 4) - for (int x = 0; x < width; x += 4) - for (int iy = 0; iy < 4; iy++, src += 8) - //decodebytesRGB5A3((u32*)dst+(y+iy)*width+x, (u16*)src, 4); - decodebytesRGB5A3((u32*)dst+(y+iy)*width+x, (u16*)src); - } - return PC_TEX_FMT_BGRA32; - case GX_TF_RGBA8: // speed critical - { - for (int y = 0; y < height; y += 4) - for (int x = 0; x < width; x += 4) - { - for (int iy = 0; iy < 4; iy++) - decodebytesARGB8_4((u32*)dst + (y+iy)*width + x, (u16*)src + 4 * iy, (u16*)src + 4 * iy + 16); - src += 64; - } - } - return PC_TEX_FMT_BGRA32; - case GX_TF_CMPR: // speed critical - // The metroid games use this format almost exclusively. - { - for (int y = 0; y < height; y += 8) - { - for (int x = 0; x < width; x += 8) - { - decodeDXTBlock((u32*)dst + y * width + x, (DXTBlock*)src, width); - src += sizeof(DXTBlock); - decodeDXTBlock((u32*)dst + y * width + x + 4, (DXTBlock*)src, width); - src += sizeof(DXTBlock); - decodeDXTBlock((u32*)dst + (y + 4) * width + x, (DXTBlock*)src, width); - src += sizeof(DXTBlock); - decodeDXTBlock((u32*)dst + (y + 4) * width + x + 4, (DXTBlock*)src, width); - src += sizeof(DXTBlock); - } - } - return PC_TEX_FMT_BGRA32; - } - } -*/ - // The "copy" texture formats, too? return PC_TEX_FMT_NONE; } -- cgit v1.2.3