From: kernel test robot <lkp@intel.com>
To: JP Kobryn <inwardvessel@gmail.com>,
shakeel.butt@linux.dev, tj@kernel.org, mhocko@kernel.org,
hannes@cmpxchg.org, yosryahmed@google.com,
akpm@linux-foundation.org
Cc: oe-kbuild-all@lists.linux.dev, linux-mm@kvack.org,
cgroups@vger.kernel.org, kernel-team@meta.com
Subject: Re: [PATCH 04/11] cgroup: introduce cgroup_rstat_ops
Date: Wed, 19 Feb 2025 15:21:05 +0800 [thread overview]
Message-ID: <202502191558.xCTZRkPs-lkp@intel.com> (raw)
In-Reply-To: <20250218031448.46951-5-inwardvessel@gmail.com>
Hi JP,
kernel test robot noticed the following build warnings:
[auto build test WARNING on tj-cgroup/for-next]
[also build test WARNING on bpf-next/master bpf/master linus/master v6.14-rc3 next-20250219]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/JP-Kobryn/cgroup-move-rstat-pointers-into-struct-of-their-own/20250218-111725
base: https://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup.git for-next
patch link: https://lore.kernel.org/r/20250218031448.46951-5-inwardvessel%40gmail.com
patch subject: [PATCH 04/11] cgroup: introduce cgroup_rstat_ops
config: arc-randconfig-002-20250219 (https://download.01.org/0day-ci/archive/20250219/202502191558.xCTZRkPs-lkp@intel.com/config)
compiler: arc-elf-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250219/202502191558.xCTZRkPs-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202502191558.xCTZRkPs-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> kernel/cgroup/rstat.c:210: warning: Function parameter or struct member 'ops' not described in 'cgroup_rstat_push_children'
>> kernel/cgroup/rstat.c:269: warning: Function parameter or struct member 'ops' not described in 'cgroup_rstat_updated_list'
>> kernel/cgroup/rstat.c:465: warning: Function parameter or struct member 'ops' not described in '__cgroup_rstat_flush_release'
kernel/cgroup/rstat.c:465: warning: expecting prototype for cgroup_rstat_flush_release(). Prototype was for __cgroup_rstat_flush_release() instead
vim +210 kernel/cgroup/rstat.c
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 194
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 195 /**
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 196 * cgroup_rstat_push_children - push children cgroups into the given list
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 197 * @head: current head of the list (= subtree root)
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 198 * @child: first child of the root
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 199 * @cpu: target cpu
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 200 * Return: A new singly linked list of cgroups to be flush
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 201 *
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 202 * Iteratively traverse down the cgroup_rstat_cpu updated tree level by
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 203 * level and push all the parents first before their next level children
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 204 * into a singly linked list built from the tail backward like "pushing"
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 205 * cgroups into a stack. The root is pushed by the caller.
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 206 */
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 207 static struct cgroup_rstat *cgroup_rstat_push_children(
d67ed623c585f2 kernel/cgroup/rstat.c JP Kobryn 2025-02-17 208 struct cgroup_rstat *head, struct cgroup_rstat *child, int cpu,
d67ed623c585f2 kernel/cgroup/rstat.c JP Kobryn 2025-02-17 209 struct cgroup_rstat_ops *ops)
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 @210 {
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 211 struct cgroup_rstat *chead = child; /* Head of child cgroup level */
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 212 struct cgroup_rstat *ghead = NULL; /* Head of grandchild cgroup level */
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 213 struct cgroup_rstat *parent, *grandchild;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 214 struct cgroup_rstat_cpu *crstatc;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 215
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 216 child->rstat_flush_next = NULL;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 217
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 218 next_level:
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 219 while (chead) {
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 220 child = chead;
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 221 chead = child->rstat_flush_next;
d67ed623c585f2 kernel/cgroup/rstat.c JP Kobryn 2025-02-17 222 parent = ops->parent_fn(child);
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 223
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 224 /* updated_next is parent cgroup terminated */
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 225 while (child != parent) {
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 226 child->rstat_flush_next = head;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 227 head = child;
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 228 crstatc = rstat_cpu(child, cpu);
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 229 grandchild = crstatc->updated_children;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 230 if (grandchild != child) {
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 231 /* Push the grand child to the next level */
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 232 crstatc->updated_children = child;
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 233 grandchild->rstat_flush_next = ghead;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 234 ghead = grandchild;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 235 }
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 236 child = crstatc->updated_next;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 237 crstatc->updated_next = NULL;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 238 }
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 239 }
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 240
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 241 if (ghead) {
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 242 chead = ghead;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 243 ghead = NULL;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 244 goto next_level;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 245 }
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 246 return head;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 247 }
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 248
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 249 /**
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 250 * cgroup_rstat_updated_list - return a list of updated cgroups to be flushed
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 251 * @root: root of the cgroup subtree to traverse
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 252 * @cpu: target cpu
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 253 * Return: A singly linked list of cgroups to be flushed
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 254 *
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 255 * Walks the updated rstat_cpu tree on @cpu from @root. During traversal,
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 256 * each returned cgroup is unlinked from the updated tree.
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 257 *
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 258 * The only ordering guarantee is that, for a parent and a child pair
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 259 * covered by a given traversal, the child is before its parent in
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 260 * the list.
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 261 *
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 262 * Note that updated_children is self terminated and points to a list of
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 263 * child cgroups if not empty. Whereas updated_next is like a sibling link
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 264 * within the children list and terminated by the parent cgroup. An exception
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 265 * here is the cgroup root whose updated_next can be self terminated.
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 266 */
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 267 static struct cgroup_rstat *cgroup_rstat_updated_list(
d67ed623c585f2 kernel/cgroup/rstat.c JP Kobryn 2025-02-17 268 struct cgroup_rstat *root, int cpu, struct cgroup_rstat_ops *ops)
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 @269 {
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 270 raw_spinlock_t *cpu_lock = per_cpu_ptr(&cgroup_rstat_cpu_lock, cpu);
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 271 struct cgroup_rstat_cpu *rstatc = rstat_cpu(root, cpu);
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 272 struct cgroup_rstat *head = NULL, *parent, *child;
d67ed623c585f2 kernel/cgroup/rstat.c JP Kobryn 2025-02-17 273 struct cgroup *cgrp;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 274 unsigned long flags;
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 275
d67ed623c585f2 kernel/cgroup/rstat.c JP Kobryn 2025-02-17 276 cgrp = ops->cgroup_fn(root);
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 277 flags = _cgroup_rstat_cpu_lock(cpu_lock, cpu, cgrp, false);
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 278
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 279 /* Return NULL if this subtree is not on-list */
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 280 if (!rstatc->updated_next)
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 281 goto unlock_ret;
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 282
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 283 /*
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 284 * Unlink @root from its parent. As the updated_children list is
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 285 * singly linked, we have to walk it to find the removal point.
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 286 */
d67ed623c585f2 kernel/cgroup/rstat.c JP Kobryn 2025-02-17 287 parent = ops->parent_fn(root);
dc26532aed0ab2 kernel/cgroup/rstat.c Johannes Weiner 2021-04-29 288 if (parent) {
dc26532aed0ab2 kernel/cgroup/rstat.c Johannes Weiner 2021-04-29 289 struct cgroup_rstat_cpu *prstatc;
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 290 struct cgroup_rstat **nextp;
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 291
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 292 prstatc = rstat_cpu(parent, cpu);
c58632b3631cb2 kernel/cgroup/rstat.c Tejun Heo 2018-04-26 293 nextp = &prstatc->updated_children;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 294 while (*nextp != root) {
dc26532aed0ab2 kernel/cgroup/rstat.c Johannes Weiner 2021-04-29 295 struct cgroup_rstat_cpu *nrstatc;
dc26532aed0ab2 kernel/cgroup/rstat.c Johannes Weiner 2021-04-29 296
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 297 nrstatc = rstat_cpu(*nextp, cpu);
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 298 WARN_ON_ONCE(*nextp == parent);
c58632b3631cb2 kernel/cgroup/rstat.c Tejun Heo 2018-04-26 299 nextp = &nrstatc->updated_next;
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 300 }
c58632b3631cb2 kernel/cgroup/rstat.c Tejun Heo 2018-04-26 301 *nextp = rstatc->updated_next;
dc26532aed0ab2 kernel/cgroup/rstat.c Johannes Weiner 2021-04-29 302 }
9a9e97b2f1f27e kernel/cgroup/rstat.c Tejun Heo 2018-04-26 303
dc26532aed0ab2 kernel/cgroup/rstat.c Johannes Weiner 2021-04-29 304 rstatc->updated_next = NULL;
e76d28bdf9ba53 kernel/cgroup/rstat.c Waiman Long 2023-11-03 305
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 306 /* Push @root to the list first before pushing the children */
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 307 head = root;
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 308 root->rstat_flush_next = NULL;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 309 child = rstatc->updated_children;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 310 rstatc->updated_children = root;
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 311 if (child != root)
d67ed623c585f2 kernel/cgroup/rstat.c JP Kobryn 2025-02-17 312 head = cgroup_rstat_push_children(head, child, cpu, ops);
d499fd418fa159 kernel/cgroup/rstat.c Waiman Long 2023-11-30 313 unlock_ret:
3d844899ba042a kernel/cgroup/rstat.c JP Kobryn 2025-02-17 314 _cgroup_rstat_cpu_unlock(cpu_lock, cpu, cgrp, flags, false);
e76d28bdf9ba53 kernel/cgroup/rstat.c Waiman Long 2023-11-03 315 return head;
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 316 }
041cd640b2f3c5 kernel/cgroup/stat.c Tejun Heo 2017-09-25 317
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2025-02-19 7:22 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-18 3:14 [PATCH 00/11] cgroup: separate rstat trees JP Kobryn
2025-02-18 3:14 ` [PATCH 01/11] cgroup: move rstat pointers into struct of their own JP Kobryn
2025-02-19 1:05 ` Shakeel Butt
2025-02-19 1:23 ` Shakeel Butt
2025-02-20 16:53 ` Yosry Ahmed
2025-02-24 17:06 ` JP Kobryn
2025-02-24 18:36 ` Yosry Ahmed
2025-02-18 3:14 ` [PATCH 02/11] cgroup: add level of indirection for cgroup_rstat struct JP Kobryn
2025-02-19 2:26 ` Shakeel Butt
2025-02-20 17:08 ` Yosry Ahmed
2025-02-19 5:57 ` kernel test robot
2025-02-18 3:14 ` [PATCH 03/11] cgroup: move cgroup_rstat from cgroup to cgroup_subsys_state JP Kobryn
2025-02-20 17:06 ` Shakeel Butt
2025-02-20 17:22 ` Yosry Ahmed
2025-02-25 19:20 ` JP Kobryn
2025-02-18 3:14 ` [PATCH 04/11] cgroup: introduce cgroup_rstat_ops JP Kobryn
2025-02-19 7:21 ` kernel test robot [this message]
2025-02-20 17:50 ` Shakeel Butt
2025-02-18 3:14 ` [PATCH 05/11] cgroup: separate rstat for bpf cgroups JP Kobryn
2025-02-21 18:14 ` Shakeel Butt
2025-02-18 3:14 ` [PATCH 06/11] cgroup: rstat lock indirection JP Kobryn
2025-02-21 22:09 ` Shakeel Butt
2025-02-18 3:14 ` [PATCH 07/11] cgroup: fetch cpu-specific lock in rstat cpu lock helpers JP Kobryn
2025-02-21 22:35 ` Shakeel Butt
2025-02-18 3:14 ` [PATCH 08/11] cgroup: rstat cpu lock indirection JP Kobryn
2025-02-19 8:48 ` kernel test robot
2025-02-22 0:18 ` Shakeel Butt
2025-02-18 3:14 ` [PATCH 09/11] cgroup: separate rstat locks for bpf cgroups JP Kobryn
2025-02-18 3:14 ` [PATCH 10/11] cgroup: separate rstat locks for subsystems JP Kobryn
2025-02-22 0:23 ` Shakeel Butt
2025-02-18 3:14 ` [PATCH 11/11] cgroup: separate rstat list pointers from base stats JP Kobryn
2025-02-22 0:28 ` Shakeel Butt
2025-02-20 15:51 ` [PATCH 00/11] cgroup: separate rstat trees Tejun Heo
2025-02-27 23:44 ` JP Kobryn
2025-02-20 17:26 ` Yosry Ahmed
2025-02-20 17:53 ` Shakeel Butt
2025-02-20 17:59 ` Yosry Ahmed
2025-02-20 18:14 ` JP Kobryn
2025-02-20 20:04 ` Yosry Ahmed
2025-02-20 20:22 ` Yosry Ahmed
2025-02-24 21:13 ` Shakeel Butt
2025-02-24 21:54 ` Yosry Ahmed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202502191558.xCTZRkPs-lkp@intel.com \
--to=lkp@intel.com \
--cc=akpm@linux-foundation.org \
--cc=cgroups@vger.kernel.org \
--cc=hannes@cmpxchg.org \
--cc=inwardvessel@gmail.com \
--cc=kernel-team@meta.com \
--cc=linux-mm@kvack.org \
--cc=mhocko@kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=shakeel.butt@linux.dev \
--cc=tj@kernel.org \
--cc=yosryahmed@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox