From ce99921c20ef63ea7ed1361a6453bc6384ab0748 Mon Sep 17 00:00:00 2001 From: Ryan Houdek Date: Fri, 27 Dec 2013 10:56:03 -0600 Subject: [PATCH] [buffer_storage] Implement ARB_buffer_storage. Disable it for GL_ARRAY_BUFFER due to a bug in Nvidia's drivers that causes black screen with it. --- Source/Core/VideoBackends/OGL/Src/Render.cpp | 2 + Source/Core/VideoBackends/OGL/Src/Render.h | 1 + .../VideoBackends/OGL/Src/StreamBuffer.cpp | 52 +++++++++++++++---- .../Core/VideoBackends/OGL/Src/StreamBuffer.h | 5 +- Source/Core/VideoCommon/Src/DriverDetails.cpp | 1 + Source/Core/VideoCommon/Src/DriverDetails.h | 8 +++ 6 files changed, 56 insertions(+), 13 deletions(-) diff --git a/Source/Core/VideoBackends/OGL/Src/Render.cpp b/Source/Core/VideoBackends/OGL/Src/Render.cpp index 4d9980fc65..694301b6f8 100644 --- a/Source/Core/VideoBackends/OGL/Src/Render.cpp +++ b/Source/Core/VideoBackends/OGL/Src/Render.cpp @@ -478,6 +478,7 @@ Renderer::Renderer() g_ogl_config.bSupportsGLSync = TO_BOOL(GLEW_ARB_sync); g_ogl_config.bSupportsGLBaseVertex = TO_BOOL(GLEW_ARB_draw_elements_base_vertex) && !DriverDetails::HasBug(DriverDetails::BUG_BROKENPINNEDMEMORY); + g_ogl_config.bSupportsGLBufferStorage = TO_BOOL(GLEW_ARB_buffer_storage); g_ogl_config.bSupportCoverageMSAA = TO_BOOL(GLEW_NV_framebuffer_multisample_coverage); g_ogl_config.bSupportSampleShading = TO_BOOL(GLEW_ARB_sample_shading); g_ogl_config.bSupportOGL31 = TO_BOOL(GLEW_VERSION_3_1); @@ -554,6 +555,7 @@ Renderer::Renderer() g_ogl_config.bSupportsGLPinnedMemory ? "" : "PinnedMemory ", g_ogl_config.bSupportsGLSLCache ? "" : "ShaderCache ", g_ogl_config.bSupportsGLBaseVertex ? "" : "BaseVertex ", + g_ogl_config.bSupportsGLBufferStorage ? "" : "BufferStorage ", g_ogl_config.bSupportsGLSync ? "" : "Sync ", g_ogl_config.bSupportCoverageMSAA ? "" : "CSAA ", g_ogl_config.bSupportSampleShading ? "" : "SSAA " diff --git a/Source/Core/VideoBackends/OGL/Src/Render.h b/Source/Core/VideoBackends/OGL/Src/Render.h index 9489d5c7ac..58ba086d18 100644 --- a/Source/Core/VideoBackends/OGL/Src/Render.h +++ b/Source/Core/VideoBackends/OGL/Src/Render.h @@ -22,6 +22,7 @@ extern struct VideoConfig { bool bSupportsGLPinnedMemory; bool bSupportsGLSync; bool bSupportsGLBaseVertex; + bool bSupportsGLBufferStorage; bool bSupportCoverageMSAA; bool bSupportSampleShading; GLSL_VERSION eSupportedGLSLVersion; diff --git a/Source/Core/VideoBackends/OGL/Src/StreamBuffer.cpp b/Source/Core/VideoBackends/OGL/Src/StreamBuffer.cpp index 8ec9c3c1cb..5e3d4dfd38 100644 --- a/Source/Core/VideoBackends/OGL/Src/StreamBuffer.cpp +++ b/Source/Core/VideoBackends/OGL/Src/StreamBuffer.cpp @@ -33,7 +33,11 @@ StreamBuffer::StreamBuffer(u32 type, size_t size, StreamType uploadType) g_Config.bHackedBufferUpload = false; } - if(!g_ogl_config.bSupportsGLBaseVertex && (m_uploadtype & BUFFERSUBDATA) + if (g_ogl_config.bSupportsGLBufferStorage && + !(DriverDetails::HasBug(DriverDetails::BUG_BROKENBUFFERSTORAGE) && type == GL_ARRAY_BUFFER) && + (m_uploadtype & BUFFERSTORAGE)) + m_uploadtype = BUFFERSTORAGE; + else if(!g_ogl_config.bSupportsGLBaseVertex && (m_uploadtype & BUFFERSUBDATA) && !DriverDetails::HasBug(DriverDetails::BUG_BROKENBUFFERSTREAM)) m_uploadtype = BUFFERSUBDATA; else if(!g_ogl_config.bSupportsGLBaseVertex && (m_uploadtype & BUFFERDATA)) @@ -79,9 +83,9 @@ void StreamBuffer::Alloc ( size_t size, u32 stride ) break; case MAP_AND_SYNC: case PINNED_MEMORY: - + case BUFFERSTORAGE: // insert waiting slots for used memory - for(size_t i=SLOT(m_used_iterator); i= m_size) { + if (iter_end >= m_size) { // insert waiting slots in unused space at the end of the buffer for (size_t i = SLOT(m_used_iterator); i < SYNC_POINTS; i++) @@ -109,7 +113,7 @@ void StreamBuffer::Alloc ( size_t size, u32 stride ) iter_end = size; // wait for space at the start - for(u32 i=0; i<=SLOT(iter_end); i++) + for (u32 i = 0; i <= SLOT(iter_end); i++) { glClientWaitSync(fences[i], GL_SYNC_FLUSH_COMMANDS_BIT, GL_TIMEOUT_IGNORED); glDeleteSync(fences[i]); @@ -128,7 +132,7 @@ void StreamBuffer::Alloc ( size_t size, u32 stride ) m_iterator_aligned = 0; break; case STREAM_DETECT: - case DETECT_MASK: // Just to shutup warnings + case DETECT_MASK: // To shutup compiler warnings break; } m_iterator = m_iterator_aligned; @@ -149,8 +153,9 @@ size_t StreamBuffer::Upload ( u8* data, size_t size ) break; case PINNED_MEMORY: case MAP_AND_RISK: - if(pointer) - memcpy(pointer+m_iterator, data, size); + case BUFFERSTORAGE: + if (pointer) + memcpy(pointer + m_iterator, data, size); break; case BUFFERSUBDATA: glBufferSubData(m_buffertype, m_iterator, size, data); @@ -159,7 +164,7 @@ size_t StreamBuffer::Upload ( u8* data, size_t size ) glBufferData(m_buffertype, size, data, GL_STREAM_DRAW); break; case STREAM_DETECT: - case DETECT_MASK: // Just to shutup warnings + case DETECT_MASK: // To shutup compiler warnings break; } size_t ret = m_iterator; @@ -204,6 +209,25 @@ void StreamBuffer::Init() Init(); } break; + + case BUFFERSTORAGE: + glGetError(); // errors before this allocation should be ignored + fences = new GLsync[SYNC_POINTS]; + for (u32 i = 0; i m_bugs; diff --git a/Source/Core/VideoCommon/Src/DriverDetails.h b/Source/Core/VideoCommon/Src/DriverDetails.h index 1b024cefca..dd8eeade69 100644 --- a/Source/Core/VideoCommon/Src/DriverDetails.h +++ b/Source/Core/VideoCommon/Src/DriverDetails.h @@ -141,6 +141,14 @@ namespace DriverDetails // Ended Version: -1 // If a shader includes a textureSize function call then the shader compiler will call abort() BUG_BROKENTEXTURESIZE, + // Bug: ARB_buffer_storage doesn't work with ARRAY_BUFFER type streams + // Affected devices: Geforce 4xx+ + // Started Version: -1 + // Ended Version: -1 + // The buffer_storage streaming method is required for greater speed gains in our buffer streaming + // It reduces what is needed for streaming to basically a memcpy call + // It seems to work for all buffer types except GL_ARRAY_BUFFER + BUG_BROKENBUFFERSTORAGE, }; // Initializes our internal vendor, device family, and driver version