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

pmm: fix initial physical memory frames processing #46

Merged
merged 2 commits into from
Aug 16, 2020

Conversation

wipawel
Copy link
Contributor

@wipawel wipawel commented Aug 16, 2020

It is important to assign initial 4K and 2M frames to the front of
available frames list. Initial virtual memory mapping is small and
the first frames are being used by pagetables initialization.

Always consume entire region of physical memory by adding tailing 4K
frames till the end of the region.

Detect properly incorrectly processed regions and panic with some
details for debugging.

Signed-off-by: Pawel Wieczorkiewicz [email protected]

*Issue #35

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

It is important to assign initial 4K and 2M frames to the front of
available frames list.  Initial virtual memory mapping is small and
the first frames are being used by pagetables initialization.

Always consume entire region of physical memory by adding tailing 4K
frames till the end of the region.

Detect properly incorrectly processed regions and panic with some
details for debugging.

Signed-off-by: Pawel Wieczorkiewicz <[email protected]>
@wipawel wipawel added the bug Something isn't working label Aug 16, 2020
Copy link
Contributor

@dkgupta-amzn dkgupta-amzn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wipawel wipawel merged commit 22c0105 into KernelTestFramework:mainline Aug 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants