On Thu, Jul 04, 2019 at 07:46:13PM +0800, Yi Wang wrote: > When make defconfig, CONFIG_SCHEDSTATS is set to be y, so > sched_info_on() is 'likely' to be true. However, some functions > invoke this function with unlikely hint or use no hint. Let's > fix this. How about remove the hint entirely? likely(1) is as rediculous as unlikely(1), a constant is a constant and no amount of hinting should make the compiler do anything else. And if you want to retain the hint for the TASK_DELAY_ACCT nonsense, stick it there. Also, fix the lack of { } while you're there.