Received-SPF: pass (google.com: domain of linux-kernel+bounces-178288-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: zhida312@outlook.com
To: rafael@kernel.org,
	viresh.kumar@linaro.org
Cc: Peng Ma <andypma@tencent.com>,
	Huang Rui <ray.huang@amd.com>,
	"Gautham R. Shenoy" <gautham.shenoy@amd.com>,
	Mario Limonciello <mario.limonciello@amd.com>,
	Perry Yuan <perry.yuan@amd.com>,
	linux-pm@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: [PATCH] cpufreq: amd-pstate: fix the memory to free after epp exist
Date: Tue, 14 May 2024 11:17:06 +0800
Message-ID:
 <PUZPR01MB51206E9B32294882EC34B09592E32@PUZPR01MB5120.apcprd01.prod.exchangelabs.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
X-MS-PublicTrafficType: Email
X-MS-TrafficTypeDiagnostic: PUZPR01MB5120:EE_|TY0PR0101MB4666:EE_
X-MS-Office365-Filtering-Correlation-Id: 1afdb210-4ceb-4e6b-22ec-08dc73c47681
X-Microsoft-Antispam:
	BCL:0;ARA:14566002|461199019|440099019|3420499023|3430499023|3412199016;
X-Microsoft-Antispam-Message-Info:
	H8Cmg+yx078EQJqmpsaabwivN3+ZwWpfDqUh3k98fqaw4a0btXgoBFKmeBNP34sqtU01t5zz0r9ppJsw4oRPUJbjJ8kbUGVdZ2DJ2vFuuG7Z2uYNmrjXG52VcNIxuGGTjFox4Q5XDbKRx/W47uHw/5gpnLmQwhZ9PdMBK+Wj5M+V5/pC3pANqcMATXuA6RPvzdY8DW8CW+GMDZ2OU7qJqminWopSnENqaNDSAI0WEaRMDKQTImEYeD63BmL7D6BVFLlJQiRmt8meHIxoDG8Lm3Ucam0b6ItlJuixeHQVPFX8SZGKsux1OLXrW6X5M0X8fWI6Iu9BjWeKCbNI/G5r6baFnf5WWgTPEu6+lo0mwoM2nyIcuYtH54dTsU3NSORk6HdMhYVEw8ZDs96iproRnTtDqgprytJmchMrbzAwp+unoCgNibXhD9ZIq2cT1mtya7JdIW+V/ICu3GBDCmqgvPQkA5TVwQT7m4VDOwllRAF7FUmNfiAA8tePZr+puC+SUp1sCgL0H8BaGbiN3WjFIFY8xf8Oz2z2VSHHpk7evQkvgpzRT8L9RazihzJP/uCV
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?ZTRstFSH9s+WcekZjMDdAD+SWfVk16Outyuowon9rF0vTZOm/fpyCosokyy0?=
 =?us-ascii?Q?3N8wO1kMPjVaMij8kyOD3uKDSAIe5pQoozWXY0zeSLpYY3MtgyRm90iETi8f?=
 =?us-ascii?Q?+PuYGsmc5ysIaZx/fvRrsOaBSza6504L37FT7FbzuveZeJk9I0GcRtnTM9FD?=
 =?us-ascii?Q?sYLks+EZKJ4Bxb1krBOlZitVSbUJs2fVz91NO5vG7vS1FoIaFtSVf8EeTD6w?=
 =?us-ascii?Q?PSqMaVsdH9MhGZlAkbHcNtze40qMCeJ8cS61KjuZcTwLKudP2kAl50mpkGHx?=
 =?us-ascii?Q?M9fJgIvTWDbEAcBldvUb+UcejlCqtFIwDTqdr0vq4ufTslIYHw98YQigNg0S?=
 =?us-ascii?Q?sy5OUmhRp/4PkvAeCn2R1oLNKVk38dwXobpszdtYraMtyivNIArUjg2ROELZ?=
 =?us-ascii?Q?04DNJHX2ZbxhiULlbo6GNj8GoNZYC13+jY4DmoBeISiiqX9R7t1C8c5Bhk6t?=
 =?us-ascii?Q?e+uS8tLTBuX3lO0NpQ54eaFczVMjdwfZ2MjIMeboI6HwVRGMMbqBO00p0PPX?=
 =?us-ascii?Q?MefWXGjPE5mcC6i2nXiXfT1wLaUyvkd5InGPx6Z8Lb6P+oUChEXzMQesb3wn?=
 =?us-ascii?Q?DMW6R1Unc8+v7dxBJxqKdjq5AYsyOjZebmu32SS2VjCJIXR/yRSRIgkG2DpK?=
 =?us-ascii?Q?q3OrIgcda1BjdCiXg8FL3DYt5mDN2N1q+76HizpzUKq8LvbP63jGcJYmu0rf?=
 =?us-ascii?Q?qH0C5ePVU+K8gAlbmJ8zN670mRCHLw01p2418C7cLxlcpRj3JVVYH3YIzcxt?=
 =?us-ascii?Q?lHvAwCgKWYnAc8CF5B2QF0KFc/CgLed6cjxGOGOJ1DuP5KIlzWkn0iTsCf5J?=
 =?us-ascii?Q?MdML8cyJDEi+B9l2/tvbDEDXNw7c4xbfotKEyKUcC7k9HBsMoQ1oluKOfK3M?=
 =?us-ascii?Q?KSocAFHDr5HaGV094gUcfLkaeCC+ihKuOpmOyuMqLTnVN2NiwuNFrQSzWRUd?=
 =?us-ascii?Q?Z26a0ZAJfUs2vEdKlm7DwlhLhVTSCLrcgRwUoli0I+E2gBgEhq3SGjCQA2g1?=
 =?us-ascii?Q?ogvseMfqX7OOiq7jCG5hOGJnUulq0NBjjYThiB2lGG3L/jXz1pPbXa/5WaiU?=
 =?us-ascii?Q?cwIWnWQO8STiRlFfalVUh9/WXREhF+mCPexjsmLBggfLzB3GtrGo7gZotsqk?=
 =?us-ascii?Q?1e1dBa694Nv02NNZaRJX4qMJOwXj+tLRhPyewmBh0gah4dMp+Xfq+11f+ZcO?=
 =?us-ascii?Q?u8fKFCfIoNSon5LWCluZYNU9Yq81Zo23e/y1WD1OFlYd0PM3MlIcbFfp5iI?=
 =?us-ascii?Q?=3D?=
X-OriginatorOrg: outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1afdb210-4ceb-4e6b-22ec-08dc73c47681
X-MS-Exchange-CrossTenant-AuthSource: PUZPR01MB5120.apcprd01.prod.exchangelabs.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 May 2024 03:18:02.5917
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg:
	00000000-0000-0000-0000-000000000000
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TY0PR0101MB4666

From: andypma <andypma.tencent.com>

the cpudata memory from kzmalloc in epp init function is
not free after epp exist, so we should free it.

Signed-off-by: Peng Ma <andypma@tencent.com>
---
 drivers/cpufreq/amd-pstate.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/cpufreq/amd-pstate.c b/drivers/cpufreq/amd-pstate.c
index 2db095867d03..96a323f59a93 100644
--- a/drivers/cpufreq/amd-pstate.c
+++ b/drivers/cpufreq/amd-pstate.c
@@ -1425,6 +1425,9 @@ static int amd_pstate_epp_cpu_init(struct cpufreq_policy *policy)
 
 static int amd_pstate_epp_cpu_exit(struct cpufreq_policy *policy)
 {
+	struct amd_cpudata *cpudata = policy->driver_data;
+
+	kfree(cpudata);
 	pr_debug("CPU %d exiting\n", policy->cpu);
 	return 0;
 }
-- 
2.33.0