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

star citizen gamefixes #132

Closed
wants to merge 1 commit into from

Conversation

mactan-sc
Copy link
Contributor

  • AMD environment variable is no longer needed

  • Nvidia variable enables Vulkan but prevents DLSS. Alternate method of patching libcuda has become favorable as it enables both Vulkan and DLSS

jp7677/dxvk-nvapi#174 (comment)

https://github.com/starcitizen-lug/knowledge-base/wiki/Troubleshooting#dlssdeep-learning-super-sampling--vulkan

@R1kaB3rN
Copy link
Member

R1kaB3rN commented Sep 20, 2024

In the NVIDIA case, supposing that fix works, have you already explored being able to automate the patching of libcuda.so in a fix so users don't need to do it manually? I think we should strive to do that here.

For instance, what we can do is create another copy, patch it with those instructions, then mount that copy at the same install path in a nested bwrap. Or copy the file to /tmp, patch it, then set LD_PRELOAD to the patched path. Certainly, that would be better practice than instructing users to patch that file directly in the system path. In fact, the comment says to copy the .so file elsewhere.

Just copy libcuda.so to some folder and run the command echo -ne $(od -An -tx1 -v libcuda.so | tr -d '\n' | sed -e 's/00 00 00 f8 ff 00 00 00/00 00 00 f8 ff ff 00 00/g' -e 's/ /\x/g') > libcuda.patched.so... a libcuda.patched.so will be created, then... use LD_PRELOAD env to load libcuda.patched.so( like... LD_PRELOAD=/path/to/the/libcuda.patched.so ).

@mactan-sc
Copy link
Contributor Author

right the star citizen LUG's recommendation presently is to copy the .so and use the LD_PRELOAD env var to use the modified file. Would those sort of file operations be welcome in a protonfix or is that best left to the end user to take action?

ProjectSynchro added a commit to ProjectSynchro/umu-protonfixes that referenced this pull request Sep 20, 2024
…ure patched libcuda for NVIDIA

This expands Open-Wine-Components#132 by implementing the patch that was mentioned, and integrates that into the protonfix.
@R1kaB3rN
Copy link
Member

Closing as #136 supersedes this

@R1kaB3rN R1kaB3rN closed this Sep 22, 2024
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