[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 10/10] hw/arm/dyn_sysbus_devtree: enable simp
From: |
Eric Auger |
Subject: |
Re: [Qemu-devel] [PATCH v5 10/10] hw/arm/dyn_sysbus_devtree: enable simple VFIO dynamic instantiation |
Date: |
Tue, 19 Aug 2014 08:59:50 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 |
On 08/18/2014 11:54 PM, Joel Schopp wrote:
>
> +static void vfio_fdt_add_device_node(SysBusDevice *sbdev, void *opaque)
> +{
> + PlatformDevtreeData *data = opaque;
> + void *fdt = data->fdt;
> + const char *parent_node = data->node;
> + int compat_str_len;
> + char *nodename;
> + int i, ret;
> + uint32_t *irq_attr;
> + uint64_t *reg_attr;
> + uint64_t mmio_base;
> + uint64_t irq_number;
> + gchar mmio_base_prop[8];
> + gchar irq_number_prop[8];
> + VFIOPlatformDevice *vdev = VFIO_PLATFORM_DEVICE(sbdev);
> + VFIODevice *vbasedev = &vdev->vbasedev;
> + Object *obj = OBJECT(sbdev);
> +
> + mmio_base = object_property_get_int(obj, "mmio[0]", NULL);
> +
> + nodename = g_strdup_printf("%s/address@hidden" PRIx64, parent_node,
> + vbasedev->name,
> + mmio_base);
> +
> + qemu_fdt_add_subnode(fdt, nodename);
> +
> + compat_str_len = strlen(vdev->compat) + 1;
> + qemu_fdt_setprop(fdt, nodename, "compatible",
> + vdev->compat, compat_str_len);
> +
> + reg_attr = g_new(uint64_t, vbasedev->num_regions*4);
> +
> + for (i = 0; i < vbasedev->num_regions; i++) {
> + snprintf(mmio_base_prop, sizeof(mmio_base_prop), "mmio[%d]", i);
> + mmio_base = object_property_get_int(obj, mmio_base_prop, NULL);
> + reg_attr[2*i] = 1;
> + reg_attr[2*i+1] = mmio_base;
> + reg_attr[2*i+2] = 1;
> + reg_attr[2*i+3] = memory_region_size(&vdev->regions[i]->mem);
> + }
>
> This should be 4 instead of 2.
Hi Joel,
Yes definitively! Forgot to restore the original value after trying
different qemu_fdt_setprop_* functions. sorry for that.
Best Regards
Eric
> Also, to support 64 bit systems I think this should be 2 instead of 1.
>
- [Qemu-devel] [PATCH v5 09/10] hw/vfio/platform: Add irqfd support, (continued)
- Re: [Qemu-devel] [PATCH v5 10/10] hw/arm/dyn_sysbus_devtree: enable simple VFIO dynamic instantiation,
Eric Auger <=