> +#ifdef CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID > +#define early_pfn_to_nid(pfn) pa_to_nid(((unsigned long)pfn) << PAGE_SHIFT) > +#endif Is there a reason we didn't just use pfn_to_nid() directly here instead of pa_to_nid()? I'm just thinking of having DISCONTIG/NUMA off and pfn_to_nid() being #defined to zero for those cases.