Hi Peter,
Thanks, I think that I should receive credit for this patch.
Please attribute it under my technion mail: bda...@cs.technion.ac.il.
The signed-off line should be:
Signed-off-by: Aviv Ben-David
Thanks,
Aviv.
On Thu, Mar 16, 2017 at 6:05 AM, Peter Xu wrote:
> On Tue, Feb 07, 2017 at
On Tue, Nov 29, 2016 at 5:23 AM 蓝天宇 wrote:
> 2016-11-28 23:51 GMT+08:00 Aviv B.D :
> > From: "Aviv Ben-David"
> >
> > Adds a list of registered vtd_as's to intel iommu state to save
> > iteration over each PCI device in a search of the corrosponding dom
On Mon, Nov 28, 2016 at 6:36 PM Alex Williamson
wrote:
> On Mon, 28 Nov 2016 17:51:55 +0200
> "Aviv B.D" wrote:
>
> > From: "Aviv Ben-David"
> >
> > Currently the implementation preventing VFIO to work together with
> > intel_iommu.
> &
From: "Aviv Ben-David"
Adds a list of registered vtd_as's to intel iommu state to save
iteration over each PCI device in a search of the corrosponding domain.
Signed-off-by: Aviv Ben-David
---
hw/i386/intel_iommu.c | 94 ++
hw/i386/intel_iommu_i
From: "Aviv Ben-David"
Supports translation trials without reporting error to guest on
translation failure.
Signed-off-by: Aviv Ben-David
---
exec.c | 3 ++-
hw/alpha/typhoon.c | 2 +-
hw/i386/amd_iommu.c | 4 ++--
hw/i386/intel_iommu.c | 59 +
From: "Aviv Ben-David"
The default implementation scans the address space and try to find
translation for each page, if exists.
This callback enables effiecent implementation for intel_iommu and other
subsystems with large address space.
Signed-off-by: Aviv Ben-David
---
include/exec/memory.h
From: "Aviv Ben-David"
This capability asks the guest to invalidate cache before each map operation.
We can use this invalidation to trap map operations in the hypervisor.
Signed-off-by: Aviv Ben-David
---
hw/i386/intel_iommu.c | 5 +
hw/i386/intel_iommu_internal.h | 1 +
include/
From: "Aviv Ben-David"
Currently the implementation preventing VFIO to work together with
intel_iommu.
Signed-off-by: Aviv Ben-David
---
hw/i386/intel_iommu.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/hw/i386/intel_iommu.c b/hw/i386/intel_iommu.c
index d872969..0787714 100644
From: "Aviv Ben-David"
* Advertize Cache Mode capability in iommu cap register.
This capability is controlled by "cache-mode" property of intel-iommu device.
To enable this option call QEMU with "-device intel-iommu,cache-mode=true".
* On page cache invalidation in intel vIOMMU, check if th
wrote:
>>>
>>>> > >On Fri, Nov 11, 2016 at 10:32:42AM +0800, Jason Wang wrote:
>>>>
>>>>> > > >
>>>>> > > >On 2016年11月10日 06:00, Michael S. Tsirkin wrote:
>>>>>
>>>>>> > > > &g
> On Wed, Nov 09, 2016 at 03:28:02PM +0800, Jason Wang wrote:
>>>>
>>>>>
>>>>>> On 2016年11月08日 19:04, Aviv B.D wrote:
>>>>>>
>>>>>>> From: "Aviv Ben-David"
>>>>>>>>
&g
On Wed, Nov 16, 2016 at 9:57 PM, Michael S. Tsirkin wrote:
> On Wed, Nov 16, 2016 at 09:50:46PM +0200, Aviv B.D. wrote:
> >
> >
> > On Wed, Nov 16, 2016 at 5:34 PM, Alex Williamson <
> alex.william...@redhat.com>
> > wrote:
> >
> > On We
gt; > On Thu, Nov 10, 2016 at 08:30:21AM -0700, Alex Williamson wrote:
> > > > > > > On Thu, 10 Nov 2016 17:14:24 +0200
> > > > > > > "Michael S. Tsirkin" wrote:
> > > > > > >
> > > > > > > > On Tue,
Williamson wrote:
> > > > On Thu, 10 Nov 2016 17:14:24 +0200
> > > > "Michael S. Tsirkin" wrote:
> > > >
> > > > > On Tue, Nov 08, 2016 at 01:04:21PM +0200, Aviv B.D wrote:
> > > > > > From: "Aviv Ben-David"
From: "Aviv Ben-David"
This capability asks the guest to invalidate cache before each map operation.
We can use this invalidation to trap map operations in the hypervisor.
Signed-off-by: Aviv Ben-David
---
hw/i386/intel_iommu.c | 5 +
hw/i386/intel_iommu_internal.h | 1 +
include/
From: "Aviv Ben-David"
Adds a list of registered vtd_as's to intel iommu state to save
iteration over each PCI device in a search of the corrosponding domain.
Signed-off-by: Aviv Ben-David
---
hw/i386/intel_iommu.c | 96 +++---
hw/i386/intel_iommu_i
From: "Aviv Ben-David"
Supports translation trials without reporting error to guest on
translation failure.
Signed-off-by: Aviv Ben-David
---
exec.c | 3 ++-
hw/alpha/typhoon.c | 2 +-
hw/i386/amd_iommu.c | 4 ++--
hw/i386/intel_iommu.c | 59 +
From: "Aviv Ben-David"
* Advertize Cache Mode capability in iommu cap register.
This capability is controlled by "cache-mode" property of intel-iommu device.
To enable this option call QEMU with "-device intel-iommu,cache-mode=true".
* On page cache invalidation in intel vIOMMU, check if th
I will fix the warning in apb.c...
Can you post your qemu build configuration?
Thanks,
Aviv.
On Wed, Oct 26, 2016 at 6:37 PM, Chao Gao wrote:
> On Fri, Oct 21, 2016 at 12:07:18AM +0300, Aviv B.D wrote:
> >From: "Aviv Ben-David"
> >
> >* Advertize Cache Mode c
On Mon, Oct 24, 2016 at 11:02 AM, Peter Xu wrote:
> On Mon, Oct 24, 2016 at 10:53:01AM +0300, Aviv B.D. wrote:
>
> [...]
>
> > This was my first algorithm, but VFIO do not support remapping of mapped
> > page.
> > Before each MAP operation in VFIO one must do unmap,
On Fri, Oct 21, 2016 at 6:57 AM, Peter Xu wrote:
> On Thu, Oct 20, 2016 at 10:11:15PM +0300, Aviv B.D. wrote:
>
> [...]
>
> > > > +static void vtd_iotlb_page_invalidate_notify(IntelIOMMUState *s,
> > > > + uin
From: "Aviv Ben-David"
Supports translation trials without reporting error to guest on
translation failure.
Signed-off-by: Aviv Ben-David
---
exec.c| 3 ++-
hw/i386/amd_iommu.c | 4 ++--
hw/i386/intel_iommu.c | 59 +--
hw/ppc/
From: "Aviv Ben-David"
Adds a list of registered vtd_as's to intel iommu state to save
iteration over each PCI device in a search of the corrosponding domain.
Signed-off-by: Aviv Ben-David
---
hw/i386/intel_iommu.c | 109 ++---
hw/i386/intel_iommu_i
From: "Aviv Ben-David"
* Advertize Cache Mode capability in iommu cap register.
This capability is controlled by "cache-mode" property of intel-iommu device.
To enable this option call QEMU with "-device intel-iommu,cache-mode=true".
* On page cache invalidation in intel vIOMMU, check if th
From: "Aviv Ben-David"
This capability asks the guest to invalidate cache before each map operation.
We can use this invalidation to trap map operations in the hypervisor.
Signed-off-by: Aviv Ben-David
---
hw/i386/intel_iommu.c | 5 +
hw/i386/intel_iommu_internal.h | 1 +
include/
On Mon, Oct 17, 2016 at 7:07 PM, Alex Williamson wrote:
> On Mon, 17 Oct 2016 18:44:21 +0300
> "Aviv B.D" wrote:
>
> > From: "Aviv Ben-David"
> >
> > * Advertize Cache Mode capability in iommu cap register.
> > This capability is contro
On Wed, Oct 19, 2016 at 12:33 PM, Peter Xu wrote:
> On Mon, Oct 17, 2016 at 06:44:24PM +0300, Aviv B.D wrote:
> > From: "Aviv Ben-David"
> >
> > Adds a list of registered vtd_as's to intel iommu state to save
> > iteration over each PCI devi
I will fix all those issues.
Thanks,
Aviv.
On Thu, Oct 20, 2016 at 10:11 PM, Aviv B.D. wrote:
>
>
> On Wed, Oct 19, 2016 at 12:33 PM, Peter Xu wrote:
>
>> On Mon, Oct 17, 2016 at 06:44:24PM +0300, Aviv B.D wrote:
>> > From: "Aviv Ben-David"
>> &
Hi,
You are right, and I will revert those functions you mentioned.
Thanks,
Aviv.
On Wed, Oct 19, 2016 at 11:35 AM, Peter Xu wrote:
> On Mon, Oct 17, 2016 at 06:44:23PM +0300, Aviv B.D wrote:
>
> [...]
>
> > @@ -364,7 +364,7 @@ static void vtd_set_frcd_and_update_ppf(In
From: "Aviv Ben-David"
* Advertize Cache Mode capability in iommu cap register.
This capability is controlled by "cache-mode" property of intel-iommu device.
To enable this option call QEMU with "-device intel-iommu,cache-mode=true".
* On page cache invalidation in intel vIOMMU, check if th
From: "Aviv Ben-David"
Supports translation trials without reporting error to guest on
translation failure.
Signed-off-by: Aviv Ben-David
---
exec.c| 3 ++-
hw/i386/amd_iommu.c | 4 +--
hw/i386/intel_iommu.c | 70 +--
include/
From: "Aviv Ben-David"
This capability asks the guest to invalidate cache before each map operation.
We can use this invalidation to trap map operations in the hypervisor.
Signed-off-by: Aviv Ben-David
---
hw/i386/intel_iommu.c | 5 +
hw/i386/intel_iommu_internal.h | 1 +
include/
From: "Aviv Ben-David"
Adds a list of registered vtd_as's to intel iommu state to save
iteration over each PCI device in a search of the corrosponding domain.
Signed-off-by: Aviv Ben-David
---
hw/i386/intel_iommu.c | 102 ++---
hw/i386/intel_iommu_i
From: "Aviv Ben-David"
Supports translation trials without reporting error to guest on
translation failure.
Signed-off-by: Aviv Ben-David
---
exec.c| 2 +-
hw/i386/amd_iommu.c | 4 ++--
hw/i386/intel_iommu.c | 65 ---
include/
From: "Aviv Ben-David"
This capability asks the guest to invalidate cache before each map operation.
We can use this invalidation to trap map operations in the hypervisor.
This capability only enabled when cache-mode property of the device is true.
Signed-off-by: Aviv Ben-David
---
hw/i386/in
From: "Aviv Ben-David"
* Advertize Cache Mode capability in iommu cap register.
This capability is controlled by "cache-mode" property of intel-iommu device.
To enable this option call QEMU with "-device intel-iommu,cache-mode=true".
* On page cache invalidation in intel vIOMMU, check if th
From: "Aviv Ben-David"
* register notifiers when vtd_iommu_notify_flag_changed is called.
* Notify on IOTLB page invalidation about unmap and map operations.
Adds a list of registered vtd_as's to intel iommu state to save
iterations over each PCI device in a search of the corrosponding domain.
, 28 May 2016 18:14:18 +
> "Aviv B.D." wrote:
>
> > Hi,
> > As far as I tested the disabled code (call to memory_region_iommu_replay)
> > hangup
> > QEMU on startup if IOMMU is enabled (scaning 64 bit address space takes
> > more
> > than an
, 28 May 2016 16:10:55 +
> "Aviv B.D." wrote:
>
> > On Sat, May 28, 2016 at 7:02 PM Alex Williamson <
> alex.william...@redhat.com>
> > wrote:
> >
> > > On Sat, 28 May 2016 10:52:58 +
> > > "Aviv B.D." wrote:
> >
What is the best way to add this configuration option?
Aviv.
On Tue, May 24, 2016 at 12:25 PM Jan Kiszka wrote:
> On 2016-05-24 10:14, Jason Wang wrote:
> > On 2016年05月22日 00:19, Aviv B.D wrote:
> >> From: "Aviv Ben-David"
> >>
> >> This flag tell
On Sat, May 28, 2016 at 7:02 PM Alex Williamson
wrote:
> On Sat, 28 May 2016 10:52:58 +
> "Aviv B.D." wrote:
>
> > Hi,
> > Your idea to search the relevent VTDAddressSpace and call it's notifier
> > will
> > probably work. Next week I&#
ote:
>
> > On Sat, 21 May 2016 19:19:50 +0300
> > "Aviv B.D" wrote:
> >
> > > From: "Aviv Ben-David"
> > >
> >
> > Some commentary about the changes necessary to achieve $SUBJECT would
> > be nice h
From: "Aviv Ben-David"
Supports translation trials without reporting error to guest on translation
failures.
Signed-off-by: Aviv Ben-David
---
exec.c| 2 +-
hw/i386/intel_iommu.c | 65 ---
include/exec/memory.h | 4 ++--
memory
From: "Aviv Ben-David"
This flag tells the guest to invalidate tlb cache also after unmap operations.
Signed-off-by: Aviv Ben-David
---
hw/i386/intel_iommu.c | 3 ++-
hw/i386/intel_iommu_internal.h | 1 +
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/hw/i386/intel_iomm
From: "Aviv Ben-David"
Signed-off-by: Aviv Ben-David
---
hw/i386/intel_iommu.c | 69 --
hw/i386/intel_iommu_internal.h | 2 ++
hw/vfio/common.c | 11 +--
include/hw/i386/intel_iommu.h | 4 +++
include/hw/vfio/vfio-common.h |
From: "Aviv Ben-David"
* Advertize Cache Mode capability in iommu cap register.
* Register every VFIO device with IOMMU state.
* On page cache invalidation in vIOMMU, check if the domain belong to
VFIO device and mirror the guest requests to host.
Changes from v1 to v2:
* remove assumption tha
See my comments below,
Thanks,
Aviv.
On Mon, Apr 11, 2016 at 11:25 PM, Alex Williamson
wrote:
>
> Some more detailed comments now that I have some faith that the host
> IOMMU domain is working correctly...
>
> On Sat, 9 Apr 2016 21:03:38 +0300
> "Aviv B.D." wrote
see comments below.
Thanks for your inputs,
Aviv
On Mon, Apr 11, 2016 at 6:40 AM, Peter Xu wrote:
> Hi, Aviv,
>
> On Sat, Apr 09, 2016 at 09:03:38PM +0300, Aviv B.D. wrote:
>
> [...]
>
>> +static int vtd_get_did_dev(IntelIOMMUState *s, uint8_t bus_num,
>> uint8
From: "Aviv Ben-David"
Date: Tue, 23 Feb 2016 00:24:54 +0200
Subject: [PATCH] IOMMU: Add Support to VFIO devices with vIOMMU present
* Fix bug that prevent qemu from starting up with vIOMMU and VFIO
device are present.
* Advertize Cache Mode capability in iommu cap register.
* Register every VF
On Wed, Mar 23, 2016 at 5:33 PM, Michael S. Tsirkin wrote:
> On Sat, Mar 12, 2016 at 06:13:17PM +0200, Aviv B.D. wrote:
> > From: "Aviv B.D."
> >
> > * Fix bug that prevent qemu from starting up when vIOMMU and VFIO
> > device are present.
> >
On Mon, Mar 21, 2016 at 4:30 AM, Peter Xu wrote:
> On Sat, Mar 19, 2016 at 11:40:04AM +0200, Aviv B.D. wrote:
> [...]
> > As far as I understand the code, currently there is no way to turn off
> the
> > IOTLB.
> > Furthermore. the IOTLB is not implemented
On Fri, Mar 18, 2016 at 5:06 AM, Peter Xu wrote:
> On Thu, Mar 17, 2016 at 01:17:30PM +0200, Aviv B.D. wrote:
> [...]
> > vtd_get_did_dev returns -1 if the device is not mapped to any domain
> > (generally, the CE is not present).
> > probably a better interface is to r
On Tue, Mar 15, 2016 at 10:52 AM, Peter Xu wrote:
> On Mon, Mar 14, 2016 at 08:52:33PM +0200, Marcel Apfelbaum wrote:
> > On 03/12/2016 06:13 PM, Aviv B.D. wrote:
> > Adding (possibly) interested developers to the thread.
>
> Thanks CC.
>
> Hi, Avi
On Tue, Mar 15, 2016 at 12:53 PM, Michael S. Tsirkin wrote:
> On Sat, Mar 12, 2016 at 06:13:17PM +0200, Aviv B.D. wrote:
> > From: "Aviv B.D."
> >
> > * Fix bug that prevent qemu from starting up when vIOMMU and VFIO
> > device are present.
> >
From: "Aviv B.D."
* Fix bug that prevent qemu from starting up when vIOMMU and VFIO
device are present.
* Advertise Cache Mode capability in iommu cap register.
* Register every VFIO device with IOMMU state.
* On page cache invalidation in vIOMMU, check if the domain belong
55 matches
Mail list logo
