[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] Ping: [PATCH] x86: re-add stack alignment check
On Fri, Nov 25, 2016 at 10:34:30AM +0000, Andrew Cooper wrote: > On 22/11/16 10:14, Jan Beulich wrote: > >>>> On 15.11.16 at 11:46, <JBeulich@xxxxxxxx> wrote: > >>>>> On 15.11.16 at 11:26, <andrew.cooper3@xxxxxxxxxx> wrote: > >>> Everything will indeed break if it is off by 8, but everything will also > >>> be similarly-broken if it is off by 16 or off by -8. > >> Off by -8 will be caught as well. And there's no possible off-by-16, > >> as then the calculations will be right again (read: you can freely > >> add two longs to struct cpu_info, but you mustn't add just one). > > This discussion appears to have stalled, and so far I've seen no > > reason to change other than the position of the checks. I'm > > hesitant to send out v2 though without having reached agreement > > on the other aspects. > > Fine - lets get this fix back in for 4.8, then argue over how it should > be improved. > > Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> Release-acked-by: Wei Liu <wei.liu2@xxxxxxxxxx> _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |