Re: [PATCH 2/2] target/i386: Use X86ASIdx_MEM in kvm_init()

["Philippe Mathieu-Daudé" <philmd@linaro.org>]

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().
> 
> Change the leftover in kvm_init() to make the usage consistent.
> 
> [0] https://lore.kernel.org/qemu-devel/20250730095253.1833411-3-xiaoyao.li@intel.com/
> 
> Signed-off-by: Xiaoyao Li <xiaoyao.li@intel.com>
> ---
>   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.