On 3/3/25 14:09, Igor Mammedov wrote:
On Wed, 26 Feb 2025 08:16:52 +0100
Philippe Mathieu-Daudé <phi...@linaro.org> wrote:
On 7/2/25 17:20, Igor Mammedov wrote:
cpu_list_add() was doing 2 distinct things:
- assign some index to vCPU
- add unrealized (thus in inconsistent state) vCPU to &cpus_queue
Code using CPU_FOREACH() macro would iterate over possibly
unrealized vCPUs, often dealt with special casing.
Instead of working around of vCPU existence in cpus_queue,
split out cpu_index assignment from cpu_list_add(),
Better split 2 distinct changes in 2 patches for clarity.
Will do it later, once folks decide how to fix broken TCG reset path.
do you mean:
#1 - introduce cpu_auto_assign_cpu_index()
#2 - move cpu_list_add() to later stage but keep cpu_auto_assign_cpu_index()
where it's now?
Exactly, thanks!
