[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH 27/34] hw/pci: declare pci_nic_init_nofail() in "h
|
From: |
David Gibson |
|
Subject: |
Re: [Qemu-arm] [PATCH 27/34] hw/pci: declare pci_nic_init_nofail() in "hw/net/pci.h" |
|
Date: |
Sat, 23 Sep 2017 15:12:27 +1000 |
|
User-agent: |
Mutt/1.9.0 (2017-09-02) |
On Fri, Sep 22, 2017 at 01:01:04PM -0300, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
ppc portions
Acked-by: David Gibson <address@hidden>
> ---
> hw/pci/pci_internal.h | 16 +++++++++++
> include/hw/net/pci.h | 20 +++++++++++++
> include/hw/pci/pci.h | 4 ---
> hw/arm/virt.c | 1 +
> hw/mips/mips_malta.c | 1 +
> hw/pci/pci.c | 67 ++------------------------------------------
> hw/pci/pci_nic.c | 77
> +++++++++++++++++++++++++++++++++++++++++++++++++++
> hw/ppc/e500.c | 1 +
> hw/ppc/spapr.c | 1 +
> hw/pci/Makefile.objs | 1 +
> 10 files changed, 120 insertions(+), 69 deletions(-)
> create mode 100644 hw/pci/pci_internal.h
> create mode 100644 include/hw/net/pci.h
> create mode 100644 hw/pci/pci_nic.c
>
> diff --git a/hw/pci/pci_internal.h b/hw/pci/pci_internal.h
> new file mode 100644
> index 0000000000..d967468767
> --- /dev/null
> +++ b/hw/pci/pci_internal.h
> @@ -0,0 +1,16 @@
> +/*
> + * QEMU PCI internal
> + *
> + * Copyright (c) 2005 Fabrice Bellard
> + *
> + * This work is licensed under the terms of the GNU GPL, version 2 or later.
> + * See the COPYING file in the top-level directory.
> + */
> +#ifndef QEMU_HW_PCI_INTERNAL_H
> +#define QEMU_HW_PCI_INTERNAL_H
> +
> +#include "hw/pci/pci_bus.h"
> +
> +PCIBus *pci_get_bus_devfn(int *devfnp, PCIBus *root, const char *devaddr);
> +
> +#endif
> diff --git a/include/hw/net/pci.h b/include/hw/net/pci.h
> new file mode 100644
> index 0000000000..529591b7f3
> --- /dev/null
> +++ b/include/hw/net/pci.h
> @@ -0,0 +1,20 @@
> +/*
> + * QEMU network devices
> + *
> + * Copyright (c) 2005 Fabrice Bellard
> + *
> + * This work is licensed under the terms of the GNU GPL, version 2 or later.
> + * See the COPYING file in the top-level directory.
> + */
> +#ifndef QEMU_HW_NET_PCI_H
> +#define QEMU_HW_NET_PCI_H
> +
> +#include "net/net.h"
> +#include "hw/pci/pci.h"
> +#include "hw/pci/pci_bus.h"
> +
> +PCIDevice *pci_nic_init_nofail(NICInfo *nd, PCIBus *rootbus,
> + const char *default_model,
> + const char *default_devaddr);
> +
> +#endif
> diff --git a/include/hw/pci/pci.h b/include/hw/pci/pci.h
> index aa7ef9cf69..6a0f7b5472 100644
> --- a/include/hw/pci/pci.h
> +++ b/include/hw/pci/pci.h
> @@ -422,10 +422,6 @@ void pci_device_set_intx_routing_notifier(PCIDevice *dev,
> PCIINTxRoutingNotifier notifier);
> void pci_device_reset(PCIDevice *dev);
>
> -PCIDevice *pci_nic_init_nofail(NICInfo *nd, PCIBus *rootbus,
> - const char *default_model,
> - const char *default_devaddr);
> -
> PCIDevice *pci_vga_init(PCIBus *bus);
>
> int pci_bus_num(PCIBus *s);
> diff --git a/hw/arm/virt.c b/hw/arm/virt.c
> index 9e18b410d7..39fab3acb9 100644
> --- a/hw/arm/virt.c
> +++ b/hw/arm/virt.c
> @@ -35,6 +35,7 @@
> #include "hw/arm/primecell.h"
> #include "hw/arm/virt.h"
> #include "hw/devices.h"
> +#include "hw/net/pci.h"
> #include "net/net.h"
> #include "sysemu/block-backend.h"
> #include "sysemu/device_tree.h"
> diff --git a/hw/mips/mips_malta.c b/hw/mips/mips_malta.c
> index 6945fa47c3..fb6a2f9363 100644
> --- a/hw/mips/mips_malta.c
> +++ b/hw/mips/mips_malta.c
> @@ -48,6 +48,7 @@
> #include "hw/timer/mc146818rtc.h"
> #include "hw/input/i8042.h"
> #include "hw/timer/i8254.h"
> +#include "hw/net/pci.h"
> #include "sysemu/blockdev.h"
> #include "exec/address-spaces.h"
> #include "hw/sysbus.h" /* SysBusDevice */
> diff --git a/hw/pci/pci.c b/hw/pci/pci.c
> index 1e6fb88eba..9b678c8fd0 100644
> --- a/hw/pci/pci.c
> +++ b/hw/pci/pci.c
> @@ -28,7 +28,6 @@
> #include "hw/pci/pci_bus.h"
> #include "hw/pci/pci_host.h"
> #include "monitor/monitor.h"
> -#include "net/net.h"
> #include "sysemu/sysemu.h"
> #include "hw/loader.h"
> #include "qemu/error-report.h"
> @@ -41,6 +40,7 @@
> #include "hw/hotplug.h"
> #include "hw/boards.h"
> #include "qemu/cutils.h"
> +#include "pci_internal.h"
>
> //#define DEBUG_PCI
> #ifdef DEBUG_PCI
> @@ -671,8 +671,7 @@ static int pci_parse_devaddr(const char *addr, int *domp,
> int *busp,
> return 0;
> }
>
> -static PCIBus *pci_get_bus_devfn(int *devfnp, PCIBus *root,
> - const char *devaddr)
> +PCIBus *pci_get_bus_devfn(int *devfnp, PCIBus *root, const char *devaddr)
> {
> int dom, bus;
> unsigned slot;
> @@ -1812,68 +1811,6 @@ PciInfoList *qmp_query_pci(Error **errp)
> return head;
> }
>
> -static const char * const pci_nic_models[] = {
> - "ne2k_pci",
> - "i82551",
> - "i82557b",
> - "i82559er",
> - "rtl8139",
> - "e1000",
> - "pcnet",
> - "virtio",
> - "sungem",
> - NULL
> -};
> -
> -static const char * const pci_nic_names[] = {
> - "ne2k_pci",
> - "i82551",
> - "i82557b",
> - "i82559er",
> - "rtl8139",
> - "e1000",
> - "pcnet",
> - "virtio-net-pci",
> - "sungem",
> - NULL
> -};
> -
> -/* Initialize a PCI NIC. */
> -PCIDevice *pci_nic_init_nofail(NICInfo *nd, PCIBus *rootbus,
> - const char *default_model,
> - const char *default_devaddr)
> -{
> - const char *devaddr = nd->devaddr ? nd->devaddr : default_devaddr;
> - PCIBus *bus;
> - PCIDevice *pci_dev;
> - DeviceState *dev;
> - int devfn;
> - int i;
> -
> - if (qemu_show_nic_models(nd->model, pci_nic_models)) {
> - exit(0);
> - }
> -
> - i = qemu_find_nic_model(nd, pci_nic_models, default_model);
> - if (i < 0) {
> - exit(1);
> - }
> -
> - bus = pci_get_bus_devfn(&devfn, rootbus, devaddr);
> - if (!bus) {
> - error_report("Invalid PCI device address %s for device %s",
> - devaddr, pci_nic_names[i]);
> - exit(1);
> - }
> -
> - pci_dev = pci_create(bus, devfn, pci_nic_names[i]);
> - dev = &pci_dev->qdev;
> - qdev_set_nic_properties(dev, nd);
> - qdev_init_nofail(dev);
> -
> - return pci_dev;
> -}
> -
> PCIDevice *pci_vga_init(PCIBus *bus)
> {
> switch (vga_interface_type) {
> diff --git a/hw/pci/pci_nic.c b/hw/pci/pci_nic.c
> new file mode 100644
> index 0000000000..fb1a10ff12
> --- /dev/null
> +++ b/hw/pci/pci_nic.c
> @@ -0,0 +1,77 @@
> +/*
> + * QEMU PCI network interface
> + *
> + * Copyright (c) 2004 Fabrice Bellard
> + *
> + * This work is licensed under the terms of the GNU GPL, version 2 or later.
> + * See the COPYING file in the top-level directory.
> + */
> +#include "qemu/osdep.h"
> +#include "qemu/error-report.h"
> +#include "hw/pci/pci.h"
> +#include "hw/pci/pci_bus.h"
> +#include "hw/net/pci.h"
> +#include "net/net.h"
> +#include "pci_internal.h"
> +
> +static const char *const pci_nic_models[] = {
> + "ne2k_pci",
> + "i82551",
> + "i82557b",
> + "i82559er",
> + "rtl8139",
> + "e1000",
> + "pcnet",
> + "virtio",
> + "sungem",
> + NULL
> +};
> +
> +static const char *const pci_nic_names[] = {
> + "ne2k_pci",
> + "i82551",
> + "i82557b",
> + "i82559er",
> + "rtl8139",
> + "e1000",
> + "pcnet",
> + "virtio-net-pci",
> + "sungem",
> + NULL
> +};
> +
> +/* Initialize a PCI NIC. */
> +PCIDevice *pci_nic_init_nofail(NICInfo *nd, PCIBus *rootbus,
> + const char *default_model,
> + const char *default_devaddr)
> +{
> + const char *devaddr = nd->devaddr ? nd->devaddr : default_devaddr;
> + PCIBus *bus;
> + PCIDevice *pci_dev;
> + DeviceState *dev;
> + int devfn;
> + int i;
> +
> + if (qemu_show_nic_models(nd->model, pci_nic_models)) {
> + exit(0);
> + }
> +
> + i = qemu_find_nic_model(nd, pci_nic_models, default_model);
> + if (i < 0) {
> + exit(1);
> + }
> +
> + bus = pci_get_bus_devfn(&devfn, rootbus, devaddr);
> + if (!bus) {
> + error_report("Invalid PCI device address %s for device %s",
> + devaddr, pci_nic_names[i]);
> + exit(1);
> + }
> +
> + pci_dev = pci_create(bus, devfn, pci_nic_names[i]);
> + dev = &pci_dev->qdev;
> + qdev_set_nic_properties(dev, nd);
> + qdev_init_nofail(dev);
> +
> + return pci_dev;
> +}
> diff --git a/hw/ppc/e500.c b/hw/ppc/e500.c
> index db0e49ab8f..482757ca7b 100644
> --- a/hw/ppc/e500.c
> +++ b/hw/ppc/e500.c
> @@ -24,6 +24,7 @@
> #include "hw/hw.h"
> #include "hw/char/serial.h"
> #include "hw/pci/pci.h"
> +#include "hw/net/pci.h"
> #include "hw/boards.h"
> #include "sysemu/sysemu.h"
> #include "sysemu/kvm.h"
> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> index 17ea77618c..96007051e2 100644
> --- a/hw/ppc/spapr.c
> +++ b/hw/ppc/spapr.c
> @@ -56,6 +56,7 @@
> #include "hw/pci-host/spapr.h"
> #include "hw/ppc/xics.h"
> #include "hw/pci/msi.h"
> +#include "hw/net/pci.h"
>
> #include "hw/pci/pci.h"
> #include "hw/scsi/scsi.h"
> diff --git a/hw/pci/Makefile.objs b/hw/pci/Makefile.objs
> index 9f905e6344..125428fd54 100644
> --- a/hw/pci/Makefile.objs
> +++ b/hw/pci/Makefile.objs
> @@ -1,4 +1,5 @@
> common-obj-$(CONFIG_PCI) += pci.o pci_bridge.o
> +common-obj-$(CONFIG_PCI) += pci_nic.o
> common-obj-$(CONFIG_PCI) += msix.o msi.o
> common-obj-$(CONFIG_PCI) += shpc.o
> common-obj-$(CONFIG_PCI) += slotid_cap.o
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature