Re: [PATCH v7 10/17] KVM: arm64: introduce new KVM ITS device

2016-07-05 Thread Peter Maydell
On 5 July 2016 at 09:59, Andre Przywara wrote: > Ah, OK, so you _do_ the address setup _after_ the INIT. > My understanding of the KVM API was that this isn't allowed, as with the > INIT _everything_ should have been setup. kvmtool works this way. > > So we obviously can't change this for GICv3, b

Re: [PATCH v7 10/17] KVM: arm64: introduce new KVM ITS device

2016-07-05 Thread Auger Eric
On 05/07/2016 10:59, Andre Przywara wrote: > Hi Eric, > > thank you very much for the elaborate explanation! > > On 05/07/16 08:40, Auger Eric wrote: >> Hi Andre, >> On 04/07/2016 19:40, Andre Przywara wrote: >>> Hi Eric, >>> >>> On 04/07/16 16:00, Auger Eric wrote: Hi Peter, On

Re: [PATCH v7 10/17] KVM: arm64: introduce new KVM ITS device

2016-07-05 Thread Andre Przywara
Hi Eric, thank you very much for the elaborate explanation! On 05/07/16 08:40, Auger Eric wrote: > Hi Andre, > On 04/07/2016 19:40, Andre Przywara wrote: >> Hi Eric, >> >> On 04/07/16 16:00, Auger Eric wrote: >>> Hi Peter, >>> >>> On 04/07/2016 16:32, Peter Maydell wrote: On 4 July 2016 at 1

Re: [PATCH v7 10/17] KVM: arm64: introduce new KVM ITS device

2016-07-05 Thread Auger Eric
Hi Andre, On 04/07/2016 19:40, Andre Przywara wrote: > Hi Eric, > > On 04/07/16 16:00, Auger Eric wrote: >> Hi Peter, >> >> On 04/07/2016 16:32, Peter Maydell wrote: >>> On 4 July 2016 at 15:27, Auger Eric wrote: Andre, On 04/07/2016 16:05, Andre Przywara wrote: > Hi, > >>

Re: [PATCH v7 10/17] KVM: arm64: introduce new KVM ITS device

2016-07-05 Thread Auger Eric
Hi Andre, On 04/07/2016 19:40, Andre Przywara wrote: > Hi Eric, > > On 04/07/16 16:00, Auger Eric wrote: >> Hi Peter, >> >> On 04/07/2016 16:32, Peter Maydell wrote: >>> On 4 July 2016 at 15:27, Auger Eric wrote: Andre, On 04/07/2016 16:05, Andre Przywara wrote: > Hi, > >>>

Re: [PATCH v7 10/17] KVM: arm64: introduce new KVM ITS device

2016-07-04 Thread Andre Przywara
Hi Eric, On 04/07/16 16:00, Auger Eric wrote: > Hi Peter, > > On 04/07/2016 16:32, Peter Maydell wrote: >> On 4 July 2016 at 15:27, Auger Eric wrote: >>> Andre, >>> >>> On 04/07/2016 16:05, Andre Przywara wrote: Hi, On 04/07/16 10:00, Auger Eric wrote: > From a QEMU integratio

Re: [PATCH v7 10/17] KVM: arm64: introduce new KVM ITS device

2016-07-04 Thread Auger Eric
Hi Peter, On 04/07/2016 16:32, Peter Maydell wrote: > On 4 July 2016 at 15:27, Auger Eric wrote: >> Andre, >> >> On 04/07/2016 16:05, Andre Przywara wrote: >>> Hi, >>> >>> On 04/07/16 10:00, Auger Eric wrote: From a QEMU integration point of view this means the init sequence used for KV

Re: [PATCH v7 10/17] KVM: arm64: introduce new KVM ITS device

2016-07-04 Thread Peter Maydell
On 4 July 2016 at 15:27, Auger Eric wrote: > Andre, > > On 04/07/2016 16:05, Andre Przywara wrote: >> Hi, >> >> On 04/07/16 10:00, Auger Eric wrote: >>> From a QEMU integration point of view this means the init sequence used >>> for KVM GIC interrupt controllers cannot be reused for ITS and more >

Re: [PATCH v7 10/17] KVM: arm64: introduce new KVM ITS device

2016-07-04 Thread Auger Eric
Andre, On 04/07/2016 16:05, Andre Przywara wrote: > Hi, > > On 04/07/16 10:00, Auger Eric wrote: >> Hi Andre, >> >> On 28/06/2016 14:32, Andre Przywara wrote: >>> Introduce a new KVM device that represents an ARM Interrupt Translation >>> Service (ITS) controller. Since there can be multiple of t

Re: [PATCH v7 10/17] KVM: arm64: introduce new KVM ITS device

2016-07-04 Thread Andre Przywara
Hi, On 04/07/16 10:00, Auger Eric wrote: > Hi Andre, > > On 28/06/2016 14:32, Andre Przywara wrote: >> Introduce a new KVM device that represents an ARM Interrupt Translation >> Service (ITS) controller. Since there can be multiple of this per guest, >> we can't piggy back on the existing GICv3 d

Re: [PATCH v7 10/17] KVM: arm64: introduce new KVM ITS device

2016-07-04 Thread Auger Eric
Hi Andre, On 28/06/2016 14:32, Andre Przywara wrote: > Introduce a new KVM device that represents an ARM Interrupt Translation > Service (ITS) controller. Since there can be multiple of this per guest, > we can't piggy back on the existing GICv3 distributor device, but create > a new type of KVM d

[PATCH v7 10/17] KVM: arm64: introduce new KVM ITS device

2016-06-28 Thread Andre Przywara
Introduce a new KVM device that represents an ARM Interrupt Translation Service (ITS) controller. Since there can be multiple of this per guest, we can't piggy back on the existing GICv3 distributor device, but create a new type of KVM device. On the KVM_CREATE_DEVICE ioctl we allocate and initiali