Re: [RFC PATCH 11/17] mm/damon/sysfs: implement probe prep directory
[email protected] Sat, 25 Jul 2026 21:10:40 +0000
| Newsgroups | dev.linux.lists.damon |
|---|---|
| Message-ID | <[email protected]> |
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider: - [Low] The static constant `damon_sysfs_prep_ktype` is defined but not used anywhere in the file. -- --- Patch [11]: [RFC PATCH 11/17] mm/damon/sysfs: implement probe prep directory --- commit a150eac865fe6562090f70dc8b3909c07dadeb09 Author: SJ Park <[email protected]> mm/damon/sysfs: implement probe prep directory Implement DAMON sysfs directory and files for specifying each probe prep action. Introduce only preparatory data structure and functions, without wiring them up yet. > diff --git a/mm/damon/sysfs.c b/mm/damon/sysfs.c > index 38affef08d0fb..8f9a5df598c51 100644 > --- a/mm/damon/sysfs.c > +++ b/mm/damon/sysfs.c [ ... ] > +static struct attribute *damon_sysfs_prep_attrs[] = { > + &damon_sysfs_prep_prep_action_attr.attr, > + NULL, > +}; > +ATTRIBUTE_GROUPS(damon_sysfs_prep); > + > +static const struct kobj_type damon_sysfs_prep_ktype = { > + .release = damon_sysfs_prep_release, > + .sysfs_ops = &kobj_sysfs_ops, > + .default_groups = damon_sysfs_prep_groups, > +}; [Severity: Low] Since this patch introduces preparatory data structures that are not yet wired up, the static constant damon_sysfs_prep_ktype is defined but not used anywhere in mm/damon/sysfs.c. Could this trigger a -Wunused-const-variable warning when building the kernel with W=1? -- Sashiko AI review ยท https://sashiko.dev/#/patchset/[email protected]?part=11