Commit 16c977f8 authored by Dhananjay Ugwekar's avatar Dhananjay Ugwekar Committed by Mario Limonciello
Browse files

cpufreq/amd-pstate: Convert the amd_pstate_get/set_epp() to static calls 



MSR and shared memory based systems have different mechanisms to get and
set the epp value. Split those mechanisms into different functions and
assign them appropriately to the static calls at boot time. This eliminates
the need for the "if(cpu_feature_enabled(X86_FEATURE_CPPC))" checks at
runtime.

Also, propagate the error code from rdmsrl_on_cpu() and cppc_get_epp_perf()
to *_get_epp()'s caller, instead of returning -EIO unconditionally.

Signed-off-by: default avatarDhananjay Ugwekar <Dhananjay.Ugwekar@amd.com>
Reviewed-by: default avatarMario Limonciello <mario.limonciello@amd.com>
Reviewed-by: default avatarGautham R. Shenoy <gautham.shenoy@amd.com>
Link: https://lore.kernel.org/r/20241204144842.164178-2-Dhananjay.Ugwekar@amd.com


Signed-off-by: default avatarMario Limonciello <mario.limonciello@amd.com>
parent 2993b29b

drivers/cpufreq/amd-pstate.c

+60 −32
Original line number Diff line number Diff line
@@ -180,25 +180,39 @@ static inline int get_mode_idx_from_str(const char *str, size_t size) 
static DEFINE_MUTEX(amd_pstate_limits_lock);

static DEFINE_MUTEX(amd_pstate_driver_lock);



static s16 amd_pstate_get_epp(struct amd_cpudata *cpudata, u64 cppc_req_cached)

static s16 msr_get_epp(struct amd_cpudata *cpudata, u64 cppc_req_cached)

{

	u64 epp;

	int ret;



	if (cpu_feature_enabled(X86_FEATURE_CPPC)) {

	if (!cppc_req_cached) {

			epp = rdmsrl_on_cpu(cpudata->cpu, MSR_AMD_CPPC_REQ,

					&cppc_req_cached);

			if (epp)

				return epp;

		ret = rdmsrl_on_cpu(cpudata->cpu, MSR_AMD_CPPC_REQ, &cppc_req_cached);

		if (ret < 0) {

			pr_debug("Could not retrieve energy perf value (%d)\n", ret);

			return ret;

		}

	}

	epp = (cppc_req_cached >> 24) & 0xFF;

	} else {



	return (s16)epp;

}



DEFINE_STATIC_CALL(amd_pstate_get_epp, msr_get_epp);



static inline s16 amd_pstate_get_epp(struct amd_cpudata *cpudata, u64 cppc_req_cached)

{

	return static_call(amd_pstate_get_epp)(cpudata, cppc_req_cached);

}



static s16 shmem_get_epp(struct amd_cpudata *cpudata, u64 dummy)

{

	u64 epp;

	int ret;



	ret = cppc_get_epp_perf(cpudata->cpu, &epp);

	if (ret < 0) {

		pr_debug("Could not retrieve energy perf value (%d)\n", ret);

			return -EIO;

		}

		return ret;

	}



	return (s16)(epp & 0xff);
@@ -253,12 +267,10 @@ static inline void amd_pstate_update_perf(struct amd_cpudata *cpudata, 
					    max_perf, fast_switch);

}



static int amd_pstate_set_epp(struct amd_cpudata *cpudata, u32 epp)

static int msr_set_epp(struct amd_cpudata *cpudata, u32 epp)

{

	int ret;

	struct cppc_perf_ctrls perf_ctrls;



	if (cpu_feature_enabled(X86_FEATURE_CPPC)) {

	u64 value = READ_ONCE(cpudata->cppc_req_cached);



	value &= ~GENMASK_ULL(31, 24);
@@ -268,7 +280,22 @@ static int amd_pstate_set_epp(struct amd_cpudata *cpudata, u32 epp) 
	ret = wrmsrl_on_cpu(cpudata->cpu, MSR_AMD_CPPC_REQ, value);

	if (!ret)

		cpudata->epp_cached = epp;

	} else {



	return ret;

}



DEFINE_STATIC_CALL(amd_pstate_set_epp, msr_set_epp);



static inline int amd_pstate_set_epp(struct amd_cpudata *cpudata, u32 epp)

{

	return static_call(amd_pstate_set_epp)(cpudata, epp);

}



static int shmem_set_epp(struct amd_cpudata *cpudata, u32 epp)

{

	int ret;

	struct cppc_perf_ctrls perf_ctrls;



	amd_pstate_update_perf(cpudata, cpudata->min_limit_perf, 0U,

				     cpudata->max_limit_perf, false);
@@ -279,7 +306,6 @@ static int amd_pstate_set_epp(struct amd_cpudata *cpudata, u32 epp) 
		return ret;

	}

	cpudata->epp_cached = epp;

	}



	return ret;

}
@@ -1869,6 +1895,8 @@ static int __init amd_pstate_init(void) 
		static_call_update(amd_pstate_cppc_enable, shmem_cppc_enable);

		static_call_update(amd_pstate_init_perf, shmem_init_perf);

		static_call_update(amd_pstate_update_perf, shmem_update_perf);

		static_call_update(amd_pstate_get_epp, shmem_get_epp);

		static_call_update(amd_pstate_set_epp, shmem_set_epp);

	}



	if (amd_pstate_prefcore) {