On Fri, 25 Aug 2023, Vikram Garhwal wrote:
> From: Oleksandr Tyshchenko <oleksandr_tyshche...@epam.com>
>
> In order to use virtio backends we need to allocate virtio-mmio
> parameters (irq and base) and register corresponding buses.
>
> Use the constants defined in public header arch-arm.h to be
> aligned with the toolstack. So the number of current supported
> virtio-mmio devices is 10.
>
> For the interrupts triggering use already existing on Arm
> device-model hypercall.
>
> The toolstack should then insert the same amount of device nodes
> into guest device-tree.
>
> Signed-off-by: Oleksandr Tyshchenko <oleksandr_tyshche...@epam.com>
> Signed-off-by: Vikram Garhwal <vikram.garh...@amd.com>
> ---
> hw/arm/xen_arm.c | 51 ++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 51 insertions(+)
>
> diff --git a/hw/arm/xen_arm.c b/hw/arm/xen_arm.c
> index 1d3e6d481a..d1e9f7b488 100644
> --- a/hw/arm/xen_arm.c
> +++ b/hw/arm/xen_arm.c
> @@ -26,6 +26,7 @@
> #include "qapi/qapi-commands-migration.h"
> #include "qapi/visitor.h"
> #include "hw/boards.h"
> +#include "hw/irq.h"
> #include "hw/sysbus.h"
> #include "sysemu/block-backend.h"
> #include "sysemu/tpm_backend.h"
> @@ -59,6 +60,54 @@ struct XenArmState {
> } cfg;
> };
>
> +/*
> + * VIRTIO_MMIO_DEV_SIZE is imported from tools/libs/light/libxl_arm.c under
> Xen
> + * repository.
> + *
> + * Origin: git://xenbits.xen.org/xen.git 2128143c114c
> + */
> +#define VIRTIO_MMIO_DEV_SIZE 0x200
> +
> +#define NR_VIRTIO_MMIO_DEVICES \
> + (GUEST_VIRTIO_MMIO_SPI_LAST - GUEST_VIRTIO_MMIO_SPI_FIRST)
> +
> +#if CONFIG_XEN_CTRL_INTERFACE_VERSION <= 41500
> +static int xendevicemodel_set_irq_level(
> + xendevicemodel_handle *dmod, domid_t domid, uint32_t irq,
> + unsigned int level)
> +{
> + return 0;
> +}
> +#endif
> +
> +#if CONFIG_XEN_CTRL_INTERFACE_VERSION <= 41700
> +#define GUEST_VIRTIO_MMIO_BASE xen_mk_ullong(0x02000000)
> +#define GUEST_VIRTIO_MMIO_SIZE xen_mk_ullong(0x00100000)
> +#define GUEST_VIRTIO_MMIO_SPI_FIRST 33
> +#define GUEST_VIRTIO_MMIO_SPI_LAST 43
> +#endif
Thanks Vikram. Please move this compat definitions to
include/hw/xen/xen_native.h
> +static void xen_set_irq(void *opaque, int irq, int level)
> +{
> + xendevicemodel_set_irq_level(xen_dmod, xen_domid, irq, level);
> +}
> +
> +static void xen_create_virtio_mmio_devices(XenArmState *xam)
> +{
> + int i;
> +
> + for (i = 0; i < NR_VIRTIO_MMIO_DEVICES; i++) {
> + hwaddr base = GUEST_VIRTIO_MMIO_BASE + i * VIRTIO_MMIO_DEV_SIZE;
> + qemu_irq irq = qemu_allocate_irq(xen_set_irq, NULL,
> + GUEST_VIRTIO_MMIO_SPI_FIRST + i);
> +
> + sysbus_create_simple("virtio-mmio", base, irq);
> +
> + DPRINTF("Created virtio-mmio device %d: irq %d base 0x%lx\n",
> + i, GUEST_VIRTIO_MMIO_SPI_FIRST + i, base);
> + }
> +}
> +
> void arch_handle_ioreq(XenIOState *state, ioreq_t *req)
> {
> hw_error("Invalid ioreq type 0x%x\n", req->type);
> @@ -110,6 +159,8 @@ static void xen_arm_init(MachineState *machine)
>
> xen_register_ioreq(xam->state, machine->smp.cpus, &xen_memory_listener);
>
> + xen_create_virtio_mmio_devices(xam);
> +
> #ifdef CONFIG_TPM
> if (xam->cfg.tpm_base_addr) {
> xen_enable_tpm(xam);
> --
> 2.17.1
>
