* Re: [PVE-User] [pve-devel] [PATCH qemu-server 08/10] memory: add virtio-mem support
[not found] ` <15f5554d-3708-ac78-d2e2-1d797e55e211@proxmox.com>
@ 2022-12-20 12:16 ` DERUMIER, Alexandre
0 siblings, 0 replies; only message in thread
From: DERUMIER, Alexandre @ 2022-12-20 12:16 UTC (permalink / raw)
To: pve-user, f.ebner
Le mardi 20 décembre 2022 à 11:26 +0100, Fiona Ebner a écrit :
> Isn't ($MAX_MEM - $static_memory) / 32000 always strictly greater
> than
> 1? And if it could get smaller than 1, we also might have issues with
> the int()+1 approach, because the result of the first log() will
> become
> negative.
>
> To be on the safe side we could just move the minimum check up:
>
> my $blocksize = ($MAX_MEM - $static_memory) / 32000;
> $blocksize = 2 if $blocksize < 2;
> $blocksize = 2**(ceil(log($blocksize)/log(2)));
I think your are right.
I totally forget than mem was in bytes,
so the minimum blocksize is 2048
with a MAX_MEM of 64gb, the minimum blocksize is 2048.
(I remember now that I wanted 64GB minimum to have transparent huge
working out of the box).
if MAX_MEM was allowed 32gb ,the minimum blocksize with ceil is 1024.
so we need to force it to 2048
I'll rework the patch, thanks !
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2022-12-20 12:23 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20221209192726.1499142-1-aderumier@odiso.com>
[not found] ` <20221209192726.1499142-9-aderumier@odiso.com>
[not found] ` <87423a6b-a17e-5ea4-9176-cd81e96c5693@proxmox.com>
[not found] ` <7b9306c429440304fb37601ece5ffdbad0b90e5f.camel@groupe-cyllene.com>
[not found] ` <15f5554d-3708-ac78-d2e2-1d797e55e211@proxmox.com>
2022-12-20 12:16 ` [PVE-User] [pve-devel] [PATCH qemu-server 08/10] memory: add virtio-mem support DERUMIER, Alexandre
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox