[Qemu-devel] Re: [PATCH 1/6] qemu-kvm: Introduce bit-based phys_ram_dirty for VGA, CODE and MIGRATION.

[Avi Kivity]

On 03/16/2010 03:01 PM, Yoshiaki Tamura wrote:
> > > -uint8_t *phys_ram_dirty;
> > > +unsigned long *phys_ram_vga_dirty;
> > > +unsigned long *phys_ram_code_dirty;
> > > +unsigned long *phys_ram_migration_dirty;
> >
> > Would be nice to make this an array.
>
>
> Thanks for pointing out.
> I have a question regarding the index of the array.
> From the compatibility perspective, I would prefer using the existing 
> macros.
>
> #define VGA_DIRTY_FLAG       0x01
> #define CODE_DIRTY_FLAG      0x02
> #define MIGRATION_DIRTY_FLAG 0x08
>
> However, if I use them as is, I'll get a sparse array...
> Is it acceptable to change these values like 0, 1, 2?

Sure.

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