>>> On 18.05.17 at 20:35, <sstabell...@kernel.org> 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 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 multiplication in conditional operators. "Hide" it, >> > at the risk of the next compiler version becoming smarter and >> > recognizing even that. (The hope is that added smartness then would >> > also better deal with legitimate cases like the one here.) The change >> > could have been done in access_ok(), but I think we better keep it at >> > the place the compiler is actually unhappy about. >> >> I am wondering if we should drop array_access_ok and access_ok as they are >> not >> used. >> >> Anyway, I am happy with both way: >> >> Reviewed-by: Julien Grall <julien.gr...@arm.com> > > Is this series for 4.9?
Yes, as mentioned in 0/3. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
