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

07 memory solution #137

Open
wants to merge 7 commits into
base: Serhii.Chekanov
Choose a base branch
from

Conversation

SergeyChekanov
Copy link

Please review pull request 07 lesson

Add homework readme lesson 6

Signed-off-by: Chekanov Serhii <[email protected]>
Add solution, absolute time return module

Signed-off-by: Chekanov Serhii <[email protected]>
Add solution, CPU load calc module

Signed-off-by: Chekanov Serhii <[email protected]>
Implementation of relation time kernel module

Signed-off-by: Chekanov Serhii <[email protected]>
Add homework lesson 7

Signed-off-by: Chekanov Serhii <[email protected]>
Memory allocating test program

Signed-off-by: Chekanov Serhii <[email protected]>
Memory allocating test kernel module

Signed-off-by: Chekanov Serhii <[email protected]>
@SergeyChekanov SergeyChekanov added the ready The PR is ready for review (set by author). label Dec 8, 2019
1, 0x3, 0.000000113, 0.000000093
2, 0x4, 0.000000128, 0.000000105
2, 0x5, 0.000000140, 0.000000128
pt = NULL
Copy link
Contributor

Choose a reason for hiding this comment

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

What is the maximum memory size allocated by get_free_pages?

Copy link
Author

@SergeyChekanov SergeyChekanov Jan 8, 2020

Choose a reason for hiding this comment

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

Allocated 2^5 pages

Copy link
Contributor

Choose a reason for hiding this comment

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

Please check the correctness of this number.

@yekovalyov yekovalyov added changes requested The PR review revealed issues which should be fixed (set by reviewer) and removed ready The PR is ready for review (set by author). labels Dec 16, 2019
@SergeyChekanov SergeyChekanov added ready The PR is ready for review (set by author). and removed changes requested The PR review revealed issues which should be fixed (set by reviewer) labels Jan 9, 2020
@yekovalyov yekovalyov added changes requested The PR review revealed issues which should be fixed (set by reviewer) and removed ready The PR is ready for review (set by author). labels Jan 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes requested The PR review revealed issues which should be fixed (set by reviewer)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants