Enabling v-sync doesn't work on X11 but works on Wayland

[DragonSWDev]

Hello, so basically after creating window surface and context from display I'm enabling or disabling v-sync based on flag passed to the function. Code looks like this:

match vsync_enabled {
            false => gl_surface.set_swap_interval(&gl_context, glutin::surface::SwapInterval::DontWait).unwrap(),
            true => gl_surface.set_swap_interval(&gl_context, glutin::surface::SwapInterval::Wait(NonZeroU32::new(1).unwrap())).unwrap()
        }

it works as expected on Wayland but doesn't work on X11 where v-sync is always disabled. I'm on Fedora 40 with NVIDIA GPU. "Sync to VBlank" option is enabled in NVIDIA Settings. Other OpenGL applications (e.g. glxgears) are running with FPS blocked to refresh rate. Am I missing something?

[kchibisov]

Are you using EGL or GLX here?

[DragonSWDev]

@kchibisov How can I check this?

[kchibisov]

Print https://docs.rs/glutin/latest/glutin/display/trait.GlDisplay.html#tymethod.version_string .

[spearman]

I have the same issue (GLX)

Just confirmed that vsync was working with glutin v0.29 (using ContextBuilder.with_vsync(true))

I've tried calling SwapIntervalEXT with a NULL pointer for the display here and there was no error:

glutin/glutin/src/api/glx/surface.rs

Line 257 in 73ce4f9

extra.SwapIntervalEXT(self.display.inner.raw.cast(), self.raw, interval as _);

Also if I bypass the SwapIntervalEXT call it will go on to call SwapIntervalSGI and "succeed" (applied = true) but there is still no vsync.

[kchibisov]

You can have vsync force disabled inside the driver itself.

[spearman]

You can have vsync force disabled inside the driver itself.

It's probably not the case though if running glutin v0.29.1 with ContextBuilder.with_vsync(true) has vsync enabled?

[spearman]

Querying the SWAP_INTERVAL_EXT segfaults:

                self.display.inner.glx.QueryDrawable(                            
                     self.display.inner.raw.cast(),                               
                     self.raw,                                                    
                     glx_extra::SWAP_INTERVAL_EXT as i32,                         
                     &mut swap,                                                   
                 );

Querying normal GLX attributes (e.g. GLX_WIDTH) works fine.

[spearman]

You can have vsync force disabled inside the driver itself.

It's probably not the case though if running glutin v0.29.1 with ContextBuilder.with_vsync(true) has vsync enabled?

So it looks like with glutin v0.29.1 creating the GLX prototype context fails with "NoAvailablePixelFormat" so it actually falls back to EGL. With EGL it says that the default is to have Vsync enabled, and building with ContextBuilder.with_vsync(false) successfully disables it.

Confirmed that vsync does work with EGL on glutin v0.32.1 and setting the swap interval to DontWait also successfully disables it. So it seems to be a problem with GLX.