Re: [Xen-devel] [PATCH 3/3] arm: fix build with gcc 7

2017-05-19 Thread Julien Grall
Hi Jan, On 19/05/2017 07:16, Jan Beulich wrote: On 18.05.17 at 17:12, wrote: On 18/05/17 09:41, Jan Beulich wrote: The compiler dislikes duplicat "const", and the ones it complains about s/duplicat/duplicate/ look like they we in fact meant to be placed differently. Also fix array_access

Re: [Xen-devel] [PATCH 3/3] arm: fix build with gcc 7

2017-05-18 Thread Jan Beulich
>>> On 18.05.17 at 20:35, wrote: > On Thu, 18 May 2017, Julien Grall wrote: >> Hi, >> >> On 18/05/17 09:41, Jan Beulich wrote: >> > The compiler dislikes duplicat "const", and the ones it complains about >> >> s/duplicat/duplicate/ >> >> > look like they we in fact meant to be placed differentl

Re: [Xen-devel] [PATCH 3/3] arm: fix build with gcc 7

2017-05-18 Thread Jan Beulich
>>> On 18.05.17 at 17:12, wrote: > On 18/05/17 09:41, Jan Beulich wrote: >> The compiler dislikes duplicat "const", and the ones it complains about > > s/duplicat/duplicate/ > >> look like they we in fact meant to be placed differently. >> >> Also fix array_access_okay() (just like on x86), desp

Re: [Xen-devel] [PATCH 3/3] arm: fix build with gcc 7

2017-05-18 Thread Stefano Stabellini
On Thu, 18 May 2017, Julien Grall wrote: > Hi, > > On 18/05/17 09:41, Jan Beulich wrote: > > The compiler dislikes duplicat "const", and the ones it complains about > > s/duplicat/duplicate/ > > > look like they we in fact meant to be placed differently. > > > > Also fix array_access_okay() (ju

Re: [Xen-devel] [PATCH 3/3] arm: fix build with gcc 7

2017-05-18 Thread Julien Grall
Hi, On 18/05/17 09:41, Jan Beulich wrote: The compiler dislikes duplicat "const", and the ones it complains about s/duplicat/duplicate/ look like they we in fact meant to be placed differently. Also fix array_access_okay() (just like on x86), despite the construct being unused on ARM: -Wint

[Xen-devel] [PATCH 3/3] arm: fix build with gcc 7

2017-05-18 Thread Jan Beulich
The compiler dislikes duplicat "const", and the ones it complains about look like they we in fact meant to be placed differently. Also fix array_access_okay() (just like on x86), despite the construct being unused on ARM: -Wint-in-bool-context, enabled by default in gcc 7, doesn't like multiplicat