* [Qemu-devel] [PATCH 0/2] spapr/xive: rework the mapping the KVM memory regions
@ 2019-06-14 16:59 Cédric Le Goater
2019-06-14 16:59 ` [Qemu-devel] [PATCH 1/2] " Cédric Le Goater
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Cédric Le Goater @ 2019-06-14 16:59 UTC (permalink / raw)
To: David Gibson; +Cc: Cédric Le Goater, qemu-ppc, Greg Kurz, qemu-devel
Hello,
Here is a small series simplifying the initialization sequence of the
interrupt device by using memory regions specific for KVM. These are
mapped as overlaps on top of the emulated device.
Thanks,
C.
Cédric Le Goater (2):
spapr/xive: rework the mapping the KVM memory regions
spapr/xive: simplify spapr_irq_init_device() to remove the emulated
init
include/hw/ppc/spapr_irq.h | 1 -
include/hw/ppc/spapr_xive.h | 2 +-
include/hw/ppc/xive.h | 1 +
hw/intc/spapr_xive.c | 38 ++++++++++---------------------------
hw/intc/spapr_xive_kvm.c | 21 +++++++++++---------
hw/ppc/spapr_irq.c | 21 +++-----------------
6 files changed, 27 insertions(+), 57 deletions(-)
--
2.21.0
^ permalink raw reply [flat|nested] 6+ messages in thread* [Qemu-devel] [PATCH 1/2] spapr/xive: rework the mapping the KVM memory regions 2019-06-14 16:59 [Qemu-devel] [PATCH 0/2] spapr/xive: rework the mapping the KVM memory regions Cédric Le Goater @ 2019-06-14 16:59 ` Cédric Le Goater 2019-06-14 17:41 ` Greg Kurz 2019-06-14 16:59 ` [Qemu-devel] [PATCH 2/2] spapr/xive: simplify spapr_irq_init_device() to remove the emulated init Cédric Le Goater 2019-07-01 5:15 ` [Qemu-devel] [PATCH 0/2] spapr/xive: rework the mapping the KVM memory regions David Gibson 2 siblings, 1 reply; 6+ messages in thread From: Cédric Le Goater @ 2019-06-14 16:59 UTC (permalink / raw) To: David Gibson; +Cc: Cédric Le Goater, qemu-ppc, Greg Kurz, qemu-devel Today, the interrupt device is fully initialized at reset when the CAS negotiation process has completed. Depending on the KVM capabilities, the SpaprXive memory regions (ESB, TIMA) are initialized with a host MMIO backend or a QEMU emulated backend. This results in a complex initialization sequence partially done at realize and later at reset, and some memory region leaks. To simplify this sequence and to remove of the late initialization of the emulated device which is required to be done only once, we introduce new memory regions specific for KVM. These regions are mapped as overlaps on top of the emulated device to make use of the host MMIOs. Also provide proper cleanups of these regions when the XIVE KVM device is destroyed to fix the leaks. Signed-off-by: Cédric Le Goater <clg@kaod.org> --- include/hw/ppc/spapr_xive.h | 2 +- include/hw/ppc/xive.h | 1 + hw/intc/spapr_xive.c | 38 ++++++++++--------------------------- hw/intc/spapr_xive_kvm.c | 21 +++++++++++--------- hw/ppc/spapr_irq.c | 1 - 5 files changed, 24 insertions(+), 39 deletions(-) diff --git a/include/hw/ppc/spapr_xive.h b/include/hw/ppc/spapr_xive.h index b26befcf6b56..719714426524 100644 --- a/include/hw/ppc/spapr_xive.h +++ b/include/hw/ppc/spapr_xive.h @@ -42,6 +42,7 @@ typedef struct SpaprXive { /* KVM support */ int fd; void *tm_mmap; + MemoryRegion tm_mmio_kvm; VMChangeStateEntry *change; } SpaprXive; @@ -66,7 +67,6 @@ void spapr_xive_map_mmio(SpaprXive *xive); int spapr_xive_end_to_target(uint8_t end_blk, uint32_t end_idx, uint32_t *out_server, uint8_t *out_prio); -void spapr_xive_init(SpaprXive *xive, Error **errp); /* * KVM XIVE device helpers diff --git a/include/hw/ppc/xive.h b/include/hw/ppc/xive.h index a6ee7e831d8b..55c53c741776 100644 --- a/include/hw/ppc/xive.h +++ b/include/hw/ppc/xive.h @@ -197,6 +197,7 @@ typedef struct XiveSource { /* KVM support */ void *esb_mmap; + MemoryRegion esb_mmio_kvm; XiveNotifier *xive; } XiveSource; diff --git a/hw/intc/spapr_xive.c b/hw/intc/spapr_xive.c index 58c2e5d890bd..3ae311d9ff7f 100644 --- a/hw/intc/spapr_xive.c +++ b/hw/intc/spapr_xive.c @@ -194,13 +194,6 @@ void spapr_xive_pic_print_info(SpaprXive *xive, Monitor *mon) } } -void spapr_xive_map_mmio(SpaprXive *xive) -{ - sysbus_mmio_map(SYS_BUS_DEVICE(xive), 0, xive->vc_base); - sysbus_mmio_map(SYS_BUS_DEVICE(xive), 1, xive->end_base); - sysbus_mmio_map(SYS_BUS_DEVICE(xive), 2, xive->tm_base); -} - void spapr_xive_mmio_set_enabled(SpaprXive *xive, bool enable) { memory_region_set_enabled(&xive->source.esb_mmio, enable); @@ -305,6 +298,7 @@ static void spapr_xive_realize(DeviceState *dev, Error **errp) error_propagate(errp, local_err); return; } + sysbus_init_mmio(SYS_BUS_DEVICE(xive), &xsrc->esb_mmio); /* * Initialize the END ESB source @@ -318,6 +312,7 @@ static void spapr_xive_realize(DeviceState *dev, Error **errp) error_propagate(errp, local_err); return; } + sysbus_init_mmio(SYS_BUS_DEVICE(xive), &end_xsrc->esb_mmio); /* Set the mapping address of the END ESB pages after the source ESBs */ xive->end_base = xive->vc_base + (1ull << xsrc->esb_shift) * xsrc->nr_irqs; @@ -333,31 +328,18 @@ static void spapr_xive_realize(DeviceState *dev, Error **errp) qemu_register_reset(spapr_xive_reset, dev); - /* Define all XIVE MMIO regions on SysBus */ - sysbus_init_mmio(SYS_BUS_DEVICE(xive), &xsrc->esb_mmio); - sysbus_init_mmio(SYS_BUS_DEVICE(xive), &end_xsrc->esb_mmio); - sysbus_init_mmio(SYS_BUS_DEVICE(xive), &xive->tm_mmio); -} - -void spapr_xive_init(SpaprXive *xive, Error **errp) -{ - XiveSource *xsrc = &xive->source; - - /* - * The emulated XIVE device can only be initialized once. If the - * ESB memory region has been already mapped, it means we have been - * through there. - */ - if (memory_region_is_mapped(&xsrc->esb_mmio)) { - return; - } - /* TIMA initialization */ memory_region_init_io(&xive->tm_mmio, OBJECT(xive), &xive_tm_ops, xive, "xive.tima", 4ull << TM_SHIFT); + sysbus_init_mmio(SYS_BUS_DEVICE(xive), &xive->tm_mmio); - /* Map all regions */ - spapr_xive_map_mmio(xive); + /* + * Map all regions. These will be enabled or disabled at reset and + * can also be overridden by KVM memory regions if active + */ + sysbus_mmio_map(SYS_BUS_DEVICE(xive), 0, xive->vc_base); + sysbus_mmio_map(SYS_BUS_DEVICE(xive), 1, xive->end_base); + sysbus_mmio_map(SYS_BUS_DEVICE(xive), 2, xive->tm_base); } static int spapr_xive_get_eas(XiveRouter *xrtr, uint8_t eas_blk, diff --git a/hw/intc/spapr_xive_kvm.c b/hw/intc/spapr_xive_kvm.c index b48f135838f2..5559f8bce5ef 100644 --- a/hw/intc/spapr_xive_kvm.c +++ b/hw/intc/spapr_xive_kvm.c @@ -728,8 +728,10 @@ void kvmppc_xive_connect(SpaprXive *xive, Error **errp) return; } - memory_region_init_ram_device_ptr(&xsrc->esb_mmio, OBJECT(xsrc), + memory_region_init_ram_device_ptr(&xsrc->esb_mmio_kvm, OBJECT(xsrc), "xive.esb", esb_len, xsrc->esb_mmap); + memory_region_add_subregion_overlap(&xsrc->esb_mmio, 0, + &xsrc->esb_mmio_kvm, 1); /* * 2. END ESB pages (No KVM support yet) @@ -744,8 +746,10 @@ void kvmppc_xive_connect(SpaprXive *xive, Error **errp) error_propagate(errp, local_err); return; } - memory_region_init_ram_device_ptr(&xive->tm_mmio, OBJECT(xive), + memory_region_init_ram_device_ptr(&xive->tm_mmio_kvm, OBJECT(xive), "xive.tima", tima_len, xive->tm_mmap); + memory_region_add_subregion_overlap(&xive->tm_mmio, 0, + &xive->tm_mmio_kvm, 1); xive->change = qemu_add_vm_change_state_handler( kvmppc_xive_change_state_handler, xive); @@ -771,9 +775,6 @@ void kvmppc_xive_connect(SpaprXive *xive, Error **errp) kvm_kernel_irqchip = true; kvm_msi_via_irqfd_allowed = true; kvm_gsi_direct_mapping = true; - - /* Map all regions */ - spapr_xive_map_mmio(xive); } void kvmppc_xive_disconnect(SpaprXive *xive, Error **errp) @@ -795,13 +796,15 @@ void kvmppc_xive_disconnect(SpaprXive *xive, Error **errp) xsrc = &xive->source; esb_len = (1ull << xsrc->esb_shift) * xsrc->nr_irqs; - sysbus_mmio_unmap(SYS_BUS_DEVICE(xive), 0); + memory_region_del_subregion(&xsrc->esb_mmio, &xsrc->esb_mmio_kvm); + object_unparent(OBJECT(&xsrc->esb_mmio_kvm)); munmap(xsrc->esb_mmap, esb_len); + xsrc->esb_mmap = NULL; - sysbus_mmio_unmap(SYS_BUS_DEVICE(xive), 1); - - sysbus_mmio_unmap(SYS_BUS_DEVICE(xive), 2); + memory_region_del_subregion(&xive->tm_mmio, &xive->tm_mmio_kvm); + object_unparent(OBJECT(&xive->tm_mmio_kvm)); munmap(xive->tm_mmap, 4ull << TM_SHIFT); + xive->tm_mmap = NULL; /* * When the KVM device fd is closed, the KVM device is destroyed diff --git a/hw/ppc/spapr_irq.c b/hw/ppc/spapr_irq.c index 75654fc67aba..73e6f10da165 100644 --- a/hw/ppc/spapr_irq.c +++ b/hw/ppc/spapr_irq.c @@ -413,7 +413,6 @@ static const char *spapr_irq_get_nodename_xive(SpaprMachineState *spapr) static void spapr_irq_init_emu_xive(SpaprMachineState *spapr, Error **errp) { - spapr_xive_init(spapr->xive, errp); } static void spapr_irq_init_kvm_xive(SpaprMachineState *spapr, Error **errp) -- 2.21.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH 1/2] spapr/xive: rework the mapping the KVM memory regions 2019-06-14 16:59 ` [Qemu-devel] [PATCH 1/2] " Cédric Le Goater @ 2019-06-14 17:41 ` Greg Kurz 0 siblings, 0 replies; 6+ messages in thread From: Greg Kurz @ 2019-06-14 17:41 UTC (permalink / raw) To: Cédric Le Goater; +Cc: qemu-ppc, qemu-devel, David Gibson On Fri, 14 Jun 2019 18:59:19 +0200 Cédric Le Goater <clg@kaod.org> wrote: > Today, the interrupt device is fully initialized at reset when the CAS > negotiation process has completed. Depending on the KVM capabilities, > the SpaprXive memory regions (ESB, TIMA) are initialized with a host > MMIO backend or a QEMU emulated backend. This results in a complex > initialization sequence partially done at realize and later at reset, > and some memory region leaks. > > To simplify this sequence and to remove of the late initialization of > the emulated device which is required to be done only once, we > introduce new memory regions specific for KVM. These regions are > mapped as overlaps on top of the emulated device to make use of the > host MMIOs. Also provide proper cleanups of these regions when the > XIVE KVM device is destroyed to fix the leaks. > > Signed-off-by: Cédric Le Goater <clg@kaod.org> > --- Nice ! Reviewed-by: Greg Kurz <groug@kaod.org> > include/hw/ppc/spapr_xive.h | 2 +- > include/hw/ppc/xive.h | 1 + > hw/intc/spapr_xive.c | 38 ++++++++++--------------------------- > hw/intc/spapr_xive_kvm.c | 21 +++++++++++--------- > hw/ppc/spapr_irq.c | 1 - > 5 files changed, 24 insertions(+), 39 deletions(-) > > diff --git a/include/hw/ppc/spapr_xive.h b/include/hw/ppc/spapr_xive.h > index b26befcf6b56..719714426524 100644 > --- a/include/hw/ppc/spapr_xive.h > +++ b/include/hw/ppc/spapr_xive.h > @@ -42,6 +42,7 @@ typedef struct SpaprXive { > /* KVM support */ > int fd; > void *tm_mmap; > + MemoryRegion tm_mmio_kvm; > VMChangeStateEntry *change; > } SpaprXive; > > @@ -66,7 +67,6 @@ void spapr_xive_map_mmio(SpaprXive *xive); > > int spapr_xive_end_to_target(uint8_t end_blk, uint32_t end_idx, > uint32_t *out_server, uint8_t *out_prio); > -void spapr_xive_init(SpaprXive *xive, Error **errp); > > /* > * KVM XIVE device helpers > diff --git a/include/hw/ppc/xive.h b/include/hw/ppc/xive.h > index a6ee7e831d8b..55c53c741776 100644 > --- a/include/hw/ppc/xive.h > +++ b/include/hw/ppc/xive.h > @@ -197,6 +197,7 @@ typedef struct XiveSource { > > /* KVM support */ > void *esb_mmap; > + MemoryRegion esb_mmio_kvm; > > XiveNotifier *xive; > } XiveSource; > diff --git a/hw/intc/spapr_xive.c b/hw/intc/spapr_xive.c > index 58c2e5d890bd..3ae311d9ff7f 100644 > --- a/hw/intc/spapr_xive.c > +++ b/hw/intc/spapr_xive.c > @@ -194,13 +194,6 @@ void spapr_xive_pic_print_info(SpaprXive *xive, Monitor *mon) > } > } > > -void spapr_xive_map_mmio(SpaprXive *xive) > -{ > - sysbus_mmio_map(SYS_BUS_DEVICE(xive), 0, xive->vc_base); > - sysbus_mmio_map(SYS_BUS_DEVICE(xive), 1, xive->end_base); > - sysbus_mmio_map(SYS_BUS_DEVICE(xive), 2, xive->tm_base); > -} > - > void spapr_xive_mmio_set_enabled(SpaprXive *xive, bool enable) > { > memory_region_set_enabled(&xive->source.esb_mmio, enable); > @@ -305,6 +298,7 @@ static void spapr_xive_realize(DeviceState *dev, Error **errp) > error_propagate(errp, local_err); > return; > } > + sysbus_init_mmio(SYS_BUS_DEVICE(xive), &xsrc->esb_mmio); > > /* > * Initialize the END ESB source > @@ -318,6 +312,7 @@ static void spapr_xive_realize(DeviceState *dev, Error **errp) > error_propagate(errp, local_err); > return; > } > + sysbus_init_mmio(SYS_BUS_DEVICE(xive), &end_xsrc->esb_mmio); > > /* Set the mapping address of the END ESB pages after the source ESBs */ > xive->end_base = xive->vc_base + (1ull << xsrc->esb_shift) * xsrc->nr_irqs; > @@ -333,31 +328,18 @@ static void spapr_xive_realize(DeviceState *dev, Error **errp) > > qemu_register_reset(spapr_xive_reset, dev); > > - /* Define all XIVE MMIO regions on SysBus */ > - sysbus_init_mmio(SYS_BUS_DEVICE(xive), &xsrc->esb_mmio); > - sysbus_init_mmio(SYS_BUS_DEVICE(xive), &end_xsrc->esb_mmio); > - sysbus_init_mmio(SYS_BUS_DEVICE(xive), &xive->tm_mmio); > -} > - > -void spapr_xive_init(SpaprXive *xive, Error **errp) > -{ > - XiveSource *xsrc = &xive->source; > - > - /* > - * The emulated XIVE device can only be initialized once. If the > - * ESB memory region has been already mapped, it means we have been > - * through there. > - */ > - if (memory_region_is_mapped(&xsrc->esb_mmio)) { > - return; > - } > - > /* TIMA initialization */ > memory_region_init_io(&xive->tm_mmio, OBJECT(xive), &xive_tm_ops, xive, > "xive.tima", 4ull << TM_SHIFT); > + sysbus_init_mmio(SYS_BUS_DEVICE(xive), &xive->tm_mmio); > > - /* Map all regions */ > - spapr_xive_map_mmio(xive); > + /* > + * Map all regions. These will be enabled or disabled at reset and > + * can also be overridden by KVM memory regions if active > + */ > + sysbus_mmio_map(SYS_BUS_DEVICE(xive), 0, xive->vc_base); > + sysbus_mmio_map(SYS_BUS_DEVICE(xive), 1, xive->end_base); > + sysbus_mmio_map(SYS_BUS_DEVICE(xive), 2, xive->tm_base); > } > > static int spapr_xive_get_eas(XiveRouter *xrtr, uint8_t eas_blk, > diff --git a/hw/intc/spapr_xive_kvm.c b/hw/intc/spapr_xive_kvm.c > index b48f135838f2..5559f8bce5ef 100644 > --- a/hw/intc/spapr_xive_kvm.c > +++ b/hw/intc/spapr_xive_kvm.c > @@ -728,8 +728,10 @@ void kvmppc_xive_connect(SpaprXive *xive, Error **errp) > return; > } > > - memory_region_init_ram_device_ptr(&xsrc->esb_mmio, OBJECT(xsrc), > + memory_region_init_ram_device_ptr(&xsrc->esb_mmio_kvm, OBJECT(xsrc), > "xive.esb", esb_len, xsrc->esb_mmap); > + memory_region_add_subregion_overlap(&xsrc->esb_mmio, 0, > + &xsrc->esb_mmio_kvm, 1); > > /* > * 2. END ESB pages (No KVM support yet) > @@ -744,8 +746,10 @@ void kvmppc_xive_connect(SpaprXive *xive, Error **errp) > error_propagate(errp, local_err); > return; > } > - memory_region_init_ram_device_ptr(&xive->tm_mmio, OBJECT(xive), > + memory_region_init_ram_device_ptr(&xive->tm_mmio_kvm, OBJECT(xive), > "xive.tima", tima_len, xive->tm_mmap); > + memory_region_add_subregion_overlap(&xive->tm_mmio, 0, > + &xive->tm_mmio_kvm, 1); > > xive->change = qemu_add_vm_change_state_handler( > kvmppc_xive_change_state_handler, xive); > @@ -771,9 +775,6 @@ void kvmppc_xive_connect(SpaprXive *xive, Error **errp) > kvm_kernel_irqchip = true; > kvm_msi_via_irqfd_allowed = true; > kvm_gsi_direct_mapping = true; > - > - /* Map all regions */ > - spapr_xive_map_mmio(xive); > } > > void kvmppc_xive_disconnect(SpaprXive *xive, Error **errp) > @@ -795,13 +796,15 @@ void kvmppc_xive_disconnect(SpaprXive *xive, Error **errp) > xsrc = &xive->source; > esb_len = (1ull << xsrc->esb_shift) * xsrc->nr_irqs; > > - sysbus_mmio_unmap(SYS_BUS_DEVICE(xive), 0); > + memory_region_del_subregion(&xsrc->esb_mmio, &xsrc->esb_mmio_kvm); > + object_unparent(OBJECT(&xsrc->esb_mmio_kvm)); > munmap(xsrc->esb_mmap, esb_len); > + xsrc->esb_mmap = NULL; > > - sysbus_mmio_unmap(SYS_BUS_DEVICE(xive), 1); > - > - sysbus_mmio_unmap(SYS_BUS_DEVICE(xive), 2); > + memory_region_del_subregion(&xive->tm_mmio, &xive->tm_mmio_kvm); > + object_unparent(OBJECT(&xive->tm_mmio_kvm)); > munmap(xive->tm_mmap, 4ull << TM_SHIFT); > + xive->tm_mmap = NULL; > > /* > * When the KVM device fd is closed, the KVM device is destroyed > diff --git a/hw/ppc/spapr_irq.c b/hw/ppc/spapr_irq.c > index 75654fc67aba..73e6f10da165 100644 > --- a/hw/ppc/spapr_irq.c > +++ b/hw/ppc/spapr_irq.c > @@ -413,7 +413,6 @@ static const char *spapr_irq_get_nodename_xive(SpaprMachineState *spapr) > > static void spapr_irq_init_emu_xive(SpaprMachineState *spapr, Error **errp) > { > - spapr_xive_init(spapr->xive, errp); > } > > static void spapr_irq_init_kvm_xive(SpaprMachineState *spapr, Error **errp) ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Qemu-devel] [PATCH 2/2] spapr/xive: simplify spapr_irq_init_device() to remove the emulated init 2019-06-14 16:59 [Qemu-devel] [PATCH 0/2] spapr/xive: rework the mapping the KVM memory regions Cédric Le Goater 2019-06-14 16:59 ` [Qemu-devel] [PATCH 1/2] " Cédric Le Goater @ 2019-06-14 16:59 ` Cédric Le Goater 2019-06-14 17:41 ` Greg Kurz 2019-07-01 5:15 ` [Qemu-devel] [PATCH 0/2] spapr/xive: rework the mapping the KVM memory regions David Gibson 2 siblings, 1 reply; 6+ messages in thread From: Cédric Le Goater @ 2019-06-14 16:59 UTC (permalink / raw) To: David Gibson; +Cc: Cédric Le Goater, qemu-ppc, Greg Kurz, qemu-devel The init_emu() handles are now empty. Remove them and rename spapr_irq_init_device() to spapr_irq_init_kvm(). Signed-off-by: Cédric Le Goater <clg@kaod.org> --- include/hw/ppc/spapr_irq.h | 1 - hw/ppc/spapr_irq.c | 20 +++----------------- 2 files changed, 3 insertions(+), 18 deletions(-) diff --git a/include/hw/ppc/spapr_irq.h b/include/hw/ppc/spapr_irq.h index 14cab73c9c07..f965a58f8954 100644 --- a/include/hw/ppc/spapr_irq.h +++ b/include/hw/ppc/spapr_irq.h @@ -48,7 +48,6 @@ typedef struct SpaprIrq { void (*reset)(SpaprMachineState *spapr, Error **errp); void (*set_irq)(void *opaque, int srcno, int val); const char *(*get_nodename)(SpaprMachineState *spapr); - void (*init_emu)(SpaprMachineState *spapr, Error **errp); void (*init_kvm)(SpaprMachineState *spapr, Error **errp); } SpaprIrq; diff --git a/hw/ppc/spapr_irq.c b/hw/ppc/spapr_irq.c index 73e6f10da165..84b9b32fe40f 100644 --- a/hw/ppc/spapr_irq.c +++ b/hw/ppc/spapr_irq.c @@ -62,7 +62,7 @@ void spapr_irq_msi_reset(SpaprMachineState *spapr) bitmap_clear(spapr->irq_map, 0, spapr->irq_map_nr); } -static void spapr_irq_init_device(SpaprMachineState *spapr, +static void spapr_irq_init_kvm(SpaprMachineState *spapr, SpaprIrq *irq, Error **errp) { MachineState *machine = MACHINE(spapr); @@ -88,8 +88,6 @@ static void spapr_irq_init_device(SpaprMachineState *spapr, error_prepend(&local_err, "kernel_irqchip allowed but unavailable: "); warn_report_err(local_err); } - - irq->init_emu(spapr, errp); } /* @@ -224,7 +222,7 @@ static void spapr_irq_reset_xics(SpaprMachineState *spapr, Error **errp) { Error *local_err = NULL; - spapr_irq_init_device(spapr, &spapr_irq_xics, &local_err); + spapr_irq_init_kvm(spapr, &spapr_irq_xics, &local_err); if (local_err) { error_propagate(errp, local_err); return; @@ -236,10 +234,6 @@ static const char *spapr_irq_get_nodename_xics(SpaprMachineState *spapr) return XICS_NODENAME; } -static void spapr_irq_init_emu_xics(SpaprMachineState *spapr, Error **errp) -{ -} - static void spapr_irq_init_kvm_xics(SpaprMachineState *spapr, Error **errp) { if (kvm_enabled()) { @@ -267,7 +261,6 @@ SpaprIrq spapr_irq_xics = { .reset = spapr_irq_reset_xics, .set_irq = spapr_irq_set_irq_xics, .get_nodename = spapr_irq_get_nodename_xics, - .init_emu = spapr_irq_init_emu_xics, .init_kvm = spapr_irq_init_kvm_xics, }; @@ -385,7 +378,7 @@ static void spapr_irq_reset_xive(SpaprMachineState *spapr, Error **errp) spapr_xive_set_tctx_os_cam(spapr_cpu_state(cpu)->tctx); } - spapr_irq_init_device(spapr, &spapr_irq_xive, &local_err); + spapr_irq_init_kvm(spapr, &spapr_irq_xive, &local_err); if (local_err) { error_propagate(errp, local_err); return; @@ -411,10 +404,6 @@ static const char *spapr_irq_get_nodename_xive(SpaprMachineState *spapr) return spapr->xive->nodename; } -static void spapr_irq_init_emu_xive(SpaprMachineState *spapr, Error **errp) -{ -} - static void spapr_irq_init_kvm_xive(SpaprMachineState *spapr, Error **errp) { if (kvm_enabled()) { @@ -446,7 +435,6 @@ SpaprIrq spapr_irq_xive = { .reset = spapr_irq_reset_xive, .set_irq = spapr_irq_set_irq_xive, .get_nodename = spapr_irq_get_nodename_xive, - .init_emu = spapr_irq_init_emu_xive, .init_kvm = spapr_irq_init_kvm_xive, }; @@ -624,7 +612,6 @@ SpaprIrq spapr_irq_dual = { .reset = spapr_irq_reset_dual, .set_irq = spapr_irq_set_irq_dual, .get_nodename = spapr_irq_get_nodename_dual, - .init_emu = NULL, /* should not be used */ .init_kvm = NULL, /* should not be used */ }; @@ -840,6 +827,5 @@ SpaprIrq spapr_irq_xics_legacy = { .reset = spapr_irq_reset_xics, .set_irq = spapr_irq_set_irq_xics, .get_nodename = spapr_irq_get_nodename_xics, - .init_emu = spapr_irq_init_emu_xics, .init_kvm = spapr_irq_init_kvm_xics, }; -- 2.21.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH 2/2] spapr/xive: simplify spapr_irq_init_device() to remove the emulated init 2019-06-14 16:59 ` [Qemu-devel] [PATCH 2/2] spapr/xive: simplify spapr_irq_init_device() to remove the emulated init Cédric Le Goater @ 2019-06-14 17:41 ` Greg Kurz 0 siblings, 0 replies; 6+ messages in thread From: Greg Kurz @ 2019-06-14 17:41 UTC (permalink / raw) To: Cédric Le Goater; +Cc: qemu-ppc, qemu-devel, David Gibson On Fri, 14 Jun 2019 18:59:20 +0200 Cédric Le Goater <clg@kaod.org> wrote: > The init_emu() handles are now empty. Remove them and rename > spapr_irq_init_device() to spapr_irq_init_kvm(). > > Signed-off-by: Cédric Le Goater <clg@kaod.org> > --- Reviewed-by: Greg Kurz <groug@kaod.org> > include/hw/ppc/spapr_irq.h | 1 - > hw/ppc/spapr_irq.c | 20 +++----------------- > 2 files changed, 3 insertions(+), 18 deletions(-) > > diff --git a/include/hw/ppc/spapr_irq.h b/include/hw/ppc/spapr_irq.h > index 14cab73c9c07..f965a58f8954 100644 > --- a/include/hw/ppc/spapr_irq.h > +++ b/include/hw/ppc/spapr_irq.h > @@ -48,7 +48,6 @@ typedef struct SpaprIrq { > void (*reset)(SpaprMachineState *spapr, Error **errp); > void (*set_irq)(void *opaque, int srcno, int val); > const char *(*get_nodename)(SpaprMachineState *spapr); > - void (*init_emu)(SpaprMachineState *spapr, Error **errp); > void (*init_kvm)(SpaprMachineState *spapr, Error **errp); > } SpaprIrq; > > diff --git a/hw/ppc/spapr_irq.c b/hw/ppc/spapr_irq.c > index 73e6f10da165..84b9b32fe40f 100644 > --- a/hw/ppc/spapr_irq.c > +++ b/hw/ppc/spapr_irq.c > @@ -62,7 +62,7 @@ void spapr_irq_msi_reset(SpaprMachineState *spapr) > bitmap_clear(spapr->irq_map, 0, spapr->irq_map_nr); > } > > -static void spapr_irq_init_device(SpaprMachineState *spapr, > +static void spapr_irq_init_kvm(SpaprMachineState *spapr, > SpaprIrq *irq, Error **errp) > { > MachineState *machine = MACHINE(spapr); > @@ -88,8 +88,6 @@ static void spapr_irq_init_device(SpaprMachineState *spapr, > error_prepend(&local_err, "kernel_irqchip allowed but unavailable: "); > warn_report_err(local_err); > } > - > - irq->init_emu(spapr, errp); > } > > /* > @@ -224,7 +222,7 @@ static void spapr_irq_reset_xics(SpaprMachineState *spapr, Error **errp) > { > Error *local_err = NULL; > > - spapr_irq_init_device(spapr, &spapr_irq_xics, &local_err); > + spapr_irq_init_kvm(spapr, &spapr_irq_xics, &local_err); > if (local_err) { > error_propagate(errp, local_err); > return; > @@ -236,10 +234,6 @@ static const char *spapr_irq_get_nodename_xics(SpaprMachineState *spapr) > return XICS_NODENAME; > } > > -static void spapr_irq_init_emu_xics(SpaprMachineState *spapr, Error **errp) > -{ > -} > - > static void spapr_irq_init_kvm_xics(SpaprMachineState *spapr, Error **errp) > { > if (kvm_enabled()) { > @@ -267,7 +261,6 @@ SpaprIrq spapr_irq_xics = { > .reset = spapr_irq_reset_xics, > .set_irq = spapr_irq_set_irq_xics, > .get_nodename = spapr_irq_get_nodename_xics, > - .init_emu = spapr_irq_init_emu_xics, > .init_kvm = spapr_irq_init_kvm_xics, > }; > > @@ -385,7 +378,7 @@ static void spapr_irq_reset_xive(SpaprMachineState *spapr, Error **errp) > spapr_xive_set_tctx_os_cam(spapr_cpu_state(cpu)->tctx); > } > > - spapr_irq_init_device(spapr, &spapr_irq_xive, &local_err); > + spapr_irq_init_kvm(spapr, &spapr_irq_xive, &local_err); > if (local_err) { > error_propagate(errp, local_err); > return; > @@ -411,10 +404,6 @@ static const char *spapr_irq_get_nodename_xive(SpaprMachineState *spapr) > return spapr->xive->nodename; > } > > -static void spapr_irq_init_emu_xive(SpaprMachineState *spapr, Error **errp) > -{ > -} > - > static void spapr_irq_init_kvm_xive(SpaprMachineState *spapr, Error **errp) > { > if (kvm_enabled()) { > @@ -446,7 +435,6 @@ SpaprIrq spapr_irq_xive = { > .reset = spapr_irq_reset_xive, > .set_irq = spapr_irq_set_irq_xive, > .get_nodename = spapr_irq_get_nodename_xive, > - .init_emu = spapr_irq_init_emu_xive, > .init_kvm = spapr_irq_init_kvm_xive, > }; > > @@ -624,7 +612,6 @@ SpaprIrq spapr_irq_dual = { > .reset = spapr_irq_reset_dual, > .set_irq = spapr_irq_set_irq_dual, > .get_nodename = spapr_irq_get_nodename_dual, > - .init_emu = NULL, /* should not be used */ > .init_kvm = NULL, /* should not be used */ > }; > > @@ -840,6 +827,5 @@ SpaprIrq spapr_irq_xics_legacy = { > .reset = spapr_irq_reset_xics, > .set_irq = spapr_irq_set_irq_xics, > .get_nodename = spapr_irq_get_nodename_xics, > - .init_emu = spapr_irq_init_emu_xics, > .init_kvm = spapr_irq_init_kvm_xics, > }; ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH 0/2] spapr/xive: rework the mapping the KVM memory regions 2019-06-14 16:59 [Qemu-devel] [PATCH 0/2] spapr/xive: rework the mapping the KVM memory regions Cédric Le Goater 2019-06-14 16:59 ` [Qemu-devel] [PATCH 1/2] " Cédric Le Goater 2019-06-14 16:59 ` [Qemu-devel] [PATCH 2/2] spapr/xive: simplify spapr_irq_init_device() to remove the emulated init Cédric Le Goater @ 2019-07-01 5:15 ` David Gibson 2 siblings, 0 replies; 6+ messages in thread From: David Gibson @ 2019-07-01 5:15 UTC (permalink / raw) To: Cédric Le Goater; +Cc: qemu-ppc, Greg Kurz, qemu-devel [-- Attachment #1: Type: text/plain, Size: 1067 bytes --] On Fri, Jun 14, 2019 at 06:59:18PM +0200, Cédric Le Goater wrote: > Hello, > > Here is a small series simplifying the initialization sequence of the > interrupt device by using memory regions specific for KVM. These are > mapped as overlaps on top of the emulated device. Applied, thanks. > > Thanks, > > C. > > Cédric Le Goater (2): > spapr/xive: rework the mapping the KVM memory regions > spapr/xive: simplify spapr_irq_init_device() to remove the emulated > init > > include/hw/ppc/spapr_irq.h | 1 - > include/hw/ppc/spapr_xive.h | 2 +- > include/hw/ppc/xive.h | 1 + > hw/intc/spapr_xive.c | 38 ++++++++++--------------------------- > hw/intc/spapr_xive_kvm.c | 21 +++++++++++--------- > hw/ppc/spapr_irq.c | 21 +++----------------- > 6 files changed, 27 insertions(+), 57 deletions(-) > -- 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 [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-07-01 5:49 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-06-14 16:59 [Qemu-devel] [PATCH 0/2] spapr/xive: rework the mapping the KVM memory regions Cédric Le Goater 2019-06-14 16:59 ` [Qemu-devel] [PATCH 1/2] " Cédric Le Goater 2019-06-14 17:41 ` Greg Kurz 2019-06-14 16:59 ` [Qemu-devel] [PATCH 2/2] spapr/xive: simplify spapr_irq_init_device() to remove the emulated init Cédric Le Goater 2019-06-14 17:41 ` Greg Kurz 2019-07-01 5:15 ` [Qemu-devel] [PATCH 0/2] spapr/xive: rework the mapping the KVM memory regions David Gibson
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).