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

Allow plugins to use PRIVATE memory instead of SHARED on requests #2086

Merged
merged 6 commits into from
Sep 27, 2024

Conversation

PabloMK7
Copy link
Collaborator

This PR:

  • Modifies the svcMapProcessMemoryEx svc so that it takes an extra flags argument. This change was made in a way that shouldn't break existing plugins or applications.
    • Added MAPEXFLAGS_PRIVATE flag, which maps the specified memory as PRIVATE instead of SHARED.
  • Allows plugins to be mapped with PRIVATE memory instead of SHARED with a flag in the .3gx header. This allows plugins to use socket and http services without additional hacks.

@TuxSH TuxSH merged commit 3253fdb into master Sep 27, 2024
@TuxSH TuxSH deleted the privatemapex branch September 27, 2024 20:36
@LittleCube-hax
Copy link
Contributor

Well, at least setting the .3gx header bit in the plugin works on my end, but how are users expected to set this bit normally?

@TuxSH
Copy link
Member

TuxSH commented Sep 27, 2024

AFAIK https://github.com/LumaTeam/Luma3DS/pull/2086/files#diff-24fcd400ed63816ed96d0b98cb41be65cb587d12a067401a7b46a89fd95b1b3eR66 if I read it correctly during review (trusting pablo on that as he said he tested it)

@PabloMK7
Copy link
Collaborator Author

PabloMK7 commented Sep 27, 2024

Well, at least setting the .3gx header bit in the plugin works on my end, but how are users expected to set this bit normally?

https://gitlab.com/thepixellizeross/3gxtool/-/commit/b666d94e566e3e63e5260cd098bc060c70516470
(Update 3gxtool to v1.3)

@LittleCube-hax
Copy link
Contributor

Ah, I was looking at the wrong 3gxtool repo, I see now.

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.

3 participants