Thanks to visit codestin.com
Credit goes to github.com

Skip to content

vmm: config: Validate mergeable and shared are not both set - #8126

Merged
rbradford merged 1 commit into
cloud-hypervisor:mainfrom
rbradford:202605/mergeable-shareable-validation
May 4, 2026
Merged

rbradford merged 1 commit into
cloud-hypervisor:mainfrom
rbradford:202605/mergeable-shareable-validation

Conversation

@rbradford

Copy link
Copy Markdown
Member

KSM will not attempt to merge pages that are mapped as MAP_SHARED, so
configuring memory with both mergeable and shared options is invalid.
Add validation to reject configurations where both options are enabled
for memory or memory zones.

Signed-off-by: Rob Bradford [email protected]

KSM will not attempt to merge pages that are mapped as MAP_SHARED, so
configuring memory with both mergeable and shared options is invalid.
Add validation to reject configurations where both options are enabled
for memory or memory zones.

Signed-off-by: Rob Bradford <[email protected]>
@rbradford
rbradford requested a review from a team as a code owner May 1, 2026 16:28
@rbradford
rbradford added this pull request to the merge queue May 4, 2026
Merged via the queue into cloud-hypervisor:main with commit 80c56f7 May 4, 2026
41 checks passed
@likebreath likebreath added the bug-fix Bug fix to include in release notes label May 14, 2026
@rbradford
rbradford deleted the 202605/mergeable-shareable-validation branch June 12, 2026 10:36
btwiuse pushed a commit to btwiuse/snix that referenced this pull request Jun 19, 2026
 - address clippy lints
 - regenerate proto bindings
 - only set shared=on for //snix/boot
   (cloud-hypervisor/cloud-hypervisor#8126)

Change-Id: Id7dcb84be3998899e34a1ae9fc6db69aab4c953f
Reviewed-on: https://cl.snix.dev/c/snix/+/31308
Tested-by: besadii
Reviewed-by: Ilan Joselevich <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix Bug fix to include in release notes

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants