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

nvfbc cuda regression: picture does not update #1763

Closed
totaam opened this issue Feb 5, 2018 · 7 comments
Closed

nvfbc cuda regression: picture does not update #1763

totaam opened this issue Feb 5, 2018 · 7 comments

Comments

@totaam
Copy link
Collaborator

totaam commented Feb 5, 2018

Issue migrated from trac ticket # 1763

component: encodings | priority: major | resolution: fixed

2018-02-05 03:45:19: antoine created the issue


Problem spotted on win32, may also affect Linux (TBC).

The code runs and it looks like we're downloading the frames from video memory, but when scrolling detection runs it finds that the picture has not changed and so we end up not generating any screen updates.

Forcing scrolling detection off does not help: the video encoders also find the picture unchanged. This also applies to the NVENC encoder when using the picture data found in the CUDA buffer which would indicate that the API is just not capturing new data. (rather than a download / usage problem on our side)
This could be a driver bug, or a problem with the API uncovered by newer drivers.

Related tickets:

@totaam
Copy link
Collaborator Author

totaam commented Feb 5, 2018

2018-02-05 03:47:18: antoine changed status from new to assigned

@totaam
Copy link
Collaborator Author

totaam commented Feb 5, 2018

2018-02-05 03:47:18: antoine edited the issue description

@totaam
Copy link
Collaborator Author

totaam commented Feb 5, 2018

2018-02-05 04:22:59: antoine commented


r18299: CUDA buffer mode no longer enabled by default

TODO:

  • check linux (move graphics cards around... sigh)
  • fix + backports if needed
  • also backport r18300?

@totaam
Copy link
Collaborator Author

totaam commented Apr 28, 2018

2018-04-28 18:51:05: antoine commented


  • win32 fixed in r19103: we no longer push + pop the cuda context and just keep it live until we close it
  • r19104 re-enables cuda by default

@totaam
Copy link
Collaborator Author

totaam commented Apr 29, 2018

2018-04-29 07:17:48: antoine changed status from assigned to closed

@totaam
Copy link
Collaborator Author

totaam commented Apr 29, 2018

2018-04-29 07:17:48: antoine set resolution to fixed

@totaam
Copy link
Collaborator Author

totaam commented Apr 29, 2018

2018-04-29 07:17:48: antoine commented


Linux was not affected.

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

No branches or pull requests

1 participant