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.




Reply via email to