Messages in this thread | | | Date | Wed, 27 Jul 2022 23:09:53 +0800 | Subject | Re: [PATCH 9/9] sched/psi: add PSI_IRQ to track IRQ/SOFTIRQ pressure | From | Chengming Zhou <> |
| |
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@cmpxchg.org> >> Reviewed-by: Chengming Zhou <zhouchengming@bytedance.com> >> Tested-by: Chengming Zhou <zhouchengming@bytedance.com> > > 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!
| |