Bug 1357

It seems to occur with Mesa 17.2.1 but not with earlier versions.

Please can you run glxinfo?
https://dri.freedesktop.org/wiki/glxinfo/

I'll compare with the result on my machine using an older version of Mesa.

Oops, I occurred with Mesa 17.2.0 too but not with Mesa 17.1.5:
http://forum.jogamp.org/Mesa-17-2-0-renderer-driver-name-change-error-causes-GLProfile-not-mapped-initialization-error-td4038176.html

Gallium 0.4 on AMD HAWAII (DRM 3.15.0 / 4.12.12-1-ARCH, LLVM 4.0.1) -> AMD Radeon R9 200 Series (AMD HAWAII / DRM 3.15.0 / 4.12.12-1-ARCH, LLVM 4.0.1)

Using GL_VERSION might help as it should contain:
3.1 (3.1 Mesa 17.2.1)

or something like this:

3.3 (Core Profile) Mesa 17.2.1

I suggest this change:
final boolean isDriverMesa = glRenderer.contains(MesaSP) || glRenderer.contains("Gallium ");

-> 

final boolean isDriverMesa = glRenderer.contains(MesaSP) || glRenderer.contains("Gallium ") || glVersion.contains(MesaSP);

It won't break non Mesa AMD drivers :) Guiseppe's second suggestion seems to be viable.

I'm out of office today and tomorrow, if needed I can give the output of glxinfo on friday morning.
I'll try the fix suggested by Julien and report back result on my environment, I think it will work.

(In reply to gst0098 from comment #6)
> I'm out of office today and tomorrow, if needed I can give the output of
> glxinfo on friday morning.
> I'll try the fix suggested by Julien and report back result on my
> environment, I think it will work.

There's no hurry. Thank you. Feel free to make a pull request (only with this tiny change) against my repository if it works as expected.

Hello,
I tested the patch proposed by Julien and Zeno and it works.

Julien, how can I pull a request against your repo ?

I did the patch on the repo that I cloned with the command: 
git clone --recurse-submodules git://jogamp.org/srv/scm/jogl.git jogl

Giuseppe

(In reply to gst0098 from comment #8)
> Hello,
> I tested the patch proposed by Julien and Zeno and it works.
> 
> Julien, how can I pull a request against your repo ?
> 
> I did the patch on the repo that I cloned with the command: 
> git clone --recurse-submodules git://jogamp.org/srv/scm/jogl.git jogl
> 
> Giuseppe

Maybe it would be easier for you just to post here the git diff. I'll credit the change to Zeno and you.

Here it is:

diff --git a/src/jogl/classes/jogamp/opengl/GLContextImpl.java b/src/jogl/classes/jogamp/opengl/GLContextImpl.java
index 31a8e489e..c5b3ea8ac 100644
--- a/src/jogl/classes/jogamp/opengl/GLContextImpl.java
+++ b/src/jogl/classes/jogamp/opengl/GLContextImpl.java
@@ -2133,8 +2133,7 @@ public abstract class GLContextImpl extends GLContext {
     final boolean isES = 0 != ( ctp & GLContext.CTX_PROFILE_ES );
     final boolean isX11 = NativeWindowFactory.TYPE_X11 == NativeWindowFactory.getNativeWindowType(true);
     final boolean isWindows = Platform.getOSType() == Platform.OSType.WINDOWS;
-    final boolean isDriverMesa = glRenderer.contains(MesaSP) || glRenderer.contains("Gallium ");
-
+    final boolean isDriverMesa = glRenderer.contains(MesaSP) || glRenderer.contains("Gallium ") || glVersion.contains(MesaSP);
     final boolean isDriverATICatalyst;
     final boolean isDriverNVIDIAGeForce;
     final boolean isDriverIntel;

Thank you,
Giuseppe

Created attachment 799 [details]
Patched version of GLContextImpl

I've just patched GLContextImpl with JByteMod available at https://grax.info

Is this being put into JOGL?  It appears from comments that you have a fix, but maybe it has only been put into a patched version of the source file?

thanks

(In reply to Andy Skinner from comment #12)
> Is this being put into JOGL?  It appears from comments that you have a fix,
> but maybe it has only been put into a patched version of the source file?
> 
> thanks

http://forum.jogamp.org/how-can-I-build-jogl-2-3-3-and-when-is-it-due-for-release-tp4038662p4038675.html

The suggested patch causes a regression on AMD Mullins and AMD Cedar whereas it's required for AMD POLARIS (10, 11, ...). I suggest this change:
final boolean isDriverMesa = glRenderer.contains(MesaSP) || glRenderer.contains("Gallium ") || (glVersion.contains(MesaSP) && glRenderer.contains("POLARIS"));

Rather do this:
final boolean isDriverMesa = glRenderer.contains(MesaSP) || glRenderer.contains("Gallium ") || (glVersion.contains(MesaSP) && glRenderer.contains("POLARIS"));

as I suggested here:
https://github.com/sgothel/jogl/pull/105#issuecomment-449527037

Created attachment 802 [details]
Patched version of GLContextImpl

Takes into account neilcsmith-net's remark on Mesa 18.2.2.

Created attachment 803 [details]
Diff of GLContextImpl

Merged pull request #105 from serebit/patch-1 96cdafe8a32b74fed853110d593521b49448e60d

and committed Julien's patch attachment 803 [details] as e6298fa34539fc5bb636fe10e0d4f083719dee64

"Bug 1357 Related: GLRendererQuirks NoSetSwapIntervalPostRetarget and NoDoubleBufferedPBuffer no more required for Mesa >= 18.2.2"