Re: [PATCH v2 2/2] platform/x86/amd/hfi: Rank SMT siblings below primary cores for ITMT packing

[K Prateek Nayak <kprateek.nayak@amd.com>]

Hello Aiden,

On 11/13/2026 7:59 AM, Aiden Hecker wrote:
> Tested-by: Aiden Hecker <heckeraiden@gmail.com>

We assume you've tested the changes you are signing off on :-)

> Signed-off-by: Aiden Hecker <heckeraiden@gmail.com>

[..snip..]

> +	/*
> +	 * Rank secondary SMT siblings slightly below primary threads so
> +	 * SD_ASYM_PACKING fills idle physical cores first while still keeping
> +	 * classic Zen 5 SMT siblings ordered ahead of dense Zen 5c cores.
> +	 */
> +	prio = hfi_cpuinfo->ipcc_scores[0];
> +	if (cpu_feature_enabled(X86_FEATURE_AMD_HTR_CORES)) {
> +		sib_mask = topology_sibling_cpumask(cpu);
> +		if (!cpumask_empty(sib_mask) && cpumask_first(sib_mask) != cpu)
> +			prio = (prio * 7) / 8;
> +	}
> +
> +	sched_set_itmt_core_prio(prio, cpu);

If asym capacity is enabled, we should not use the IPCC information.
The scheduler is smart enough to go find an idle core if it exists
that fits the task capacity.

>  
>  	return 0;
>  }

-- 
Thanks and Regards,
Prateek