[pve-devel] [PATCH docs] ballooning example: improve wording

[pve-devel] [PATCH docs] ballooning example: improve wording

[Alexander Zeidler]

Signed-off-by: Alexander Zeidler <a.zeidler@proxmox.com>
---
 qm.adoc | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/qm.adoc b/qm.adoc
index fa6a772..c2392f3 100644
--- a/qm.adoc
+++ b/qm.adoc
@@ -689,8 +689,7 @@ database VM when spare RAM is available. For this you assign a Shares property
 of 3000 to the database VM, leaving the other VMs to the Shares default setting
 of 1000. The host server has 32GB of RAM, and is currently using 16GB, leaving 32
 * 80/100 - 16 = 9GB RAM to be allocated to the VMs. The database VM will get 9 *
-3000 / (3000 + 1000 + 1000 + 1000) = 4.5 GB extra RAM and each HTTP server will
-get 1.5 GB.
+3000 / (3000 + 1000 + 1000 + 1000) = 4.5 GB RAM and each HTTP server 1.5 GB.
 
 All Linux distributions released after 2010 have the balloon kernel driver
 included. For Windows OSes, the balloon driver needs to be added manually and can
-- 
2.39.2

Re: [pve-devel] [PATCH docs] ballooning example: improve wording

[Hannes Dürr]

I don't think this is an improvement of the wording, liked the previous 
version more.
Are there people who have not understood the wording?

On 1/16/24 14:11, Alexander Zeidler wrote:
> Signed-off-by: Alexander Zeidler <a.zeidler@proxmox.com>
> ---
>   qm.adoc | 3 +--
>   1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/qm.adoc b/qm.adoc
> index fa6a772..c2392f3 100644
> --- a/qm.adoc
> +++ b/qm.adoc
> @@ -689,8 +689,7 @@ database VM when spare RAM is available. For this you assign a Shares property
>   of 3000 to the database VM, leaving the other VMs to the Shares default setting
>   of 1000. The host server has 32GB of RAM, and is currently using 16GB, leaving 32
>   * 80/100 - 16 = 9GB RAM to be allocated to the VMs. The database VM will get 9 *
> -3000 / (3000 + 1000 + 1000 + 1000) = 4.5 GB extra RAM and each HTTP server will
> -get 1.5 GB.
> +3000 / (3000 + 1000 + 1000 + 1000) = 4.5 GB RAM and each HTTP server 1.5 GB.
>   
>   All Linux distributions released after 2010 have the balloon kernel driver
>   included. For Windows OSes, the balloon driver needs to be added manually and can

Re: [pve-devel] [PATCH docs] ballooning example: improve wording

[Alexander Zeidler]

On Tue, 2024-01-16 at 14:28 +0100, Hannes Dürr wrote:
> I don't think this is an improvement of the wording, liked the previous 
> version more.
> Are there people who have not understood the wording?
It's not an extra (additional) amount of memory for this VM, but its
total. Therefore, using the word "extra" can be misleading/confusing in
my opinion.

Re: [pve-devel] [PATCH docs] ballooning example: improve wording

[Alexander Zeidler]

On Tue, 2024-01-16 at 14:36 +0100, Alexander Zeidler wrote:
> On Tue, 2024-01-16 at 14:28 +0100, Hannes Dürr wrote:
> > I don't think this is an improvement of the wording, liked the previous 
> > version more.
> > Are there people who have not understood the wording?
> It's not an extra (additional) amount of memory for this VM, but its
> total. Therefore, using the word "extra" can be misleading/confusing in
> my opinion.
Actually it is the additional amount of memory and not the total.
Thanks! I've sent a new patch:
https://lists.proxmox.com/pipermail/pve-devel/2024-January/061396.html