Re: [PATCH v3] sched/topology: Enable topology_span_sane check only for debug builds

Naman Jain
2025-02-10T09:53:24Z

On 2/5/2025 12:53 PM, Naman Jain wrote:
>
>
> On 2/5/2025 12:50 PM, K Prateek Nayak wrote:
>> Hello Naman,
>>
>> On 2/3/2025 5:17 PM, Naman Jain wrote:
>>> From: Saurabh Sengar <ssengar@linux.microsoft.com>
>>>
>>> On a x86 system under test with 1780 CPUs, topology_span_sane() takes
>
> <.>
>
>>>   {
>>>       int i = cpu + 1;
>>> +    /* Skip the topology sanity check for non-debug, as it is a
>>> time- consuming operatin */
>>
>> s/operatin/operation/
>>
>>> +    if (!sched_debug()) {
>>> +        pr_info_once("%s: Skipping topology span sanity check. Use
>>> `sched_verbose` boot parameter to enable it.\n",
>>
>> This could be broken down as follows:
>>
>>          pr_info_once("%s: Skipping topology span sanity check."
>>                   " Use `sched_verbose` boot parameter to enable it.\n",
>>                   __func__);
>>
>> Running:
>>
>>      grep -r -A 5 "pr_info(.*[^;,]$" kernel/
>>
>> gives similar usage across kernel/*. Apart from those nits, feel
>> free to add:
>>
>> Tested-by: K Prateek Nayak <kprateek.nayak@amd.com> # x86
>>
>> if the future version does not change much.
>>
>
> Hello Prateek,
> Thanks for reviewing and testing this. I'll make changes based on your
> feedback in next version.
>
> Regards,
> Naman
>

Hi Prateek,
After breaking down the print msg based on your suggestion, checkpatch
gives a warning. There are no warnings reported with current version of
change. Even the fix suggested by checkpatch is aligned to what we have
right now. So I'll keep it like this, not push further changes as of now
and wait for the maintainers to pick the patch.

WARNING: quoted string split across lines
#57: FILE: kernel/sched/topology.c:2365:
+ pr_info_once("%s: Skipping topology span sanity check."
+ " Use `sched_verbose` boot parameter to
enable it.\n",

total: 0 errors, 1 warnings, 14 lines checked

Regards,
Naman