On Tue, Dec 03, 2024 at 02:27:46PM -0600, Mario Limonciello wrote: > I think the *_present() calls make sense for the other consumers in > suspend/resume though. Agree? * cpu_possible_mask- has bit 'cpu' set iff cpu is populatable * cpu_present_mask - has bit 'cpu' set iff cpu is populated * cpu_enabled_mask - has bit 'cpu' set iff cpu can be brought online * cpu_online_mask - has bit 'cpu' set iff cpu available to scheduler * cpu_active_mask - has bit 'cpu' set iff cpu available to migration The distinction between possible and present is physical hotplug. Using present means you care about that distinction. Using that for suspend/resume strikes me as equally odd.
