Re: [PATCH] KVM: arm/arm64: vgic: Add debugfs vgic-state file

2017-01-24 Thread Christoffer Dall
On Tue, Jan 24, 2017 at 10:23:57AM +, Andre Przywara wrote: > Hi, > > so I gave this patch (adapted to live without the soft_pending state) > some testing on my machines (Midway, Juno, GICv3 ITS h/w) and it seemed > to work well - at least if one is nice and starts only one process >

Re: [PATCH] KVM: arm/arm64: vgic: Add debugfs vgic-state file

2017-01-24 Thread Andre Przywara
Hi, [...] As I didn't understand the seq_* semantics in the first place, I didn't have a look yet what could cause this. >>> >>> Nice catch, I'll have a look at this. >>> >>> Just so I'm sure, these are two processes reading the vgic-state file >>> for the same single VM, right?

Re: [PATCH] KVM: arm/arm64: vgic: Add debugfs vgic-state file

2017-01-24 Thread Christoffer Dall
On Tue, Jan 24, 2017 at 12:25:25PM +, Andre Przywara wrote: > Hi, > > On 24/01/17 10:58, Christoffer Dall wrote: > > On Tue, Jan 24, 2017 at 10:23:57AM +, Andre Przywara wrote: > >> Hi, > >> > >> so I gave this patch (adapted to live without the soft_pending state) > >> some testing on my

Re: [PATCH] KVM: arm/arm64: vgic: Add debugfs vgic-state file

2017-01-24 Thread Andre Przywara
Hi, On 24/01/17 10:58, Christoffer Dall wrote: > On Tue, Jan 24, 2017 at 10:23:57AM +, Andre Przywara wrote: >> Hi, >> >> so I gave this patch (adapted to live without the soft_pending state) >> some testing on my machines (Midway, Juno, GICv3 ITS h/w) and it seemed >> to work well - at least

Re: [PATCH] KVM: arm/arm64: vgic: Add debugfs vgic-state file

2017-01-24 Thread Christoffer Dall
On Tue, Jan 24, 2017 at 10:23:57AM +, Andre Przywara wrote: > Hi, > > so I gave this patch (adapted to live without the soft_pending state) > some testing on my machines (Midway, Juno, GICv3 ITS h/w) and it seemed > to work well - at least if one is nice and starts only one process >

Re: [PATCH] KVM: arm/arm64: vgic: Add debugfs vgic-state file

2017-01-24 Thread Andre Przywara
Hi, so I gave this patch (adapted to live without the soft_pending state) some testing on my machines (Midway, Juno, GICv3 ITS h/w) and it seemed to work well - at least if one is nice and starts only one process accessing vgic-state (see below). I caught some IRQs red-handed and the output

Re: [PATCH] KVM: arm/arm64: vgic: Add debugfs vgic-state file

2017-01-20 Thread André Przywara
On 20/01/17 20:07, Christoffer Dall wrote Hi Christoffer, > On Fri, Jan 20, 2017 at 06:07:26PM +, Andre Przywara wrote: >> Hej Christoffer, >> >> On 20/01/17 10:33, Christoffer Dall wrote: >>> Add a file to debugfs to read the in-kernel state of the vgic. We don't >>> do any locking of the

Re: [PATCH] KVM: arm/arm64: vgic: Add debugfs vgic-state file

2017-01-20 Thread Christoffer Dall
Hi Andre, On Fri, Jan 20, 2017 at 06:07:26PM +, Andre Przywara wrote: > Hej Christoffer, > > On 20/01/17 10:33, Christoffer Dall wrote: > > Add a file to debugfs to read the in-kernel state of the vgic. We don't > > do any locking of the entire VGIC state while traversing all the IRQs, > >

Re: [PATCH] KVM: arm/arm64: vgic: Add debugfs vgic-state file

2017-01-20 Thread Andre Przywara
Hej Christoffer, On 20/01/17 10:33, Christoffer Dall wrote: > Add a file to debugfs to read the in-kernel state of the vgic. We don't > do any locking of the entire VGIC state while traversing all the IRQs, > so if the VM is running the user/developer may not see a quiesced state, > but should