Re: [PATCH v7 04/12] platform/x86: hfi: Introduce AMD Hardware Feedback Interface Driver

[Peter Zijlstra]

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.