[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 05/26] pc, pc_piix: split out pc nic initiali
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH v3 05/26] pc, pc_piix: split out pc nic initialization |
Date: |
Mon, 22 Oct 2012 15:27:56 +0200 |
On Fri, Oct 19, 2012 at 04:43:29PM -0400, Jason Baron wrote:
> From: Isaku Yamahata <address@hidden>
>
> Factor out pc nic initialization.
> This simplifies the pc initialization and will reduce the code
> duplication of q35 pc initialization.
>
> Reviewed-by: Paolo Bonzini <address@hidden>
> Signed-off-by: Isaku Yamahata <address@hidden>
> Signed-off-by: Jason Baron <address@hidden>
Anthony could you pls review/ack this patch?
> ---
> hw/pc.c | 15 +++++++++++++++
> hw/pc.h | 1 +
> hw/pc_piix.c | 9 +--------
> 3 files changed, 17 insertions(+), 8 deletions(-)
>
> diff --git a/hw/pc.c b/hw/pc.c
> index 6c0722d..2c60ff6 100644
> --- a/hw/pc.c
> +++ b/hw/pc.c
> @@ -1100,6 +1100,21 @@ void pc_basic_device_init(ISABus *isa_bus, qemu_irq
> *gsi,
> *floppy = fdctrl_init_isa(isa_bus, fd);
> }
>
> +void pc_nic_init(ISABus *isa_bus, PCIBus *pci_bus)
> +{
> + int i;
> +
> + for (i = 0; i < nb_nics; i++) {
> + NICInfo *nd = &nd_table[i];
> +
> + if (!pci_bus || (nd->model && strcmp(nd->model, "ne2k_isa") == 0)) {
> + pc_init_ne2k_isa(isa_bus, nd);
> + } else {
> + pci_nic_init_nofail(nd, "e1000", NULL);
> + }
> + }
> +}
> +
> void pc_pci_device_init(PCIBus *pci_bus)
> {
> int max_bus;
> diff --git a/hw/pc.h b/hw/pc.h
> index 9923d96..62d14e3 100644
> --- a/hw/pc.h
> +++ b/hw/pc.h
> @@ -125,6 +125,7 @@ void pc_cmos_init(ram_addr_t ram_size, ram_addr_t
> above_4g_mem_size,
> const char *boot_device,
> ISADevice *floppy, BusState *ide0, BusState *ide1,
> ISADevice *s);
> +void pc_nic_init(ISABus *isa_bus, PCIBus *pci_bus);
> void pc_pci_device_init(PCIBus *pci_bus);
>
> typedef void (*cpu_set_smm_t)(int smm, void *arg);
> diff --git a/hw/pc_piix.c b/hw/pc_piix.c
> index 82364ab..705211a 100644
> --- a/hw/pc_piix.c
> +++ b/hw/pc_piix.c
> @@ -233,14 +233,7 @@ static void pc_init1(MemoryRegion *system_memory,
> /* init basic PC hardware */
> pc_basic_device_init(isa_bus, gsi, &rtc_state, &floppy, xen_enabled());
>
> - for(i = 0; i < nb_nics; i++) {
> - NICInfo *nd = &nd_table[i];
> -
> - if (!pci_enabled || (nd->model && strcmp(nd->model, "ne2k_isa") ==
> 0))
> - pc_init_ne2k_isa(isa_bus, nd);
> - else
> - pci_nic_init_nofail(nd, "e1000", NULL);
> - }
> + pc_nic_init(isa_bus, pci_bus);
>
> ide_drive_get(hd, MAX_IDE_BUS);
> if (pci_enabled) {
> --
> 1.7.1
- [Qemu-devel] [PATCH v3 00/26] q35 qemu support, Jason Baron, 2012/10/19
- [Qemu-devel] [PATCH v3 04/26] pci: introduce pci_swizzle_map_irq_fn() for standardized interrupt pin swizzle, Jason Baron, 2012/10/19
- [Qemu-devel] [PATCH v3 05/26] pc, pc_piix: split out pc nic initialization, Jason Baron, 2012/10/19
- Re: [Qemu-devel] [PATCH v3 05/26] pc, pc_piix: split out pc nic initialization,
Michael S. Tsirkin <=
- [Qemu-devel] [PATCH v3 03/26] pci: pci capability must be in PCI space, Jason Baron, 2012/10/19
- [Qemu-devel] [PATCH v3 02/26] blockdev: Introduce IF_AHCI, Jason Baron, 2012/10/19
- Re: [Qemu-devel] [PATCH v3 02/26] blockdev: Introduce IF_AHCI, Michael S. Tsirkin, 2012/10/22
- Re: [Qemu-devel] [PATCH v3 02/26] blockdev: Introduce IF_AHCI, Kevin Wolf, 2012/10/22
- Re: [Qemu-devel] [PATCH v3 02/26] blockdev: Introduce IF_AHCI, Jason Baron, 2012/10/22
- Re: [Qemu-devel] [PATCH v3 02/26] blockdev: Introduce IF_AHCI, Markus Armbruster, 2012/10/24
- Re: [Qemu-devel] [PATCH v3 02/26] blockdev: Introduce IF_AHCI, Jason Baron, 2012/10/24
- Re: [Qemu-devel] [PATCH v3 02/26] blockdev: Introduce IF_AHCI, Markus Armbruster, 2012/10/26
[Qemu-devel] [PATCH v3 06/26] pc: Move ioapic_init() from pc_piix.c to pc.c, Jason Baron, 2012/10/19