Re: [Xen-devel] [PATCH for 4.5] xen/arm: Initialize the domain vgic lock

2015-01-06 Thread Ian Campbell
On Fri, 2014-12-19 at 09:23 -0500, Konrad Rzeszutek Wilk wrote: > On Thu, Dec 18, 2014 at 09:47:37AM +, Ian Campbell wrote: > > On Wed, 2014-12-17 at 15:40 +, Julien Grall wrote: > > > The domain vgic lock is used uninitialized. > > > > > > Signed-off-by: Julien Grall > > > > Acked-by: I

Re: [Xen-devel] [PATCH for 4.5] xen/arm: Initialize the domain vgic lock

2014-12-19 Thread Konrad Rzeszutek Wilk
On Thu, Dec 18, 2014 at 09:47:37AM +, Ian Campbell wrote: > On Wed, 2014-12-17 at 15:40 +, Julien Grall wrote: > > The domain vgic lock is used uninitialized. > > > > Signed-off-by: Julien Grall > > Acked-by: Ian Campbell Release-Acked-by: Konrad Rzeszutek Wilk > > > --- > > Thi

Re: [Xen-devel] [PATCH for 4.5] xen/arm: Initialize the domain vgic lock

2014-12-18 Thread Julien Grall
On 18/12/2014 12:12, Ian Campbell wrote: On Thu, 2014-12-18 at 12:05 +, Julien Grall wrote: Hi Ian, On 18/12/2014 09:47, Ian Campbell wrote: On Wed, 2014-12-17 at 15:40 +, Julien Grall wrote: The domain vgic lock is used uninitialized. Signed-off-by: Julien Grall Acked-by: Ian C

Re: [Xen-devel] [PATCH for 4.5] xen/arm: Initialize the domain vgic lock

2014-12-18 Thread Ian Campbell
On Thu, 2014-12-18 at 12:05 +, Julien Grall wrote: > Hi Ian, > > On 18/12/2014 09:47, Ian Campbell wrote: > > On Wed, 2014-12-17 at 15:40 +, Julien Grall wrote: > >> The domain vgic lock is used uninitialized. > >> > >> Signed-off-by: Julien Grall > > > > Acked-by: Ian Campbell > > > >>

Re: [Xen-devel] [PATCH for 4.5] xen/arm: Initialize the domain vgic lock

2014-12-18 Thread Julien Grall
Hi Ian, On 18/12/2014 09:47, Ian Campbell wrote: On Wed, 2014-12-17 at 15:40 +, Julien Grall wrote: The domain vgic lock is used uninitialized. Signed-off-by: Julien Grall Acked-by: Ian Campbell --- This is a bug fix for Xen 4.5 and Xen 4.4. The vgic lock is used unitialize

Re: [Xen-devel] [PATCH for 4.5] xen/arm: Initialize the domain vgic lock

2014-12-18 Thread Ian Campbell
On Wed, 2014-12-17 at 15:40 +, Julien Grall wrote: > The domain vgic lock is used uninitialized. > > Signed-off-by: Julien Grall Acked-by: Ian Campbell > --- > This is a bug fix for Xen 4.5 and Xen 4.4. The vgic lock is used > unitialized. Luckily we only use the field "raw" which

[Xen-devel] [PATCH for 4.5] xen/arm: Initialize the domain vgic lock

2014-12-17 Thread Julien Grall
The domain vgic lock is used uninitialized. Signed-off-by: Julien Grall --- This is a bug fix for Xen 4.5 and Xen 4.4. The vgic lock is used unitialized. Luckily we only use the field "raw" which is reset to 0 during the domain allocation. There is no harm to apply for Xen 4.5 b