[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH v7 04/13] xen/cpufreq: add missing default: case for x86 vendor
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: "Penny, Zheng" <penny.zheng@xxxxxxx>
- Date: Tue, 26 Aug 2025 04:23:43 +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=CeEMw0WAP8DEf+kBqsIYruecdXTSMqgZNOFHHH3nA9o=; b=bWpMftFlcQx65kBHDcNYmndlcDHKjo/1PpAfky7P+qQ6uhhoLTItAFvnUk4vZ6Sx9ECWCXJ7mHF1JiITihawsajjCmkMYoTXIerev8lOC0m2+pCkEaiaWPLrBzpz2GBwF4hnLuHFfoHWSCYFY3VZY8EmQAlrygpQT35HPg4L7wTUF8T/bb/SKX+F8q3u1ZoI6MATKo19LWmVWu4eItq2gwqpoKaV3b+bSWLjlbFmVqeskq9IET2649XF7+uXjMU9ALgKvi6jFROtlrnTS0aCH4HaTUplsjZUbOT4nppnJPSSVSKsir79kiyvUx5fMnBwHLszLpzenhnMs8eWl5qRig==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=fC5AZNO5gzaAxzVEbHy6ToUMO2gqJGBmTBVb/0EsP14Fa8PWBpXPvo+xgaVgvbaHTlQUXTU7gQFwMuDQSnfShwYpM/blY8cujIzG8dXvRd5PRwWNPFQOLlDqC3niNeX3rso9AAOLBpktGD6HGLmXKFKM5kp/vM3TKvnJ+1UB8c6Ju2/4ESgQtspyYLgYPuGAiEboL+/uDEa92aRqT9hviUsgYibps5o4kGDfZhi7akH+L7x9mq9xl1ihV/+IJofQdoTV/gh6C0O4kEY8g7dq8z0eyOOFsIpawun98S+9UrZE7bUGuB3rolp7wjvuWD12Rf/JWPg3M3BfHN4yj6njDA==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=amd.com;
- Cc: "Huang, Ray" <Ray.Huang@xxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Tue, 26 Aug 2025 04:24:03 +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=2025-08-26T04:23:30.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: AQHcE1LnYk6aQ+tCsUqjAHtQXx8uFLRzdoWAgADk8hA=
- Thread-topic: [PATCH v7 04/13] xen/cpufreq: add missing default: case for x86 vendor
[Public]
> -----Original Message-----
> From: Jan Beulich <jbeulich@xxxxxxxx>
> Sent: Monday, August 25, 2025 10:43 PM
> To: Penny, Zheng <penny.zheng@xxxxxxx>
> Cc: Huang, Ray <Ray.Huang@xxxxxxx>; Andrew Cooper
> <andrew.cooper3@xxxxxxxxxx>; Roger Pau Monné <roger.pau@xxxxxxxxxx>; xen-
> devel@xxxxxxxxxxxxxxxxxxxx
> Subject: Re: [PATCH v7 04/13] xen/cpufreq: add missing default: case for x86
> vendor
>
> On 22.08.2025 12:52, Penny Zheng wrote:
> > Since we are missing default case for x86 vendor, there is possibility (i.e.
> > new vendor introduced) that we will return successfully while missing
> > the whole cpufreq driver initialization process.
> > Move "ret = -ENOENTRY" forward to cover default case for x86 vendor,
Typo: -ENOENT
> > and add error log
>
> Requested-by: Jan Beulich <jbeulich@xxxxxxxx> (or Suggested-by: if you like
> that
> better)
Ack
>
> > Signed-off-by: Penny Zheng <Penny.Zheng@xxxxxxx>
>
> Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
Thx
>
> Jan
|
