Skip to content

Dynamic ram allocation #34

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.

Sign up for GitHub

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

Jump to bottom
Open
shuangwan01 wants to merge 5 commits into
base: celadon/r/mr0/stable
Choose a base branch
from
Open

Dynamic ram allocation #34

shuangwan01 wants to merge 5 commits into from

Conversation

@shuangwan01
Copy link
Contributor

@shuangwan01 shuangwan01 commented Nov 15, 2021

  1. Dynamic memory AVM allocation based on machines' memory configuration;
  2. Integration WIP patches:
  1. Yadong's CIV pause/resume patch;
  2. Shuang's socket connection hang patch.

YadongQi and others added 4 commits November 15, 2021 09:34
@YadongQi @shuangwan01
Remove duplicated script: lg_launcher.sh
97cc4c4 
Signed-off-by: Yadong Qi <yadong.qi@intel.com>
@YadongQi @shuangwan01
Refine service start/pause/resume process
5fda231 
In rc.civ.post, check sys_bootcompleted property at the begining,
then launch following daemons/services.

Forceing adb to use vsock.

Refine pause_civ/resume_civ script.

Signed-off-by: Yadong Qi <yadong.qi@intel.com>
@YadongQi @shuangwan01
refine for pgp
e02df44 
1. kill adb server when pause civ
2. increase vcpu number to 4

Signed-off-by: Yadong Qi <yadong.qi@intel.com>
@shuangwan01
Handle client clean up if socket connection closed abnormally.
4c751be 
It's observed the mwc client process killed by system when user
logout. This makes the socket closed abnormally without proper
communication for graceful socket shutdown.
The fix is to close the socket connection correctly if error
conditions detected.

Signed-off-by: Wan Shuang <shuang.wan@intel.com>
Tracked-On: OAM-100050
@shuangwan01 shuangwan01 force-pushed the dynamic_ram_allocation branch from 588e573 to 60b0f23 Compare November 15, 2021 11:37
@shuangwan01
Allocate 3G memory to AVM if system memory is less or equal ~8G.
60b0f23 
Per experiment, allocate smaller memory to AVM leads to better
user experience to whole system for lower memory configuration
machines. Added the dynamic AVM memory allocation for
performance optimization.

Signed-off-by: Wan Shuang <shuang.wan@intel.com>
Tracked-On: OAM-100077
@sysopenci sysopenci added the Stale Stale label for inactive open prs label Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@lchen43 lchen43 lchen43 approved these changes

@YadongQi YadongQi YadongQi approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Stale Stale label for inactive open prs

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@shuangwan01 @lchen43 @YadongQi @sysopenci