Skip to content

Disable SSBO for now#24755

Closed
peppy wants to merge 1 commit intoppy:masterfrom
peppy:disable-ssbo
Closed

Disable SSBO for now#24755
peppy wants to merge 1 commit intoppy:masterfrom
peppy:disable-ssbo

Conversation

@peppy
Copy link
Member

@peppy peppy commented Sep 8, 2023

Too many performance issues reported. We're going to have to do more testing before pushing this out.
See #24751.

@longnguyen2004
Copy link

Can we have an override for it, for testing purposes? It seems to work fine for me (Windows OpenGL).

@X9VoiD
Copy link

X9VoiD commented Sep 8, 2023

Doesn't seem like adding the environment variable fixes the performance regression on my machine by setting it here.
image

@peppy
Copy link
Member Author

peppy commented Sep 8, 2023

Doesn't seem like adding the environment variable fixes the performance regression on my machine by setting it here.
image

Could you test the latest release which has this PR already incorporated?

@Feodor0090
Copy link
Contributor

It's even worse for me.
On 2023.908.1 with ssbo enabled (no envs touched) there was 120 fps in song select and ~200 in gameplay.
On 2023.908.2 there are only 75 in song select and 110-120 in osu! autoplay.

I had ~170 and ~300 in previous releases. I'm on linux with legacy gl renderer. Mesa is the latest.

@GuineaGuy99
Copy link

Performance aside this may also fix #24766 and #24767

@longnguyen2004
Copy link

The hotfix is in the latest release now, yet we're still seeing errors. Is the feature flag missing in some other related area of the code?

@X9VoiD
Copy link

X9VoiD commented Sep 9, 2023

Could you test the latest release which has this PR already incorporated?

Unfortunately, it did not help.

@peppy
Copy link
Member Author

peppy commented Sep 9, 2023

More reports of things still broken:

#24769
#24767
#24766

@smoogipoo
Copy link
Contributor

smoogipoo commented Sep 10, 2023

@X9VoiD
Copy link

X9VoiD commented Sep 10, 2023

Applying that on osu-framework/master, then building osu/disable-ssbo with dotnet run --project osu.Desktop --configuration Release causes a crash. It did not print out to console but the exception is logged to event viewer.

System.ArgumentOutOfRangeException: The size of the buffer must be at least 2. (Parameter 'uboSize')
   at osu.Framework.Graphics.Rendering.ShaderStorageBufferObjectStack`1..ctor(IRenderer renderer, Int32 uboSize, Int32 ssboSize) in C:\Users\oscar\projects\osu-framework\osu.Framework\Graphics\Rendering\ShaderStorageBufferObjectStack.cs:line 67
   at osu.Framework.Graphics.Rendering.Renderer.BeginFrame(Vector2 windowSize) in C:\Users\oscar\projects\osu-framework\osu.Framework\Graphics\Rendering\Renderer.cs:line 203
   at osu.Framework.Graphics.Veldrid.VeldridRenderer.BeginFrame(Vector2 windowSize) in C:\Users\oscar\projects\osu-framework\osu.Framework\Graphics\Veldrid\VeldridRenderer.cs:line 256
   at osu.Framework.Graphics.Rendering.Renderer.osu.Framework.Graphics.Rendering.IRenderer.BeginFrame(Vector2 windowSize) in C:\Users\oscar\projects\osu-framework\osu.Framework\Graphics\Rendering\Renderer.cs:line 1132
   at osu.Framework.Threading.DrawThread.OnInitialize() in C:\Users\oscar\projects\osu-framework\osu.Framework\Threading\DrawThread.cs:line 31
   at osu.Framework.Threading.GameThread.Initialize(Boolean withThrottling) in C:\Users\oscar\projects\osu-framework\osu.Framework\Threading\GameThread.cs:line 265
   at osu.Framework.Threading.GameThread.<createThread>g__runWork|66_0() in C:\Users\oscar\projects\osu-framework\osu.Framework\Threading\GameThread.cs:line 409
   at System.Threading.Thread.StartHelper.Callback(Object state)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location ---
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.Thread.StartCallback()

So I went ahead and changed 1's to 2. It worked however, performance is still not fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants