[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 06/12] accel/tcg: better handle memory constrained systems
From: |
Richard Henderson |
Subject: |
Re: [PATCH v2 06/12] accel/tcg: better handle memory constrained systems |
Date: |
Wed, 22 Jul 2020 12:02:59 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 |
On 7/22/20 9:44 AM, Daniel P. Berrangé wrote:
> OpenStack uses TCG in alot of their CI infrastructure for example
> and runs multiple VMs. If there's 4 VMs, that's another 4 GB of
> RAM usage just silently added on top of the explicit -m value.
>
> I wouldn't be surprised if this pushes CI into OOM, even without
> containers or cgroups being involved, as they have plenty of other
> services consuming RAM in the CI VMs.
I would hope that CI would also supply a -tb_size to go along with that -m
value. Because we really can't guess on their behalf.
> The commit 600e17b261555c56a048781b8dd5ba3985650013 talks about this
> minimizing codegen cache flushes, but doesn't mention the real world
> performance impact of eliminating those flushes ?
Somewhere on the mailing list was this info. It was so dreadfully slow it was
*really* noticable. Timeouts everywhere.
>
> Presumably this makes the guest OS boot faster, but what's the before
> and after time ? And what's the time like for values in between the
> original 32mb and the new 1 GB ?
But it wasn't "the original 32MB".
It was the original "ram_size / 4", until that broke due to argument parsing
ordering.
I don't know what CI usually uses, but I usually use at least -m 4G, sometimes
more. What's the libvirt default?
r~
- [PATCH v2 00/12] candidate fixes for 5.1-rc1 (testing, semihosting, OOM tcg, x86 fpu), Alex Bennée, 2020/07/22
- [PATCH v2 01/12] shippable: add one more qemu to registry url, Alex Bennée, 2020/07/22
- [PATCH v2 02/12] semihosting: defer connect_chardevs a little more to use serialx, Alex Bennée, 2020/07/22
- [PATCH v2 04/12] util: add qemu_get_host_physmem utility function, Alex Bennée, 2020/07/22
- [PATCH v2 03/12] semihosting: don't send the trailing '\0', Alex Bennée, 2020/07/22
- [PATCH v2 06/12] accel/tcg: better handle memory constrained systems, Alex Bennée, 2020/07/22
- Re: [PATCH v2 06/12] accel/tcg: better handle memory constrained systems, Richard Henderson, 2020/07/22
- Re: [PATCH v2 06/12] accel/tcg: better handle memory constrained systems, Alex Bennée, 2020/07/22
- Re: [PATCH v2 06/12] accel/tcg: better handle memory constrained systems, Daniel P . Berrangé, 2020/07/22
- Re: [PATCH v2 06/12] accel/tcg: better handle memory constrained systems,
Richard Henderson <=
- Re: [PATCH v2 06/12] accel/tcg: better handle memory constrained systems, Daniel P . Berrangé, 2020/07/23
- Re: [PATCH v2 06/12] accel/tcg: better handle memory constrained systems, Alex Bennée, 2020/07/23
- Re: [PATCH v2 06/12] accel/tcg: better handle memory constrained systems, Daniel P . Berrangé, 2020/07/23
- Re: [PATCH v2 06/12] accel/tcg: better handle memory constrained systems, Alex Bennée, 2020/07/23
[PATCH v2 05/12] util/oslib-win32: add qemu_get_host_physmem implementation, Alex Bennée, 2020/07/22