Re: [Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-22 Thread Peter Xu
On Fri, Feb 19, 2016 at 05:38:48PM +0100, Radim Krčmář wrote: > 2016-02-19 07:46+0100, Jan Kiszka: > > - Rita Sinha is currently working on integrating my old patches with the > > split-irqchip to get KVM working (as an Outreachy project). It's > > probably a bit unfortunate to consider a different

Re: [Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-20 Thread Jan Kiszka
On 2016-02-19 10:29, Peter Xu wrote: >> Would be helpful for more convenient import and experiments. E.g. I >> could quickly through my test setup at them and tell you if they work >> for it. > > I have put codes onto github for better reference: > > https://github.com/xzpeter/qemu/tree/vt-d-intr

Re: [Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-19 Thread Radim Krčmář
2016-02-19 07:46+0100, Jan Kiszka: > - Rita Sinha is currently working on integrating my old patches with the > split-irqchip to get KVM working (as an Outreachy project). It's > probably a bit unfortunate to consider a different horse that late in to > project. What do you think, how could we bene

Re: [Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-19 Thread Radim Krčmář
2016-02-19 12:43+0100, Jan Kiszka: > On 2016-02-19 12:34, Peter Xu wrote: >> On Fri, Feb 19, 2016 at 10:58:12AM +0100, Paolo Bonzini wrote: >>> >>> The source ID can be passed to the IOMMU using the MemTxAttrs mechanism. >> >> I see that MemTxAttrs is not enabled yet? It's dropped in all the >> ac

Re: [Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-19 Thread Jan Kiszka
On 2016-02-19 12:39, Peter Xu wrote: > On Fri, Feb 19, 2016 at 11:15:06AM +0100, Jan Kiszka wrote: >> On 2016-02-19 10:58, Paolo Bonzini wrote: >>> I think you're not verifying the SVT, SID and SQ fields in the IRTE. >> >> Exactly. >> >>> >>> The source ID can be passed to the IOMMU using the MemTx

Re: [Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-19 Thread Jan Kiszka
On 2016-02-19 12:34, Peter Xu wrote: > On Fri, Feb 19, 2016 at 10:58:12AM +0100, Paolo Bonzini wrote: >> >> The source ID can be passed to the IOMMU using the MemTxAttrs mechanism. > > I see that MemTxAttrs is not enabled yet? It's dropped in all the > accessors like memory_region_write_accessor()

Re: [Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-19 Thread Peter Xu
On Fri, Feb 19, 2016 at 11:15:06AM +0100, Jan Kiszka wrote: > On 2016-02-19 10:58, Paolo Bonzini wrote: > > I think you're not verifying the SVT, SID and SQ fields in the IRTE. > > Exactly. > > > > > The source ID can be passed to the IOMMU using the MemTxAttrs mechanism. > > Ah, that's a nice

Re: [Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-19 Thread Peter Xu
On Fri, Feb 19, 2016 at 10:58:12AM +0100, Paolo Bonzini wrote: > > > On 19/02/2016 10:29, Peter Xu wrote: > > On Fri, Feb 19, 2016 at 09:34:40AM +0100, Jan Kiszka wrote: > >> On 2016-02-19 08:43, Peter Xu wrote: > >>> Actually there are several people within my working team knows that > >>> I wou

Re: [Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-19 Thread Jan Kiszka
On 2016-02-19 10:58, Paolo Bonzini wrote: > > > On 19/02/2016 10:29, Peter Xu wrote: >> On Fri, Feb 19, 2016 at 09:34:40AM +0100, Jan Kiszka wrote: >>> On 2016-02-19 08:43, Peter Xu wrote: Actually there are several people within my working team knows that I would be working on this, an

Re: [Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-19 Thread Paolo Bonzini
On 19/02/2016 10:29, Peter Xu wrote: > On Fri, Feb 19, 2016 at 09:34:40AM +0100, Jan Kiszka wrote: >> On 2016-02-19 08:43, Peter Xu wrote: >>> Actually there are several people within my working team knows that >>> I would be working on this, and I believe none of us do know your >>> work too...

Re: [Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-19 Thread Peter Xu
On Fri, Feb 19, 2016 at 09:34:40AM +0100, Jan Kiszka wrote: > On 2016-02-19 08:43, Peter Xu wrote: > > Hi, Jan, > > > > On Fri, Feb 19, 2016 at 07:46:26AM +0100, Jan Kiszka wrote: > >> Hi Peter, > >> > >> On 2016-02-19 04:30, Peter Xu wrote: > >>> This patchset provide very basic functionalities f

Re: [Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-19 Thread Jan Kiszka
On 2016-02-19 08:43, Peter Xu wrote: > Hi, Jan, > > On Fri, Feb 19, 2016 at 07:46:26AM +0100, Jan Kiszka wrote: >> Hi Peter, >> >> On 2016-02-19 04:30, Peter Xu wrote: >>> This patchset provide very basic functionalities for interrupt >>> remapping (IR) support of the emulated Intel IOMMU device.

Re: [Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-18 Thread Peter Xu
Hi, Jan, On Fri, Feb 19, 2016 at 07:46:26AM +0100, Jan Kiszka wrote: > Hi Peter, > > On 2016-02-19 04:30, Peter Xu wrote: > > This patchset provide very basic functionalities for interrupt > > remapping (IR) support of the emulated Intel IOMMU device. > > Interesting. Some questions: > > - Were

Re: [Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-18 Thread Jan Kiszka
Hi Peter, On 2016-02-19 04:30, Peter Xu wrote: > This patchset provide very basic functionalities for interrupt > remapping (IR) support of the emulated Intel IOMMU device. Interesting. Some questions: - Were you aware of http://git.kiszka.org/?p=qemu.git;a=shortlog;h=refs/heads/queues/vtd-intre

[Qemu-devel] [PATCH 00/13] IOMMU: Enable interrupt remapping for Intel IOMMU

2016-02-18 Thread Peter Xu
This patchset provide very basic functionalities for interrupt remapping (IR) support of the emulated Intel IOMMU device. By default, IR is disabled to be better compatible with current QEMU. To enable IR, we can using the following command to boot a IR-supported VM with basic network (still do no