[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH v4 0/5] virtio-iommu: VFIO integration
From: |
Linu Cherian |
Subject: |
Re: [Qemu-arm] [PATCH v4 0/5] virtio-iommu: VFIO integration |
Date: |
Wed, 4 Oct 2017 17:19:11 +0530 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Hi Eric,
On Wed Sep 27, 2017 at 11:24:01AM +0200, Auger Eric wrote:
> Hi Linu,
>
> On 27/09/2017 11:21, Linu Cherian wrote:
> > On Wed Sep 27, 2017 at 10:55:07AM +0200, Auger Eric wrote:
> >> Hi Linu,
> >>
> >> On 27/09/2017 10:30, Bharat Bhushan wrote:
> >>> Hi,
> >>>
> >>>> -----Original Message-----
> >>>> From: Linu Cherian [mailto:address@hidden
> >>>> Sent: Wednesday, September 27, 2017 1:11 PM
> >>>> To: Bharat Bhushan <address@hidden>
> >>>> Cc: address@hidden; address@hidden;
> >>>> address@hidden; address@hidden; address@hidden;
> >>>> address@hidden; address@hidden; address@hidden;
> >>>> address@hidden; address@hidden; address@hidden;
> >>>> address@hidden; address@hidden; address@hidden;
> >>>> address@hidden
> >>>> Subject: Re: [Qemu-arm] [PATCH v4 0/5] virtio-iommu: VFIO integration
> >>>>
> >>>> Hi,
> >>>>
> >>>> On Wed Sep 27, 2017 at 12:03:15PM +0530, Bharat Bhushan wrote:
> >>>>> This patch series integrates VFIO/VHOST with virtio-iommu.
> >>>>>
> >>>>> This version is mainly about rebasing on v4 version on virtio-iommu
> >>>>> device framework from Eric Augur and addresing review comments.
> >>>>>
> >>>>> This patch series allows PCI pass-through using virtio-iommu.
> >>>>>
> >>>>> This series is based on:
> >>>>> - virtio-iommu kernel driver by Jean-Philippe Brucker
> >>>>> [1] [RFC] virtio-iommu version 0.4
> >>>>> git://linux-arm.org/virtio-iommu.git branch viommu/v0.4
> >>
> >> Just to make sure, do you use the v0.4 virtio-iommu driver from above
> >> branch?
> >>
> >> Thanks
> >
> > I am using git://linux-arm.org/linux-jpb.git branch virtio-iommu/v0.4.
> > Hope you are referring to the same.
>
> Yes that's the right one. I will also investigate on my side this afternoon.
>
> Thanks
>
> Eric
With the below workaround, atleast ping works for me.
diff --git a/drivers/iommu/virtio-iommu.c b/drivers/iommu/virtio-iommu.c
index 249964a..2904617 100644
--- a/drivers/iommu/virtio-iommu.c
+++ b/drivers/iommu/virtio-iommu.c
.attach_dev = viommu_attach_dev,
.map = viommu_map,
.unmap = viommu_unmap,
- .map_sg = viommu_map_sg,
+ .map_sg = default_iommu_map_sg,
.iova_to_phys = viommu_iova_to_phys,
.add_device = viommu_add_device,
.remove_device = viommu_remove_device,
Looks like the qemu backend doesnt have support to handle the map requests from
virtio_iommu_map_sg, since it merges multiple map requests into one with
mapsize larger than page size(for eg. 0x5000).
Atleast vfio_get_vaddr called from vfio_iommu_map_notify in Qemu expects
the map size to be a power of 2.
if (len & iotlb->addr_mask) {
error_report("iommu has granularity incompatible with target AS");
return false;
}
Just trying to understand how this is not hitting in your case.
--
Linu cherian
- Re: [Qemu-arm] [PATCH v4 0/5] virtio-iommu: VFIO integration,
Linu Cherian <=
- Re: [Qemu-arm] [PATCH v4 0/5] virtio-iommu: VFIO integration, Auger Eric, 2017/10/05
- Re: [Qemu-arm] [Qemu-devel] [PATCH v4 0/5] virtio-iommu: VFIO integration, Auger Eric, 2017/10/05
- Re: [Qemu-arm] [Qemu-devel] [PATCH v4 0/5] virtio-iommu: VFIO integration, Auger Eric, 2017/10/05
- Re: [Qemu-arm] [Qemu-devel] [PATCH v4 0/5] virtio-iommu: VFIO integration, Auger Eric, 2017/10/05
- Re: [Qemu-arm] [Qemu-devel] [PATCH v4 0/5] virtio-iommu: VFIO integration, Bharat Bhushan, 2017/10/05
- Re: [Qemu-arm] [Qemu-devel] [PATCH v4 0/5] virtio-iommu: VFIO integration, Auger Eric, 2017/10/06
- Re: [Qemu-arm] [Qemu-devel] [PATCH v4 0/5] virtio-iommu: VFIO integration, Linu Cherian, 2017/10/06
- Re: [Qemu-arm] [Qemu-devel] [PATCH v4 0/5] virtio-iommu: VFIO integration, Bharat Bhushan, 2017/10/10
- Re: [Qemu-arm] [Qemu-devel] [PATCH v4 0/5] virtio-iommu: VFIO integration, Auger Eric, 2017/10/11