[pve-devel] [PATCH qemu-server] schema: memory: be precise that unit is binary prefix

[pve-devel] [PATCH qemu-server] schema: memory: be precise that unit is binary prefix

[Fiona Ebner]

In the web UI, this was fixed years ago by pve-manager commit c11c4a40
("fix #1631: change units to binary prefix").

Quickly checked with the 'query-memory-size-summary' QMP command that
this is actually the case.

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---
 PVE/QemuServer.pm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm
index 99242d54..6a29d1bf 100644
--- a/PVE/QemuServer.pm
+++ b/PVE/QemuServer.pm
@@ -342,7 +342,7 @@ my $confdesc = {
     memory => {
 	optional => 1,
 	type => 'integer',
-	description => "Amount of RAM for the VM in MB. This is the maximum available memory when"
+	description => "Amount of RAM for the VM in MiB. This is the maximum available memory when"
 	    ." you use the balloon device.",
 	minimum => 16,
 	default => 512,
@@ -350,7 +350,7 @@ my $confdesc = {
     balloon => {
 	optional => 1,
 	type => 'integer',
-	description => "Amount of target RAM for the VM in MB. Using zero disables the ballon driver.",
+	description => "Amount of target RAM for the VM in MiB. Using zero disables the ballon driver.",
 	minimum => 0,
     },
     shares => {
-- 
2.30.2

[pve-devel] applied: [PATCH qemu-server] schema: memory: be precise that unit is binary prefix

[Thomas Lamprecht]

Am 23/01/2023 um 10:46 schrieb Fiona Ebner:
> In the web UI, this was fixed years ago by pve-manager commit c11c4a40
> ("fix #1631: change units to binary prefix").
> 
> Quickly checked with the 'query-memory-size-summary' QMP command that
> this is actually the case.
> 
> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
> ---
>  PVE/QemuServer.pm | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
>

applied, thanks!