Summary: | glsl2 crash | ||
---|---|---|---|
Product: | Mesa | Reporter: | Luca Barbato <lu_zero> |
Component: | Other | Assignee: | mesa-dev |
Status: | RESOLVED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | medium | ||
Version: | git | ||
Hardware: | Other | ||
OS: | All | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Attachments: |
Add core.h
glsl: Include only main/core.h. |
Description
Luca Barbato
2010-08-22 19:54:28 UTC
It seems to be a symptom of mixing FEATURE_GL libglsl.a with FEATURE_ES2 libes2gallium.a. There are several ways a) build FEATURE_ES2 libes2sl.a as well as libglsl.a b) remove "#if FEATURE_xxx" from core mesa types (noticeably dd_function_table) c) do not use core mesa types in glsl to fix it. I am favoring b). Suggestions? Created attachment 38089 [details] [review] Add core.h This patch adds main/core.h and removes feature tests in mtypes.h and dd.h. It fixes this bug. Created attachment 38090 [details] [review] glsl: Include only main/core.h. A minor cleanup. The idea is that GLX, WGL, and GLSL should only include main/core.h from core mesa. Looks good to me. Agreed on b). Removing FEATURE_* ifdefs ftw. I've pushed the patches, with the second one updated for commit 6dad5a. There are more commits to update st/glx and st/wgl to use only main/core.h. "git grep 'main/.*\.h'" shows that main/core.h is the only core mesa header that is included outside src/mesa/. es2gears should work now. I am closing this bug. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.