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

Add virtmemFindAliasMemory to virtmem APIs #501

Closed
wants to merge 1 commit into from
Closed

Add virtmemFindAliasMemory to virtmem APIs #501

wants to merge 1 commit into from

Conversation

DCNick3
Copy link

@DCNick3 DCNick3 commented Nov 1, 2020

No description provided.

@fincs
Copy link
Contributor

fincs commented Nov 1, 2020

This won't work as intended, because _memregionFindRandom explicitly checks that the returned address isn't within the Alias region.

What is the intended use case of this PR? I suppose it is related to switchbrew/nx-hbloader#35?

@DCNick3
Copy link
Author

DCNick3 commented Nov 1, 2020

Didn't notice that, sorry.

Yes, it is related =)

My intended use is as follows: I want to reserve a large region of virtual address space (probably 4 GiB) and later map physical memory to parts of it. This would allow to write efficient emulation for 32-bit systems supporting virtual memory by simply adding the reserved region base to emulated system virtual address. This is how qemu user emulation basically works.

I want it to emulate (small subset of) windows 95 APIs to run Heroes of Might and Magic III on hos.

@fincs
Copy link
Contributor

fincs commented Nov 1, 2020

I see; that is indeed quite an interesting project.

We haven't tackled support for PhysicalMemory SVCs yet; but it is indeed in the cards.

@fincs
Copy link
Contributor

fincs commented Nov 7, 2020

I'm closing this PR. However I'm opening an issue that is specifically about the future support for PhysicalMemory SVCs: #503

@fincs fincs closed this Nov 7, 2020
@DCNick3 DCNick3 deleted the virtmemFindAliasMemory branch November 8, 2020 06:29
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

2 participants