Hello,
On Tue, Jul 02, 2024 at 02:28:22AM +0000, Chen Ridong wrote:
...
> if (cgroup_parent(cgrp)) {
> cgroup_rstat_flush_hold(cgrp);
> usage = cgrp->bstat.cputime.sum_exec_runtime;
> cputime_adjust(&cgrp->bstat.cputime, &cgrp->prev_cputime,
> &utime, &stime);
> -#ifdef CONFIG_SCHED_CORE
> - forceidle_time = cgrp->bstat.forceidle_sum;
> -#endif
> + bstat = cgrp->bstat;
Please don't copy non-trivial struct like this. Maybe add a pointer which
points to the bstat to use?
Thanks.
--
tejun
