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

ladybird: add GPU support #322203

Closed
nonetrix opened this issue Jun 24, 2024 · 7 comments
Closed

ladybird: add GPU support #322203

nonetrix opened this issue Jun 24, 2024 · 7 comments
Labels
0.kind: enhancement Add something new

Comments

@nonetrix
Copy link
Contributor

nonetrix commented Jun 24, 2024

Describe the bug

The Ladybird package assumes it doesn't need GPU so if you run with --enable-gpu-painting it will return Configured to use GPU painter, but current platform does not have accelerated graphics I have had similar issues to this with other packages so I think that it is likely to be a issue with said Nix package

Steps To Reproduce

Steps to reproduce the behavior:

  1. Run Ladybird with Ladybird --enable-gpu-painting
  2. Observe

Expected behavior

It runs with GPU support and doesn't make errors

Screenshots

image

Additional context

I have AMD RX 6800 GPU and this seems like a experimental feature, also I am pretty sure QT can use the GPU as well so I think this might enable that? Benefit of that likely to be small at best but still

Notify maintainers

@fgaz

Metadata

Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
this path will be fetched (0.01 MiB download, 0.05 MiB unpacked):
  /nix/store/d3dzfy4amjl826fb8j00qp1d9887h7hm-stdenv-linux
copying path '/nix/store/d3dzfy4amjl826fb8j00qp1d9887h7hm-stdenv-linux' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 6.9.6, NixOS, 24.11 (Vicuna), 24.11.20240622.a71e967`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - channels(root): `"home-manager, nixos"`
 - nixpkgs: `/nix/store/yqy82fn77fy3rv7lpwa9m11w3a2nnqg5-source`

Add a 👍 reaction to issues you find important.

@nonetrix nonetrix added the 0.kind: bug Something is broken label Jun 24, 2024
@nonetrix
Copy link
Contributor Author

I assume Sika needs to be included? Just a guess

@fgaz
Copy link
Member

fgaz commented Jul 2, 2024

Currently we disable hardware acceleration at compile time:

# FIXME: Enable this when launching with the commandline flag --enable-gpu-painting doesn't fail calling eglBindAPI on GNU/Linux
"-DENABLE_ACCELERATED_GRAPHICS=OFF"

I'll try to enable it again and see if it works, after I do these higher priority tasks:

@fgaz fgaz added 0.kind: enhancement Add something new and removed 0.kind: bug Something is broken labels Jul 2, 2024
@fgaz fgaz changed the title Add GPU support to Ladybird package ladybird: add GPU support Jul 2, 2024
@nonetrix
Copy link
Contributor Author

nonetrix commented Jul 2, 2024

Ah sorry didn't catch that

@ADKaster
Copy link
Contributor

ADKaster commented Jul 5, 2024

Note that the ENABLE_ACCELERATED_GRAPHICS flag (which uses OpenGL) is planned to be deprecated for using Skia to access the GPU. Ideally with a Vulkan dep on Linux and Metal dep on macOS.

@ADKaster
Copy link
Contributor

ADKaster commented Jul 5, 2024

Ah. By 'soon', I actually mean 'already' LadybirdBrowser/ladybird#428

@nonetrix
Copy link
Contributor Author

nonetrix commented Jul 6, 2024

That's what I assumed that flag did sorry

@fgaz
Copy link
Member

fgaz commented Jul 6, 2024

Thank you for informing us @ADKaster. Then I'll consider this closed when I finish #325045

@fgaz fgaz closed this as completed Jul 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.kind: enhancement Add something new
Projects
None yet
Development

No branches or pull requests

3 participants