|
From: | David Hildenbrand |
Subject: | Re: [PATCH v4 1/1] oslib-posix: initialize backend memory objects in parallel |
Date: | Wed, 31 Jan 2024 19:22:56 +0100 |
User-agent: | Mozilla Thunderbird |
On 31.01.24 17:53, Mark Kanda wrote:
QEMU initializes preallocated backend memory as the objects are parsed from the command line. This is not optimal in some cases (e.g. memory spanning multiple NUMA nodes) because the memory objects are initialized in series. Allow the initialization to occur in parallel (asynchronously). In order to ensure optimal thread placement, asynchronous initialization requires prealloc context threads to be in use. Signed-off-by: Mark Kanda <mark.kanda@oracle.com> Signed-off-by: David Hildenbrand <david@redhat.com> ---
So, this LGTM. There might be ways to not rely on phases to achieve what we want to achieve (e.g., let the machine set an internal property on memory backends we create from the cmdline), but this should do as well.
I'll wait a bit for more feedback. If there is none, I'll route this through my tree (after doing a quick sanity test).
Thanks! -- Cheers, David / dhildenb
[Prev in Thread] | Current Thread | [Next in Thread] |