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

tetrio-desktop: run with --in-process-gpu flag #309975

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

Conversation

qubitnano
Copy link
Contributor

Description of changes

Fixes #309293

Starting with electron 29, tetrio's webGL will crash when hovering over tooltips on some setups. Upstream has recommended the --in-process-gpu flag as a workaround.

Fixes the crash on my problem machine, with no noticeable performance difference for my unaffected machine.

Tested with both electron 29 and 30.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@huantianad
Copy link
Contributor

huantianad commented May 8, 2024

Ugh I hate all the electron/chromium flags that exist with like no documentation lol, there's probably more applied in the Tetrio code itself that messes with all of this, I wonder if any of those could be the actual root cause of the issue.

Should we make sure that this doesn't cause any performance regressions on more systems without this issue?

@qubitnano
Copy link
Contributor Author

I'll mark as draft for further testing. Maybe we can put this option behind a flag defaulting to false that has to be overridden?

@qubitnano qubitnano marked this pull request as draft May 8, 2024 01:30
@huantianad
Copy link
Contributor

Sorry, forgot about this. I'll test performance on my system, but if it fixes some issues without performance regression on others, probably worth.

What does this flag actually do btw?

@qubitnano qubitnano marked this pull request as ready for review May 29, 2024 21:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TETR.IO (PLUS) - "WebGL crashes" when showing tooltips
2 participants