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

Give userspace 3GB of address space #22

Closed
wants to merge 1 commit into from

Conversation

ryao
Copy link
Contributor

@ryao ryao commented Sep 24, 2018

This patch is a one-liner from here:

https://startux.de/wine-patch-to-use-3gb-userspace.html

It is fairly old, but it allows userspace program to use up to 3GB of
RAM instead of 2GB of RAM. As the blog post says, this differs from the
Windows behavior, but apparently, Microsoft's own advice to developers
is to give userspace programs access to 4GB of address space on 64-bit
Windows:

https://blogs.msdn.microsoft.com/oldnewthing/20050601-24/?p=35483

Changing the restriction to 3GB of address space should be okay. I
suspect that upstream Wine won't like this change, but it should be fine
for proton. It would probably help DXVK.

Signed-off-by: Richard Yao ryao@gentoo.org

This patch is a one-liner from here:

https://startux.de/wine-patch-to-use-3gb-userspace.html

It is fairly old, but it allows userspace program to use up to 3GB of
RAM instead of 2GB of RAM. As the blog post says, this differs from the
Windows behavior, but apparently, Microsoft's own advice to developers
is to give userspace programs access to 4GB of address space on 64-bit
Windows:

https://blogs.msdn.microsoft.com/oldnewthing/20050601-24/?p=35483

Changing the restriction to 3GB of address space should be okay. I
suspect that upstream Wine won't like this change, but it should be fine
for proton. It would probably help DXVK.

Signed-off-by: Richard Yao <ryao@gentoo.org>
@ryao
Copy link
Contributor Author

ryao commented Sep 24, 2018

@doitsujin Given that the main purpose of this is to help DXVK, would you give your opinion on this change?

@geneishouko
Copy link

That line is #ifdef'd for i386 arch. The 64 bit wine loader does not have this limitation and still runs 32 bit applications.

@ryao
Copy link
Contributor Author

ryao commented Sep 24, 2018

@geneishouko That just made me realize that Steamplay probably requires 64-bit versions. This probably would help someone, but not those using Proton. Sorry for the noise.

@ryao ryao closed this Sep 24, 2018
@mirh
Copy link

mirh commented Sep 24, 2018

No it doesn't ValveSoftware/Proton#1358

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.

None yet

3 participants