[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 11/16] hw/arm/dyn_sysbus_devtree: enable vfio
From: |
Eric Auger |
Subject: |
Re: [Qemu-devel] [PATCH v6 11/16] hw/arm/dyn_sysbus_devtree: enable vfio-calxeda-xgmac dynamic instantiation |
Date: |
Thu, 11 Sep 2014 16:20:09 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 |
On 09/10/2014 03:12 PM, Alexander Graf wrote:
>
>
> On 09.09.14 09:31, Eric Auger wrote:
>> vfio-calxeda-xgmac now can be instantiated using the -device option
>>
>> Signed-off-by: Eric Auger <address@hidden>
>>
>> ---
>>
>> v2 -> v3:
>> - correct bug of reg_attr[2*i] in vfio_fdt_add_device_node
>> - fix a bug related to compat_str_len computed on original compat
>> instead of corrected compat
>> - wrap_vfio_fdt_add_node take a node creation function: this function
>> needs to be specialized for each VFIO device. wrap function must be
>> called in sysbus_device_create_devtree
>> ---
>> hw/arm/dyn_sysbus_devtree.c | 141
>> ++++++++++++++++++++++++++++++++++++++++++++
>> 1 file changed, 141 insertions(+)
>>
>> diff --git a/hw/arm/dyn_sysbus_devtree.c b/hw/arm/dyn_sysbus_devtree.c
>> index 61e5b5f..3ef9430 100644
>> --- a/hw/arm/dyn_sysbus_devtree.c
>> +++ b/hw/arm/dyn_sysbus_devtree.c
>> @@ -20,6 +20,141 @@
>> #include "hw/arm/dyn_sysbus_devtree.h"
>> #include "qemu/error-report.h"
>> #include "sysemu/device_tree.h"
>> +#include "hw/vfio/vfio-platform.h"
>> +#include "hw/vfio/vfio-calxeda-xgmac.h"
>> +
>> +typedef void (*vfio_fdt_add_device_node_t)(SysBusDevice *sbdev, void
>> *opaque);
>> +
>> +static char *format_compat(char * compat)
>> +{
>> + char *str_ptr, *corrected_compat;
>> + /*
>> + * process compatibility property string passed by end-user
>> + * replaces / by , and ; by NUL character
>> + */
>> + corrected_compat = g_strdup(compat);
>> +
>> + str_ptr = corrected_compat;
>> + while ((str_ptr = strchr(str_ptr, '/')) != NULL) {
>> + *str_ptr = ',';
>> + }
>> +
>> + /* substitute ";" with the NUL char */
>> + str_ptr = corrected_compat;
>> + while ((str_ptr = strchr(str_ptr, ';')) != NULL) {
>> + *str_ptr = '\0';
>> + }
>> +
>> + /*
>> + * corrected compat includes a "\0" before or at the same location
>> + * as compat's one
>> + */
>> + return corrected_compat;
>> +}
>> +
>> +static void wrap_vfio_fdt_add_node(SysBusDevice *sbdev, void *opaque,
>> + vfio_fdt_add_device_node_t add_node_fn)
>> +{
>> + PlatformDevtreeData *data = opaque;
>> + VFIOPlatformDevice *vdev = VFIO_PLATFORM_DEVICE(sbdev);
>> + VFIODevice *vbasedev = &vdev->vbasedev;
>> + gchar irq_number_prop[8];
>> + Object *obj = OBJECT(sbdev);
>> + char *corrected_compat;
>> + uint64_t irq_number;
>> + int corrected_compat_str_len, i;
>> +
>> + corrected_compat = format_compat(vdev->compat);
>> + corrected_compat_str_len = strlen(corrected_compat) + 1;
>> + /* we copy the corrected_compat string + its "\0" */
>> + snprintf(vdev->compat, corrected_compat_str_len, "%s",
>> corrected_compat);
>> + g_free(corrected_compat);
>> +
>> + add_node_fn(sbdev, opaque);
>> +
>> + for (i = 0; i < vbasedev->num_irqs; i++) {
>> + snprintf(irq_number_prop, sizeof(irq_number_prop), "irq[%d]", i);
>> + irq_number = object_property_get_int(obj, irq_number_prop, NULL)
>> + + data->irq_start;
>> + /*
>> + * for setting irqfd up we must provide the virtual IRQ number
>> + * which is the sum of irq_start and actual platform bus irq
>> + * index. At realize point we do not have this info.
>> + */
>> + vfio_start_irq_injection(sbdev, i, irq_number);
>
> Does this really have anything to do with fdt?
No it doesn't, I aknowledge ;-)
Also, don't we have
> notifiers that call IRQ holders when an IRQ gets connected?
Do we? I was not able to identify such modality. the notifier would be
triggered in qemu_allocate_irq right?
That would
> probably be the cleaner approach here.
If it is and if the functionality does not exist yet, I can it, sure.
Besides I do not get how VFIO-PCI does handle the problem of late irq
binding. If someone can share some knowledge on this, it would be much
appreciated.
Best Regards
Eric
>
>
> Alex
>
- [Qemu-devel] [PATCH v6 07/16] hw/vfio/pci: use name field in format strings, (continued)
[Qemu-devel] [PATCH v6 09/16] hw/vfio/platform: add vfio-platform support, Eric Auger, 2014/09/09
[Qemu-devel] [PATCH v6 10/16] hw/vfio: calxeda xgmac device, Eric Auger, 2014/09/09
[Qemu-devel] [PATCH v6 11/16] hw/arm/dyn_sysbus_devtree: enable vfio-calxeda-xgmac dynamic instantiation, Eric Auger, 2014/09/09
[Qemu-devel] [PATCH v6 13/16] hw/vfio/platform: Add irqfd support, Eric Auger, 2014/09/09
[Qemu-devel] [PATCH v6 12/16] vfio/platform: add fake injection modality, Eric Auger, 2014/09/09
[Qemu-devel] [PATCH v6 14/16] linux-headers: Update KVM headers from linux-next tag ToBeFilled, Eric Auger, 2014/09/09
[Qemu-devel] [PATCH v6 15/16] VFIO: COMMON: vfio_kvm_device_fd moved in the common header, Eric Auger, 2014/09/09
[Qemu-devel] [PATCH v6 16/16] VFIO: PLATFORM: add forwarded irq support, Eric Auger, 2014/09/09
Re: [Qemu-devel] [PATCH v6 00/16] KVM platform device passthrough, Alex Williamson, 2014/09/11