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.

