On 2022/7/27 21:00, Johannes Weiner wrote: > On Wed, Jul 27, 2022 at 07:28:37PM +0800, Chengming Zhou wrote: >> On 2022/7/26 02:26, Johannes Weiner wrote: >>> I think we can remove the NR_CPU task count, which frees up one >>> u32. Something like the below diff should work (untested!) >> >> Hi, I tested ok, would you mind if I put this patch in this series? >> >> Subject: [PATCH] sched/psi: remove NR_ONCPU task accounting >> >> We put all fields updated by the scheduler in the first cacheline of >> struct psi_group_cpu for performance. >> >> Since we want add another PSI_IRQ_FULL to track IRQ/SOFTIRQ pressure, >> we need to reclaim space first. This patch remove NR_ONCPU task accounting >> in struct psi_group_cpu, use TSK_ONCPU in state_mask to track instead. > > Thanks for testing it, that sounds good. > >> Signed-off-by: Johannes Weiner <hannes@xxxxxxxxxxx> >> Reviewed-by: Chengming Zhou <zhouchengming@xxxxxxxxxxxxx> >> Tested-by: Chengming Zhou <zhouchengming@xxxxxxxxxxxxx> > > Since you're handling the patch, you need to add your own > Signed-off-by: as well. And keep From: Johannes (git commit --author). Got it. Thanks!
