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