Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Assert failure in featurettes branch on OpenGL 3.2 #1581

Closed
cosmic-linden opened this issue May 28, 2024 · 1 comment
Closed

Assert failure in featurettes branch on OpenGL 3.2 #1581

cosmic-linden opened this issue May 28, 2024 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@cosmic-linden
Copy link
Contributor

cosmic-linden commented May 28, 2024

Description

In materials featurette upstream (commit 7ed053d9a257087b76937d750b9f2b0c96f6df38), running on my local build, I get an assert crash when I set RenderMaxOpenGLVersion down to OpenGL 3.2 . The call stack implicates LLHeroProbeManager.

We disable reflection probes on OpenGL versions that don't support cubemaps. We should do this for mirrors as well. This could be related.

Call stack:

secondlife-bin.exe!LLGLSLShader::bind::__l13::<lambda>(int * ptr) Line 1027	C++
secondlife-bin.exe!void <lambda>(int *)::<lambda_invoker_cdecl>(int * ptr) Line 1027	C++
secondlife-bin.exe!crashdriver(void(*)(int *) callback) Line 1662	C++
secondlife-bin.exe!LLGLSLShader::bind() Line 1027	C++
secondlife-bin.exe!LLHeroProbeManager::generateRadiance(LLReflectionMap * probe) Line 403	C++
secondlife-bin.exe!LLHeroProbeManager::update() Line 250	C++
secondlife-bin.exe!display(int rebuild, float zoom_factor, int subfield, int for_snapshot) Line 668	C++
secondlife-bin.exe!LLAppViewer::doFrame() Line 1514	C++
secondlife-bin.exe!LLAppViewer::frame() Line 1347	C++
secondlife-bin.exe!wWinMain(HINSTANCE__ * hInstance, HINSTANCE__ * hPrevInstance, wchar_t * pCmdLine, int nCmdShow) Line 510	C++
[External Code]	

Specifically, this assert fails in LLGLSLShader::bind():

llassert(mProgramObject != 0);
@cosmic-linden cosmic-linden added bug Something isn't working triage Flags issues that need to be triaged labels May 28, 2024
@cosmic-linden cosmic-linden changed the title Crash in featurettes viewer on OpenGL 3.2 Assert failure in featurettes viewer on OpenGL 3.2 May 28, 2024
@cosmic-linden cosmic-linden changed the title Assert failure in featurettes viewer on OpenGL 3.2 Assert failure in featurettes branch on OpenGL 3.2 May 28, 2024
@cosmic-linden cosmic-linden removed the triage Flags issues that need to be triaged label May 28, 2024
@Geenz Geenz self-assigned this May 29, 2024
@Dan-Linden
Copy link
Contributor

Passed QA on Featurettes 7.1.8.9375512768.

No crash on Win10. Reflections and mirrors are both disabled as expected when set to OpenGL 3.2
No crash on Mac. However, on OpenGL 3.2, reflections and mirrors are still enabled. Appears that mac is not actually set to OpenGL 3.2 but rather 4.1 after checking About Second Life.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants