On 10/16/25 10:43, Philippe Mathieu-Daudé wrote:
On 14/10/25 11:42, Xiaoyao Li wrote:
When the patch introduces 'enum X86ADIdx'[0] got merged, it somehow
missed the change to replace as id 0 with X86ASIdx_MEM in kvm_init().
It wasn't missed, it broke CI. ;)
Paolo
Change the leftover in kvm_init() to make the usage consistent.
[0] https://lore.kernel.org/qemu-devel/20250730095253.1833411-3-
[email protected]/
Signed-off-by: Xiaoyao Li <[email protected]>
---
accel/kvm/kvm-all.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/accel/kvm/kvm-all.c b/accel/kvm/kvm-all.c
index 58802f7c3cc1..3030c47d55b1 100644
--- a/accel/kvm/kvm-all.c
+++ b/accel/kvm/kvm-all.c
@@ -2800,7 +2800,7 @@ static int kvm_init(AccelState *as, MachineState
*ms)
s->memory_listener.listener.coalesced_io_del =
kvm_uncoalesce_mmio_region;
kvm_memory_listener_register(s, &s->memory_listener,
- &address_space_memory, 0, "kvm-
memory");
+ &address_space_memory, X86ASIdx_MEM,
"kvm-memory");
NAck: this is a generic code use by multiple architectures.