mirror of
https://github.com/cloud-hypervisor/cloud-hypervisor.git
synced 2026-08-05 02:19:16 +00:00
vmm: Remove 'mergeable' from memory zones
The flag 'mergeable' should only apply to the entire guest RAM, which is why it is removed from the MemoryZoneConfig as it is defined as a global parameter at the MemoryConfig level. Signed-off-by: Sebastien Boeuf <sebastien.boeuf@intel.com>
This commit is contained in:
committed by
Samuel Ortiz
parent
be8f40f2b6
commit
ad5d0e4713
+1
-1
@@ -120,7 +120,7 @@ fn create_app<'a, 'b>(
|
|||||||
.help(
|
.help(
|
||||||
"User defined memory zone parameters \
|
"User defined memory zone parameters \
|
||||||
\"size=<guest_memory_region_size>,file=<backing_file>,\
|
\"size=<guest_memory_region_size>,file=<backing_file>,\
|
||||||
mergeable=on|off,shared=on|off,hugepages=on|off\"",
|
shared=on|off,hugepages=on|off\"",
|
||||||
)
|
)
|
||||||
.takes_value(true)
|
.takes_value(true)
|
||||||
.min_values(1)
|
.min_values(1)
|
||||||
|
|||||||
@@ -349,8 +349,6 @@ pub struct MemoryZoneConfig {
|
|||||||
#[serde(default)]
|
#[serde(default)]
|
||||||
pub file: Option<PathBuf>,
|
pub file: Option<PathBuf>,
|
||||||
#[serde(default)]
|
#[serde(default)]
|
||||||
pub mergeable: bool,
|
|
||||||
#[serde(default)]
|
|
||||||
pub shared: bool,
|
pub shared: bool,
|
||||||
#[serde(default)]
|
#[serde(default)]
|
||||||
pub hugepages: bool,
|
pub hugepages: bool,
|
||||||
@@ -432,7 +430,6 @@ impl MemoryConfig {
|
|||||||
parser
|
parser
|
||||||
.add("size")
|
.add("size")
|
||||||
.add("file")
|
.add("file")
|
||||||
.add("mergeable")
|
|
||||||
.add("shared")
|
.add("shared")
|
||||||
.add("hugepages");
|
.add("hugepages");
|
||||||
parser.parse(memory_zone).map_err(Error::ParseMemoryZone)?;
|
parser.parse(memory_zone).map_err(Error::ParseMemoryZone)?;
|
||||||
@@ -443,11 +440,6 @@ impl MemoryConfig {
|
|||||||
.unwrap_or(ByteSized(DEFAULT_MEMORY_MB << 20))
|
.unwrap_or(ByteSized(DEFAULT_MEMORY_MB << 20))
|
||||||
.0;
|
.0;
|
||||||
let file = parser.get("file").map(PathBuf::from);
|
let file = parser.get("file").map(PathBuf::from);
|
||||||
let mergeable = parser
|
|
||||||
.convert::<Toggle>("mergeable")
|
|
||||||
.map_err(Error::ParseMemoryZone)?
|
|
||||||
.unwrap_or(Toggle(false))
|
|
||||||
.0;
|
|
||||||
let shared = parser
|
let shared = parser
|
||||||
.convert::<Toggle>("shared")
|
.convert::<Toggle>("shared")
|
||||||
.map_err(Error::ParseMemoryZone)?
|
.map_err(Error::ParseMemoryZone)?
|
||||||
@@ -462,7 +454,6 @@ impl MemoryConfig {
|
|||||||
zones.push(MemoryZoneConfig {
|
zones.push(MemoryZoneConfig {
|
||||||
size,
|
size,
|
||||||
file,
|
file,
|
||||||
mergeable,
|
|
||||||
shared,
|
shared,
|
||||||
hugepages,
|
hugepages,
|
||||||
});
|
});
|
||||||
|
|||||||
@@ -362,7 +362,6 @@ impl MemoryManager {
|
|||||||
let zones = vec![MemoryZoneConfig {
|
let zones = vec![MemoryZoneConfig {
|
||||||
size: config.size,
|
size: config.size,
|
||||||
file: None,
|
file: None,
|
||||||
mergeable: config.mergeable,
|
|
||||||
shared: config.shared,
|
shared: config.shared,
|
||||||
hugepages: config.hugepages,
|
hugepages: config.hugepages,
|
||||||
}];
|
}];
|
||||||
|
|||||||
Reference in New Issue
Block a user