[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 1/3] pci: allow devices being tagged as not hotp
From: |
Michael S. Tsirkin |
Subject: |
[Qemu-devel] Re: [PATCH 1/3] pci: allow devices being tagged as not hotpluggable. |
Date: |
Thu, 6 Jan 2011 16:53:59 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, Nov 30, 2010 at 02:26:02PM +0100, Gerd Hoffmann wrote:
> This patch adds a field to PCIDeviceInfo to tag devices as being
> not hotpluggable. Any attempt to plug-in or -out such a device
> will throw an error.
>
> Signed-off-by: Gerd Hoffmann <address@hidden>
Acked-by: Michael S. Tsirkin <address@hidden>
> ---
> hw/pci.c | 10 ++++++++++
> hw/pci.h | 3 +++
> qerror.c | 4 ++++
> qerror.h | 3 +++
> 4 files changed, 20 insertions(+), 0 deletions(-)
>
> diff --git a/hw/pci.c b/hw/pci.c
> index 438c0d1..aee8d63 100644
> --- a/hw/pci.c
> +++ b/hw/pci.c
> @@ -1546,6 +1546,11 @@ static int pci_qdev_init(DeviceState *qdev, DeviceInfo
> *base)
> info->is_bridge);
> if (pci_dev == NULL)
> return -1;
> + if (qdev->hotplugged && info->no_hotplug) {
> + qerror_report(QERR_DEVICE_NO_HOTPLUG, info->qdev.name);
> + do_pci_unregister_device(pci_dev);
> + return -1;
> + }
> rc = info->init(pci_dev);
> if (rc != 0) {
> do_pci_unregister_device(pci_dev);
> @@ -1575,7 +1580,12 @@ static int pci_qdev_init(DeviceState *qdev, DeviceInfo
> *base)
> static int pci_unplug_device(DeviceState *qdev)
> {
> PCIDevice *dev = DO_UPCAST(PCIDevice, qdev, qdev);
> + PCIDeviceInfo *info = container_of(qdev->info, PCIDeviceInfo, qdev);
>
> + if (info->no_hotplug) {
> + qerror_report(QERR_DEVICE_NO_HOTPLUG, info->qdev.name);
> + return -1;
> + }
> return dev->bus->hotplug(dev->bus->hotplug_qdev, dev,
> PCI_HOTPLUG_DISABLED);
> }
> diff --git a/hw/pci.h b/hw/pci.h
> index 09b3e4c..1d1c515 100644
> --- a/hw/pci.h
> +++ b/hw/pci.h
> @@ -431,6 +431,9 @@ typedef struct {
> /* pcie stuff */
> int is_express; /* is this device pci express? */
>
> + /* device isn't hot-pluggable */
> + int no_hotplug;
> +
> /* rom bar */
> const char *romfile;
> } PCIDeviceInfo;
> diff --git a/qerror.c b/qerror.c
> index ac2cdaf..9d0cdeb 100644
> --- a/qerror.c
> +++ b/qerror.c
> @@ -101,6 +101,10 @@ static const QErrorStringTable qerror_table[] = {
> .desc = "Device '%(device)' has no child bus",
> },
> {
> + .error_fmt = QERR_DEVICE_NO_HOTPLUG,
> + .desc = "Device '%(device)' does not support hotplugging",
> + },
> + {
> .error_fmt = QERR_DUPLICATE_ID,
> .desc = "Duplicate ID '%(id)' for %(object)",
> },
> diff --git a/qerror.h b/qerror.h
> index 943a24b..b0f69da 100644
> --- a/qerror.h
> +++ b/qerror.h
> @@ -90,6 +90,9 @@ QError *qobject_to_qerror(const QObject *obj);
> #define QERR_DEVICE_NO_BUS \
> "{ 'class': 'DeviceNoBus', 'data': { 'device': %s } }"
>
> +#define QERR_DEVICE_NO_HOTPLUG \
> + "{ 'class': 'DeviceNoHotplug', 'data': { 'device': %s } }"
> +
> #define QERR_DUPLICATE_ID \
> "{ 'class': 'DuplicateId', 'data': { 'id': %s, 'object': %s } }"
>
> --
> 1.7.1
>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-devel] Re: [PATCH 1/3] pci: allow devices being tagged as not hotpluggable.,
Michael S. Tsirkin <=