Re: [Qemu-devel] [PATCH] Memory: unify ioport registration

[Julien Grall]

On 04/03/2012 08:51 AM, Avi Kivity wrote:
> On 04/02/2012 05:37 PM, Julien Grall wrote:
>    
> >       if (!inited) {
> >           inited = 1;
> > @@ -2814,19 +2830,8 @@ static void cirrus_init_common(CirrusVGAState * s, int 
> > device_id, int is_pci,
> >               s->bustype = CIRRUS_BUSTYPE_ISA;
> >       }
> >
> > -    register_ioport_write(0x3c0, 16, 1, cirrus_vga_ioport_write, s);
> > -
> > -    register_ioport_write(0x3b4, 2, 1, cirrus_vga_ioport_write, s);
> > -    register_ioport_write(0x3d4, 2, 1, cirrus_vga_ioport_write, s);
> > -    register_ioport_write(0x3ba, 1, 1, cirrus_vga_ioport_write, s);
> > -    register_ioport_write(0x3da, 1, 1, cirrus_vga_ioport_write, s);
> > -
> > -    register_ioport_read(0x3c0, 16, 1, cirrus_vga_ioport_read, s);
> > -
> > -    register_ioport_read(0x3b4, 2, 1, cirrus_vga_ioport_read, s);
> > -    register_ioport_read(0x3d4, 2, 1, cirrus_vga_ioport_read, s);
> > -    register_ioport_read(0x3ba, 1, 1, cirrus_vga_ioport_read, s);
> > -    register_ioport_read(0x3da, 1, 1, cirrus_vga_ioport_read, s);
> > +    portio_list_init(cirrus_port_list, cirrus_portio_list, s, "cirrus-io");
> > +    portio_list_add(cirrus_port_list, get_system_io(), 0x3b0);
> >
> >      
> Please pass either pci_address_space_io() or isa_io_address_space()
> (you'll have to write the latter) to cirrus_init_common() from its
> callers.  We really want to avoid get_system_io().
>
>    
isa_io_address_space doesn't exist. I will create it, but which
memory region we need to return ? get_system_io () ?