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

ResizeMemory in runtime for CH missing check #9640

Closed
cmaf opened this issue May 15, 2024 · 0 comments · Fixed by #9641
Closed

ResizeMemory in runtime for CH missing check #9640

cmaf opened this issue May 15, 2024 · 0 comments · Fixed by #9641
Assignees
Labels
bug Incorrect behaviour

Comments

@cmaf
Copy link
Contributor

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.

@cmaf cmaf added the bug Incorrect behaviour label May 15, 2024
@cmaf cmaf self-assigned this May 15, 2024
cmaf added a commit to cmaf/kata-containers that referenced this issue 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>
@katacontainersbot katacontainersbot moved this from To do to In progress in Issue backlog May 15, 2024
datadog-compute-robot pushed a commit to DataDog/kata-containers that referenced this issue Jun 11, 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Incorrect behaviour
Projects
Issue backlog
  
In progress
Development

Successfully merging a pull request may close this issue.

1 participant