[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 5/6] xen: Add only xen-sysdev to dynamic sysb
From: |
Anthony PERARD |
Subject: |
Re: [Qemu-devel] [PATCH v2 5/6] xen: Add only xen-sysdev to dynamic sysbus device list |
Date: |
Fri, 5 Jan 2018 14:12:01 +0000 |
User-agent: |
Mutt/1.9.2 (2017-12-15) |
On Sat, Nov 25, 2017 at 01:16:09PM -0200, Eduardo Habkost wrote:
> There's no need to make the machine allow every possible sysbus
> device. We can now just add xen-sysdev to the allowed list.
>
> Cc: Stefano Stabellini <address@hidden>
> Cc: Anthony Perard <address@hidden>
> Cc: address@hidden
> Cc: Juergen Gross <address@hidden>
> Signed-off-by: Eduardo Habkost <address@hidden>
I've tested the patch series with every hotplug things I could think of,
and it worked fine.
Acked-by: Anthony PERARD <address@hidden>
> ---
> Changes series v1 -> v2:
> * New patch added to series
> ---
> hw/xen/xen_backend.c | 7 +------
> 1 file changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/hw/xen/xen_backend.c b/hw/xen/xen_backend.c
> index 82380ea9ee..7445b506ac 100644
> --- a/hw/xen/xen_backend.c
> +++ b/hw/xen/xen_backend.c
> @@ -564,12 +564,7 @@ static void xen_set_dynamic_sysbus(void)
> ObjectClass *oc = object_get_class(machine);
> MachineClass *mc = MACHINE_CLASS(oc);
>
> - /*
> - * Emulate old mc->has_dynamic_sysbus=true assignment
> - *
> - *TODO: add only Xen devices to the list
> - */
> - machine_class_allow_dynamic_sysbus_dev(mc, TYPE_SYS_BUS_DEVICE);
> + machine_class_allow_dynamic_sysbus_dev(mc, TYPE_XENSYSDEV);
> }
>
> int xen_be_register(const char *type, struct XenDevOps *ops)
--
Anthony PERARD
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v2 5/6] xen: Add only xen-sysdev to dynamic sysbus device list,
Anthony PERARD <=