Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <6973890d-3366-0fdb-6306-77a763a979d5@amd.com>
Date:   Sun, 18 Jun 2023 21:53:21 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.9.1
Subject: Re: [PATCH v2 4/4] cpufreq: intel_pstate: Use the
 acpi_pm_profile_server() symbol
Content-Language: en-US
To:     "Rafael J. Wysocki" <rafael@kernel.org>,
        Perry Yuan <perry.yuan@amd.com>
Cc:     rafael.j.wysocki@intel.com, viresh.kumar@linaro.org,
        Ray.Huang@amd.com, Deepak.Sharma@amd.com, Wyes.Karny@amd.com,
        gautham.shenoy@amd.com, Sunpeng.Li@amd.com, Xinmei.Huang@amd.com,
        Xiaojian.Du@amd.com, Li.Meng@amd.com, linux-pm@vger.kernel.org,
        linux-kernel@vger.kernel.org
References: <20230615063333.4030395-1-perry.yuan@amd.com>
 <CAJZ5v0hMEuiv71RoEfyBJ_Z83AvMWXfEvt9kBU2SmZyXvKSx4w@mail.gmail.com>
From:   Mario Limonciello <mario.limonciello@amd.com>
In-Reply-To: <CAJZ5v0hMEuiv71RoEfyBJ_Z83AvMWXfEvt9kBU2SmZyXvKSx4w@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ZjZ1RDVFSk9DM1pEQXV1MnYzZk4xeGxnbFB6cWNzNndWdGwreGl3WjIzQWoz?=
 =?utf-8?B?UGJMNjBzdG5CcHRoOC9XcmdEdFN3bFQrLzlqR1lkQkZ1eWIycmlHT0svN0E2?=
 =?utf-8?B?U3Z1eGQrZEI4RlJZdXdndm5PZ3RRUzJUcFZLd0d2Q3JXOFdNaUI5UnZmcVpD?=
 =?utf-8?B?ZW8rNzJ1eitjdlBIYVJoWjQ5QkN0aDlkRzdhY0JWWkE1WEFXNS8ycmtBVHl1?=
 =?utf-8?B?OWpDOEZSa2k3MXFLUkRuNjRSZzdFLzVwd1MySGpzTHdvRzNLc0lUWC94Ymh5?=
 =?utf-8?B?U3piN092cVVkZStieUJZZnk4eFpvMDE3YXpiU0o4ZWNRdEE1dkNXdmxKazRL?=
 =?utf-8?B?VXRDWmo2b0Q5eEFJbjQzZCsvc0ZjZWxSV2QzNkF0NTIxMys3UnJtaGxHc2Nh?=
 =?utf-8?B?R2dOd0pveDd3S21Ea2VUaTYvbHZqZVBQM1RUVlB5WmR4eXV4NkxjNmRiSGJY?=
 =?utf-8?B?Z1dNaW1wcTZxcGdiU2JDYXNOVmpsTGtydHJwaG5mTHJmblRYd01Wb3RMOGwr?=
 =?utf-8?B?QXo4SWs5N1JLOXBWYjNaZWh2QjZuK29TMXNvRUEvNVBPOFhCbEphVGVhTGc0?=
 =?utf-8?B?bVREdmtXZUpRKzg1Y2ZBU29FeFc4OE1vNE1MMXNpK3RBM0dobnVYemNEK3px?=
 =?utf-8?B?cmFlUGRJUWhtNlI3aHNUNnZFQ2RzMXQzdjVRVldyb3lUM2h2d3lBdkNHeC9H?=
 =?utf-8?B?b1RLZEJydHZOU0NXaDZXbktiUUZDMUlqVUR0N3dBSngvWm9LczROa28zWlc5?=
 =?utf-8?B?ZDZYeWx1T3FEL1BxNzVBdlZZREVkYU1XZGFWaytmS0VteTBYaEVvNnREamo0?=
 =?utf-8?B?SUdzdVJGQURTK1NzeTNiV1E4ci80eHE2N2V0ZTYyZzBzVXNtTDB6SWdrODNk?=
 =?utf-8?B?NmRSOEhUcExSZWdXcWdKYTlmZm9KK1FpMXVIYWI4SGhvc2lmSjMzTFBXZUZX?=
 =?utf-8?B?WnhUMVdpeHBPUlJuUFVUSGlSQlc3MVJIUzA5YmNqWkU5SXlSd0ZwRDZpT3h0?=
 =?utf-8?B?WWE3T3M1UkJFZGRseS9nQzVBTkZQT1hkZDdVa083ZFljRmwrTmtGcFZGYmx6?=
 =?utf-8?B?cThxSS9aSTBVeVBIZi9mOUZhUnZqeFVyOXhYaVJlVU40RXhZQ1VVQWVrTTZv?=
 =?utf-8?B?YzVCR0Z0NSt1WnpYT1hPaVJvYktyQk9OYlg0OUYwSXJnMCt0N3NkdVoyUHcv?=
 =?utf-8?B?N0VEVm0yNlpBelE2bGJjVStRSVE0MElhRGtseS9qOVdxMXZsR21VNURZTHdX?=
 =?utf-8?B?MEVKaWQ4NHVYa0FnYUpMdko1MlJGV0Q0YnFnRm5hSVNUZS9MTnRmRWlKTWov?=
 =?utf-8?B?TVp5MzFKa043NWhMbjcwRFNHNFVRM3RqaXlFRzBWOTZmQWszZ3l2dVU3NVNL?=
 =?utf-8?B?bis1VEVMdHJFU1BBdWxxWkRMTkNwTE1IcWt6UjhSKzRPUWZzS3VaSWU3Y28z?=
 =?utf-8?B?c0FBMXYvTEV6UC9IQXVEOUtPSWRuMWRGWmFHZUR5enk2WnBSdDU0Y1hMOEd6?=
 =?utf-8?B?VjdoS2VwSi8vNlNlR2t5azV6RVlSYTlLM2k4U3NEdHZWVHFlVXo1WStqWTZi?=
 =?utf-8?B?VHl6S3VWVjY3M3FVMmV4UUFYU2YxbnNSMjNCSEM4ZzY1REFBNFZXUEJ2UjVv?=
 =?utf-8?B?czhsWDJ0OFNTazRXS0RXMkl5aSs4K2dKSkZxQXdIb003M1FMNDUzYVJoK05M?=
 =?utf-8?B?alc5UmFUSEpocVNHRnpFZEE2QkZwbE00MEZ6MFBpTU5ESnBtYjMxaExENXpa?=
 =?utf-8?B?RzZ5VEJpK3U0V2k2d0s0MDIzdjBSbHE5MVB2aXFDc0tRT1dCbnBzTmt6N3A2?=
 =?utf-8?B?cWo1WkYybUxidXhaTFJHdWlMRURXMVJ6Q1FtZTNRVlg1UUZyU3NCNXdFeG1Z?=
 =?utf-8?B?K3Y2SFVVd1paVHY1bXFDY1lnQTZuNHgvUFgrOS9XcGNMdE94d1pMeEY4bWhx?=
 =?utf-8?B?TmR5NWh1bnpiWWZwZ0xZaVJ4UitpWmd1MEFNRExRVEw2UjV5Zk5EVEI2Nisz?=
 =?utf-8?B?YW9RQStYaWk0cVhXcmxiNzdFZlBMbUM0NDUxaUduKzRqUFFUK1Z2Y3BSQTF4?=
 =?utf-8?B?QVR6cytPWFZGb3djWW5JQ2krS1RYaVRVVzBSS09obE5MaVpqYWo3anlKaHd3?=
 =?utf-8?B?QnFnMy9DQTRuck56U3lFRDFFM1hVRXlJbmYwVENZU2FRWTFzK2RrL1Y1WXAy?=
 =?utf-8?B?MGc9PQ==?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e1b015e3-9b0a-4bd6-6714-08db70705905
X-MS-Exchange-CrossTenant-AuthSource: MN0PR12MB6101.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Jun 2023 02:53:23.6899
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: ScmRagR17jJJyGXEhsseFfE0mGKG8W+tleky7sRjNYOSIQ8WLDSxP28tLau6S5h05zsBRauRKJsmq0pVeTFjcQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ1PR12MB6266
X-Spam-Status: No, score=-0.5 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,NICE_REPLY_A,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 6/15/23 10:57, Rafael J. Wysocki wrote:
> On Thu, Jun 15, 2023 at 8:35 AM Perry Yuan <perry.yuan@amd.com> wrote:
>>
>> From: Mario Limonciello <mario.limonciello@amd.com>
>>
>> Avoid duplication of functionality by using the generic symbol.
>> This does have a functional change that intel-pstate will now
>> match "SOHO server" as well.
> 
> And why do you think that it is a good idea to change this behavior?

The idea was to cover all possible server types. It seemed that it could 
be an oversight that it wasn't included initially.

Is that not the case and it is "intentionally" not including "SOHO server"?

> 
>> Reviewed-by: Gautham R. Shenoy <gautham.shenoy@amd.com>
>> Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
>> Signed-off-by: Perry Yuan <Perry.Yuan@amd.com>
>> ---
>>   drivers/cpufreq/intel_pstate.c | 19 ++++---------------
>>   1 file changed, 4 insertions(+), 15 deletions(-)
>>
>> diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
>> index 2548ec92faa2..6401338971c7 100644
>> --- a/drivers/cpufreq/intel_pstate.c
>> +++ b/drivers/cpufreq/intel_pstate.c
>> @@ -313,18 +313,9 @@ static DEFINE_MUTEX(intel_pstate_limits_lock);
>>
>>   #ifdef CONFIG_ACPI
>>
>> -static bool intel_pstate_acpi_pm_profile_server(void)
>> -{
>> -       if (acpi_gbl_FADT.preferred_profile == PM_ENTERPRISE_SERVER ||
>> -           acpi_gbl_FADT.preferred_profile == PM_PERFORMANCE_SERVER)
>> -               return true;
>> -
>> -       return false;
>> -}
>> -
>>   static bool intel_pstate_get_ppc_enable_status(void)
>>   {
>> -       if (intel_pstate_acpi_pm_profile_server())
>> +       if (acpi_pm_profile_server())
>>                  return true;
>>
>>          return acpi_ppc;
>> @@ -481,10 +472,6 @@ static inline void intel_pstate_exit_perf_limits(struct cpufreq_policy *policy)
>>   {
>>   }
>>
>> -static inline bool intel_pstate_acpi_pm_profile_server(void)
>> -{
>> -       return false;
>> -}
>>   #endif /* CONFIG_ACPI */
>>
>>   #ifndef CONFIG_ACPI_CPPC_LIB
>> @@ -2407,8 +2394,10 @@ static int intel_pstate_init_cpu(unsigned int cpunum)
>>                  if (hwp_active) {
>>                          intel_pstate_hwp_enable(cpu);
>>
>> -                       if (intel_pstate_acpi_pm_profile_server())
>> +#ifdef CONFIG_ACPI
>> +                       if (acpi_pm_profile_server())
>>                                  hwp_boost = true;
>> +#endif
>>                  }
>>          } else if (hwp_active) {
>>                  /*
>> --
>> 2.34.1
>>