Re: [PATCH 6/7] qemu: delayed MMIO support (VGA)

[Avi Kivity <avi@qumranet.com>]

Laurent Vivier wrote:
> This patch defines delayed MMIO zones for VGA cards.
>
> Signed-off-by: Laurent Vivier <Laurent.Vivier@bull.net>
> ---
>  qemu/hw/cirrus_vga.c |    2 ++
>  qemu/hw/vga.c        |    4 ++++
>  2 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/qemu/hw/cirrus_vga.c b/qemu/hw/cirrus_vga.c
> index 2c4aeec..9eb6929 100644
> --- a/qemu/hw/cirrus_vga.c
> +++ b/qemu/hw/cirrus_vga.c
> @@ -3291,6 +3291,8 @@ static void cirrus_init_common(CirrusVGAState * s, int device_id, int is_pci)
>                                             cirrus_vga_mem_write, s);
>      cpu_register_physical_memory(isa_mem_base + 0x000a0000, 0x20000,
>                                   vga_io_memory);
> +    if (kvm_enabled())
> +        qemu_kvm_register_delayed_mmio(isa_mem_base + 0x000a0000, 0x20000);
>  
>   

Cirrus has some mmio registers that cause actions (bitblt), which could 
be mapped at this address.  But I guess that since all the actions are 
synchronous anyway, no harm is done in delaying them.

-- 
error compiling committee.c: too many arguments to function