[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH 07/12] arm/sdei: override qemu_irq handler when binding i
From: |
Peter Maydell |
Subject: |
Re: [RFC PATCH 07/12] arm/sdei: override qemu_irq handler when binding interrupt |
Date: |
Mon, 30 Sep 2019 14:19:15 +0100 |
On Tue, 24 Sep 2019 at 16:23, Heyi Guo <address@hidden> wrote:
>
> Override qemu_irq handler to support trigger SDEI event transparently
> after guest binds interrupt to SDEI event. We don't have good way to
> get GIC device and to guarantee SDEI device is initialized after GIC,
> so we search GIC in system bus when the first SDEI request happens or
> in VMSTATE post_load().
>
> Signed-off-by: Heyi Guo <address@hidden>
> Cc: Peter Maydell <address@hidden>
> Cc: Dave Martin <address@hidden>
> Cc: Marc Zyngier <address@hidden>
> Cc: Mark Rutland <address@hidden>
> Cc: James Morse <address@hidden>
> +static void override_qemu_irq(QemuSDEState *s, int32_t event, uint32_t intid)
> +{
> + qemu_irq irq;
> + QemuSDE *sde;
> + CPUState *cs;
> + int cpu;
> +
> + /* SPI */
> + if (intid >= GIC_INTERNAL) {
> + cs = arm_get_cpu_by_id(0);
> + irq = qdev_get_gpio_in(s->gic_dev,
> + gic_int_to_irq(s->num_irq, intid, 0));
> + if (irq) {
> + qemu_irq_intercept_in(&irq, qemu_sdei_irq_handler, 1);
> + }
I'm not sure what this code is trying to do, but
qemu_irq_intercept_in() is a function for internal use
by the qtest testing infrastructure, so it shouldn't be
used in 'real' QEMU code.
> + sde = get_sde_no_check(s, event, cs);
> + sde->irq = irq;
> + put_sde(sde, cs);
> + return;
> + }
> @@ -1042,6 +1152,17 @@ void sdei_handle_request(CPUState *cs, struct kvm_run
> *run)
> return;
> }
>
> + if (!sde_state->gic_dev) {
> + /* Search for ARM GIC device */
> + qbus_walk_children(sysbus_get_default(), dev_walkerfn,
> + NULL, NULL, NULL, sde_state);
> + if (!sde_state->gic_dev) {
> + error_report("Cannot find ARM GIC device!");
> + run->hypercall.args[0] = SDEI_NOT_SUPPORTED;
> + return;
> + }
> + }
Walking through the qbus tree looking for particular devices
isn't really something I'd recommend either.
thanks
-- PMM
- [RFC PATCH 00/12] Add SDEI support for arm64, Heyi Guo, 2019/09/24
- [RFC PATCH 01/12] linux-headers: import arm_sdei.h, Heyi Guo, 2019/09/24
- [RFC PATCH 10/12] arm/sdei: check KVM cap and enable SDEI, Heyi Guo, 2019/09/24
- [RFC PATCH 04/12] arm/sdei: add system reset callback, Heyi Guo, 2019/09/24
- [RFC PATCH 06/12] core/irq: add qemu_irq_remove_intercept interface, Heyi Guo, 2019/09/24
- [RFC PATCH 02/12] arm/sdei: add virtual device framework, Heyi Guo, 2019/09/24
- [RFC PATCH 07/12] arm/sdei: override qemu_irq handler when binding interrupt, Heyi Guo, 2019/09/24
- Re: [RFC PATCH 07/12] arm/sdei: override qemu_irq handler when binding interrupt,
Peter Maydell <=
- [RFC PATCH 03/12] arm/sdei: add support to handle SDEI requests from guest, Heyi Guo, 2019/09/24
- [RFC PATCH 05/12] arm/sdei: add support to trigger event by GIC interrupt ID, Heyi Guo, 2019/09/24
- [RFC PATCH 09/12] linux-headers/kvm.h: add capability to forward hypercall, Heyi Guo, 2019/09/24
- [RFC PATCH 08/12] arm/sdei: add support to register interrupt bind notifier, Heyi Guo, 2019/09/24
- [RFC PATCH 11/12] arm/kvm: handle guest exit of hypercall, Heyi Guo, 2019/09/24
- [RFC PATCH 12/12] virt/acpi: add SDEI table if SDEI is enabled, Heyi Guo, 2019/09/24
- Re: [RFC PATCH 00/12] Add SDEI support for arm64, no-reply, 2019/09/25
- Re: [RFC PATCH 00/12] Add SDEI support for arm64, no-reply, 2019/09/25
- Re: [RFC PATCH 00/12] Add SDEI support for arm64, Peter Maydell, 2019/09/30