On Thu, Feb 27, 2025 at 01:55:40PM -0800, inwardvessel <inwardvessel@xxxxxxxxx> wrote: ... > --- a/kernel/cgroup/cgroup-internal.h > +++ b/kernel/cgroup/cgroup-internal.h > @@ -269,8 +269,8 @@ int cgroup_task_count(const struct cgroup *cgrp); > /* > * rstat.c > */ > -int cgroup_rstat_init(struct cgroup *cgrp); > -void cgroup_rstat_exit(struct cgroup *cgrp); > +int cgroup_rstat_init(struct cgroup_subsys_state *css); > +void cgroup_rstat_exit(struct cgroup_subsys_state *css); Given the functions have little in common with struct cgroup now, I'd not be afraid of renaming them to css_rstat_* so that names match the processed structure. (Similar, _updated and _flush and others.)
Attachment:
signature.asc
Description: PGP signature
