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

Virtual Memory #25

Closed
ghost opened this issue Apr 6, 2014 · 4 comments · Fixed by #45
Closed

Virtual Memory #25

ghost opened this issue Apr 6, 2014 · 4 comments · Fixed by #45

Comments

@ghost
Copy link

ghost commented Apr 6, 2014

Implemented hardware translations so that it's easier to allocate and manage memory

@ghost ghost added the enhancement label Apr 6, 2014
@ghost
Copy link
Author

ghost commented Apr 6, 2014

So far I have mapped physical memory (exception stacks, memory mapped io et all), kernel code and kernel data into the virtual address space. Working on enabling the MMU to recognize the page tables, but not sure how to test if it's actually translating addresses correctly.

@NULLx76
Copy link
Collaborator

NULLx76 commented Mar 2, 2020

Virtual memory kind of "works" but is in dire need of an overhaul.

The datastructures for the VAS overwrote the L1 page table, and the system only worked because we overwrote the L1 Page table after the VM did. We discovered this bug by adding page table entries before we initialized the VM, we have currently just limited the amount the VM allocates (1024 instead of 4096 VAS').

I would also propose to relocate the page table to physical address 0x4000 as there is space there anyways between the IVT and the kernel start.

@NULLx76
Copy link
Collaborator

NULLx76 commented Mar 6, 2020

This is currently being worked on in the #45 branch

@NULLx76 NULLx76 linked a pull request Mar 6, 2020 that will close this issue
@NULLx76 NULLx76 added this to the Virtual Memory milestone Mar 10, 2020
@NULLx76
Copy link
Collaborator

NULLx76 commented Mar 11, 2020

Resolved by #45

@NULLx76 NULLx76 closed this as completed Mar 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant