Re: [PATCH] sched/debug:fix stale comments of sched_debug

[Hailong Liu]

On 5/31/21 3:48 PM, Andy Shevchenko wrote:
> On Sun, May 30, 2021 at 10:51:45AM +0800, Hailong Liu wrote:
>> From: Hailong Liu <liu.hailong6@xxxxxxxxxx>
>>
>> Now /proc/sched_debug has been moved to debugfs, so make the comments
>> consistent with it.
> 
> Have you checked if the very same Kconfig option is used for something
> slightly different as well?

Do you mean that I need to check whether SCHED_DEBUG is also related to
other features? 

>> Signed-off-by: Hailong Liu <liu.hailong6@xxxxxxxxxx>
> 
> Missed SoB tag of the submitter.

In fact the exiting SoB tag of this patch is the actual submitter. 

I'm sorry that the inconsistency between the submission email and the SoB
email address caused you to feel confused. 

For some personal reasons, I had to use an email at home and mark another
email address as the submitter. 

> 
>> --- a/lib/Kconfig.debug
>> +++ b/lib/Kconfig.debug
>> @@ -1166,7 +1166,7 @@ config SCHED_DEBUG
>>  	depends on DEBUG_KERNEL && PROC_FS
> 
> Are the dependencies correct?

Based on your suggestion, I checked and it turned out that PROC_FS is not
the correct dependency, but DEBUG_FS. 

Shoud I change it to DEBUG_FS? 

>>  	default y
>>  	help
>> -	  If you say Y here, the /proc/sched_debug file will be provided
>> +	  If you say Y here, the debugfs/sched/debug file will be provided
>>  	  that can help debug the scheduler. The runtime overhead of this
>>  	  option is minimal.
> 

Thanks,

Hailong