On 07/17/2018 04:20 AM, Kirill A. Shutemov wrote: > + } else { > + /* > + * Reset __PHYSICAL_MASK. > + * Maybe needed if there's inconsistent configuation > + * between CPUs. > + */ > + physical_mask = (1ULL << __PHYSICAL_MASK_SHIFT) - 1; > + } This seems like an appropriate place for a WARN_ON(). Either that, or axe this code.