Messages in this thread | | | Date | Wed, 31 Jan 2024 16:55:47 -0800 | From | Ricardo Neri <> | Subject | Re: [PATCH v2 5/6] sched/fair: pack SD_ASYM_PACKING into sched_use_asym_prio |
| |
On Tue, Jan 30, 2024 at 09:17:07PM +0800, alexs@kernel.org wrote: > From: Alex Shi <alexs@kernel.org> pack SD_ASYM_PACKING into sched_use_asym_prio
Instead of saying `pack` you could say "Check the SD_ASYM_PACKING flag in sched_use_asym_prio()"
> > Then the flags check passed into sched_asym and sched_group_asym. > It's a code cleanup, no func changes.
I'd the changelog as follows:
"sched_use_asym_prio() checks whether CPU priorities should be used. It makes sense to check for the SD_ASYM_PACKING() inside the function. Since both sched_asym() and sched_group_asym() use sched_use_asym_prio(), remove the now superfluous checks for the flag in various places"
| |