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

For some reason NVIDIA drivers do not re-enable vsync once disabled #182

Open
wants to merge 1 commit into
base: next
Choose a base branch
from

Conversation

morgoth6
Copy link

@morgoth6 morgoth6 commented Jun 1, 2019

It seems if vsync is disabled at beginning of vsync_init() it is not enabled later (no error messages, etc) with "vsync = true" I guess compton thinks everything is OK, but glxgears runs ~1000FPS with noticable CPU usage.

After disable initia vsync for NV everything sees to work just fine. I got smooth 60FPS with almost no CPU used.

Tested on Precision 5510 with 430.14 on Quadro M1000M

@codecov
Copy link

codecov bot commented Jun 1, 2019

Codecov Report

Merging #182 into next will increase coverage by 0.34%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             next     #182      +/-   ##
==========================================
+ Coverage   12.53%   12.87%   +0.34%     
==========================================
  Files          42       42              
  Lines        8112     7858     -254     
==========================================
- Hits         1017     1012       -5     
+ Misses       7095     6846     -249
Impacted Files Coverage Δ
src/vsync.c 5.26% <ø> (-1.64%) ⬇️
src/config.h 0% <0%> (-21.43%) ⬇️
src/log.h 11.11% <0%> (-11.12%) ⬇️
src/config_libconfig.c 55.11% <0%> (-0.53%) ⬇️
src/render.c 3.24% <0%> (-0.1%) ⬇️
src/backend/gl/glx.c 12.4% <0%> (-0.1%) ⬇️
src/dbus.c 0% <0%> (ø) ⬆️
src/region.h 0% <0%> (ø) ⬆️
src/backend/gl/gl_common.c 0% <0%> (ø) ⬆️
src/backend/gl/gl_common.h 0% <0%> (ø) ⬆️
... and 14 more

@yshui
Copy link
Owner

yshui commented Jun 1, 2019

First of all, thanks for the pull request. I have some questions though:

Are you sure vsync is really set to "true"? What if you run compton --vsync --config /dev/null?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants