[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [PATCH 5/5] cpufreq/amd-cppc: move pxfreq_mhz into driver data


  • To: Jan Beulich <jbeulich@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: "Penny, Zheng" <penny.zheng@xxxxxxx>
  • Date: Mon, 26 Jan 2026 07:24:03 +0000
  • Accept-language: en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=M864oLTCS+GwUIlCTlKALeCfrneBHjM+6rKe1v7Z7O4=; b=gpBJF7w0N0FNvXGR/S8X8efGPvfTGXXPjSiKntxImDqA37uJIG0cn4g7iSSw+cPU4r35iOo+v4e6keHlz+woPNOnQwuV8rsrSGhIUPABZQ5g6MaDTKOgauejUC+pOHuDw1J0ROIPQ9CnccidjktCMRqkoeB8qGM5CZDhPW3LQ7IRS7gyu8R1DcBZ2pOpXO1SgYDXr/qJvrd+SYtsup5u16FDVjmy+mcHoc5hPQ1DagG34XWK6GVHF8puK/sn/hjjIz+R7Tnw+Fq+ZrbDGFB+Nnd42LXxVDwirzL745D1lY4v+qVVgM4yRn4JPQUgNeCkJpcqh2+SsCDEzDlhFTcQOQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=D8FTgbiCsQ/ySrqUyMbzkN3ujd/klgx4lZbZycLBSvSLmK1GnrzsvJ87KXB0aqhVQgF1/aPvfmZrRernYbW6m6fq07nT8IAmoG8kkxVALsVphjeAqpTQdue3I4d/wM25/6XUAc8mdO3sLRoamUvXB3SEbQGXi4lqniQvKmEj4WgZLXJAXPiYgPm8/vj2oEMIHnkuraPg+QCgtlN1taH/SUWyQ1T0hzWVbZ2jwcEEnM1P3RokTGqvdoJRO8AcAX/Uo1lWvjELWwD+69gNMKy9p+G25h8I7QKtUsot1Y7gkYvvHRaEHBc9OgvssfmUudwn/G2uQY8/n0Pz/UVi6o0hMQ==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=amd.com;
  • Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Delivery-date: Mon, 26 Jan 2026 07:24:17 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Msip_labels: MSIP_Label_f265efc6-e181-49d6-80f4-fae95cf838a0_Enabled=True;MSIP_Label_f265efc6-e181-49d6-80f4-fae95cf838a0_SiteId=3dd8961f-e488-4e60-8e11-a82d994e183d;MSIP_Label_f265efc6-e181-49d6-80f4-fae95cf838a0_SetDate=2026-01-26T07:21:15.0000000Z;MSIP_Label_f265efc6-e181-49d6-80f4-fae95cf838a0_Name=Open Source;MSIP_Label_f265efc6-e181-49d6-80f4-fae95cf838a0_ContentBits=3;MSIP_Label_f265efc6-e181-49d6-80f4-fae95cf838a0_Method=Privileged
  • Thread-index: AQHci4Oip3RJ/Oi400aA3juf0wC21rVkEXpA
  • Thread-topic: [PATCH 5/5] cpufreq/amd-cppc: move pxfreq_mhz into driver data

[Public]

> -----Original Message-----
> From: Jan Beulich <jbeulich@xxxxxxxx>
> Sent: Thursday, January 22, 2026 5:44 PM
> To: xen-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>; Roger Pau Monné
> <roger.pau@xxxxxxxxxx>; Penny, Zheng <penny.zheng@xxxxxxx>
> Subject: [PATCH 5/5] cpufreq/amd-cppc: move pxfreq_mhz into driver data
>
> No reason for it to be a separate per-CPU item; it's connected to a struct
> cpufreq_policy instance just like other driver data.
>
> This also eliminates the concern over amd_cppc_cpufreq_cpu_init() being 
> called for
> all CPUs, or a CPU going offline that's recorded in policy->cpu (which would 
> result
> in accesses of per-CPU data of offline CPUs).
>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
>

Reviewed-by: Penny Zheng <penny.zheng@xxxxxxx>

Later, I'll rebase your commits and test them all on my board locally, and 
complement the Tested-by:. Since I guest upstream doesn't have boards with 
amd-cppc enabled in BIOS

Many thanks,
Penny Zheng

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.