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

Fix AMD video HW Accel and lib path, #1079 #1080

Open
wants to merge 8 commits into
base: main-v2
Choose a base branch
from

Conversation

Originalimoc
Copy link
Contributor

No description provided.

@unknownskl
Copy link
Owner

Can you please elaborate what this PR should fix? I see in the code that you are checking the gpu internals and im kind of wondering which option you are trying to activate? Maybe there is another way to achieve the same goal which is more electron friendly.

@Originalimoc
Copy link
Contributor Author

Originalimoc commented Dec 28, 2023

The actual change is flatpak/io.github.unknownskl.greenlight.yml and electron to 25. First fix wrong path, second because 27 removed hardware decoder VDADecoder for H264. GPU info is in comment for possible future debugging.

@unknownskl
Copy link
Owner

I still dont fully understand what problem you are experiencing and what this should fix. As for the downgrade to electron 25 i would prefer not to pin the version of the Framework. Electron 25 is EOL in about 5/6 days from now and that would expose additional security risks and compatibility issues in the future.

@Originalimoc
Copy link
Contributor Author

Since this is a single purpose userspace app I don't think there're additional attack surface if future exploits are found in electron. Without this hw decoder patch it's unusable on Steam Deck without hammering the CPU thus battery.

@unknownskl
Copy link
Owner

Keeping dependencies up to date is quite important in the long run. On my steam deck the app only uses between 6-8 watts while streaming and results in a suggested battery life of +- 5 hours (not real benchmarks though). The cpu usage sits between 20-40%. Do you encounter different numbers on your Steam deck?

Do you have numbers on how much this would improve cpu wise and in terms of energy usage?

@Originalimoc
Copy link
Contributor Author

Originalimoc commented Jan 11, 2024

a

Around 7W. 1080P AVC decoder is not that hard on modern CPU. But bigger problem is SW decoder much higher latency on top of already awful Microsoft's streaming implementation on Xbox Series..
No idea how they messed up low latency encoder on its Zen 2 + RDNA 2 combo and no HEVC, no 4K. no HDR, only 1080P + AVC. On PC this GPU combo Sunshine + Moonlight can get full chain latency under 20ms(Click + Encode + over Network + Decode)...

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.

2 participants