>>> On 05.08.16 at 15:50, <andrew.coop...@citrix.com> wrote: > --- a/xen/arch/x86/cpu/common.c > +++ b/xen/arch/x86/cpu/common.c > @@ -476,7 +476,7 @@ void detect_extended_topology(struct cpuinfo_x86 *c) > sub_index++; > } while ( LEAFB_SUBTYPE(ecx) != INVALID_TYPE ); > > - core_select_mask = (~(-1 << core_plus_mask_width)) >> ht_mask_width; > + core_select_mask = (~(-1u << core_plus_mask_width)) >> ht_mask_width;
-1u is kind of bogus; could I talk you into using ~0u instead? With that Reviewed-by: Jan Beulich <jbeul...@suse.com> Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
