Re: [RFC PATCH v2 0/7] VFIO for device tree based platform devices (work in progress)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Dec 2, 2013 at 5:08 PM, Kim Phillips <kim.phillips@xxxxxxxxxx> wrote:
> On Mon, 2 Dec 2013 14:56:58 +0100
> Antonios Motakis <a.motakis@xxxxxxxxxxxxxxxxxxxxxx> wrote:
>
>> VFIO for platform is still under development, and cannot be used yet
>> to fully assign a device. However, looking at the example you ran, I
>> would not have expected it to fail so spectacularly, so this is
>> definitely something I will investigate for our next update to the
>
> it fails in executing this call in vfio-correctness-tests.c [1]:
>
> ioctl(container, VFIO_SET_IOMMU, VFIO_TYPE1_IOMMU);
>

Hm, I realize now that you where trying to bind the SMMU to the VFIO
driver. This is not the right way to use it, the SMMU device needs to
be bound by a valid SMMU driver. However, you need to bind the target
device to vfio-platform. To do that you need to edit the device tree,
as mentioned in the patch series description; this is because at the
time it was not possible to have a platform driver dynamically bind to
any device (this particular point will be fixed in future versions)

Also, you need to check from the device tree if the device is actually
linked with an SMMU. In my case I was testing using the MFC.

>> patch series. Thanks for pointing this out. I plan to release an
>> updated series with several improvements and fixes soon enough.
>
> may I ask how you test the patchseries currently?  If on arndale, any
> particular device?  Also, I'm assuming you're using a different
> userspace test than Alex Williamson's vfio-tests?
>

Since this is a work in progress, I initially used a custom test where
I directly put each IOCTL I was testing. Since devices on the Arndale
are not very well documented, today I am testing on the FastModels
SMMU model which includes a pl330 controller.

Keep in mind this version of the patchset is not supposed to fully let
you use a device with VFIO yet, just discover it and attempt some
basic mappings. In the meantime we have fixed several bugs when
actually trying to access memory regions, and added missing
functionality. Soon we will release a functional version along with
some easily reproducible tests.

Antonios

> Kim
>
> [1] git://github.com/awilliam/tests.git
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux