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

runtime: Add missing check in ResizeMemory for CH #9641

Merged
merged 1 commit into from
May 22, 2024

Conversation

cmaf
Copy link
Contributor

@cmaf cmaf commented May 15, 2024

ResizeMemory for Cloud Hypervisor is missing a check for the new requested memory being greater than the max hotplug size after alignment. Add the check, and since an earlier check for this setsrequested memory to the max hotplug size, do the same in the post-alignment check.

Fixes #9640

@katacontainersbot katacontainersbot added the size/tiny Smallest and simplest task label May 15, 2024
ResizeMemory for Cloud Hypervisor is missing a check for the new
requested memory being greater than the max hotplug size after
alignment. Add the check, and since an earlier check for this
setsrequested memory to the max hotplug size, do the same in the
post-alignment check.

Fixes kata-containers#9640

Signed-off-by: Chelsea Mafrica <chelsea.e.mafrica@intel.com>
@amshinde
Copy link
Member

/test

1 similar comment
@Apokleos
Copy link
Contributor

/test

@Apokleos Apokleos merged commit ce030d1 into kata-containers:main May 22, 2024
300 of 310 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test size/tiny Smallest and simplest task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ResizeMemory in runtime for CH missing check
6 participants