Re: [PATCH] sched/topology: Switch to sched_debug() for conditional sched domain printk

2021-02-03 Thread Srikar Dronamraju
* Yue Hu [2021-02-03 18:10:19]: > On Wed, 3 Feb 2021 15:22:56 +0530 > Srikar Dronamraju wrote: > > > * Yue Hu [2021-02-03 12:20:10]: > > > > > > sched_debug() would only be present in CONFIG_SCHED_DEBUG. Right? > > In which case there would be a build failure with your change in > >

Re: [PATCH] sched/topology: Switch to sched_debug() for conditional sched domain printk

2021-02-03 Thread Yue Hu
On Wed, 3 Feb 2021 15:22:56 +0530 Srikar Dronamraju wrote: > * Yue Hu [2021-02-03 12:20:10]: > > > From: Yue Hu > > > > Currently, the macro sched_debug_enabled has same function as > > sched_debug() with return false for !SCHED_DEBUG. And sched_debug() > > is a wapper of variable

Re: [PATCH] sched/topology: Switch to sched_debug() for conditional sched domain printk

2021-02-03 Thread Srikar Dronamraju
* Yue Hu [2021-02-03 12:20:10]: > From: Yue Hu > > Currently, the macro sched_debug_enabled has same function as > sched_debug() with return false for !SCHED_DEBUG. And sched_debug() > is a wapper of variable sched_debug_enabled for SCHED_DEBUG. We > can use the sched_debug() for all cases.

[PATCH] sched/topology: Switch to sched_debug() for conditional sched domain printk

2021-02-02 Thread Yue Hu
From: Yue Hu Currently, the macro sched_debug_enabled has same function as sched_debug() with return false for !SCHED_DEBUG. And sched_debug() is a wapper of variable sched_debug_enabled for SCHED_DEBUG. We can use the sched_debug() for all cases. So, let's remove the unnecessary marco, also use