Skip to content

Commit ca20849

Browse files
roumingeriknordmark
authored andcommitted
Revert "hypervisor/kvm: reduce VMM overhead to 100MiB for containers"
This reverts commit 6f76179. We notice increased number of customer issues on the latest EVE versions, when QEMU process was killed by the OOM killer. So revert the overhead to 600M value. In the EVE mater the estimation algorithm is more sophisticated, please take a look here: d88091b ("New formula to estimate VMM memory overhead (x86)") The rule of thumb is to leave the overhead tuning to a customer using the corresponding field on the controller side. Signed-off-by: Roman Penyaev <[email protected]>
1 parent a7348df commit ca20849

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

pkg/pillar/hypervisor/kvm.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -464,11 +464,7 @@ func (ctx kvmContext) Task(status *types.DomainStatus) types.Task {
464464

465465
// TBD: Have a better way to calculate this number.
466466
// For now it is based on some trial-and-error experiments.
467-
// Container limit is reduced to 100MiB.
468467
func minVMMOverhead(config types.DomainConfig) int64 {
469-
if config.IsOCIContainer() {
470-
return 100 << 20 // Mb in bytes
471-
}
472468
return 600 << 20 // Mb in bytes
473469
}
474470

0 commit comments

Comments
 (0)