Skip to content
This repository has been archived by the owner on Sep 18, 2020. It is now read-only.

image_to_vm: Cleanup memory settings #808

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

glevand
Copy link
Contributor

@glevand glevand commented Apr 18, 2018

Fix up inconsistent use of FLAGS_mem and '_get_vm_opt MEM', add function get_vm_mem to get more memory for arm64, and remove unused files in the lib directory.

glevand added 3 commits May 23, 2018 09:53
Fixup inconsistent use of FLAGS_mem and '_get_vm_opt MEM'. The
user supplied FLAGS_mem is passed to write_vm_conf, and it
should take presidence over built-in values returned by
'_get_vm_opt MEM'.

Signed-off-by: Geoff Levand <[email protected]>
ARM64 builds tend to run out of memory with only 1024 MiB, so
replace the IMG_DEFAULT_MEM constant with a new function
get_vm_mem that returns a default memory size based
the board.

Signed-off-by: Geoff Levand <[email protected]>
The replacement of DEFAULT_MEM with get_vm_mem makes all of this
unused.

Signed-off-by: Geoff Levand <[email protected]>
@glevand
Copy link
Contributor Author

glevand commented May 23, 2018

Rebased to latest.

Copy link
Contributor

@ajeddeloh ajeddeloh left a comment

Choose a reason for hiding this comment

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

LGTM. I'm all for cleaning out cruft. Tests this with a build_image and image_to_vm and it worked.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants