From: kernel test robot <lkp@intel.com>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
"Masami Hiramatsu (Google)" <mhiramat@kernel.org>
Subject: [trace:tracefs/for-next 1/3] fs/tracefs/event_inode.c:577:6: warning: variable 'ret' set but not used
Date: Fri, 10 Apr 2026 21:18:03 +0800 [thread overview]
Message-ID: <202604102117.N77kyKGR-lkp@intel.com> (raw)
tree: https://git.kernel.org/pub/scm/linux/kernel/git/trace/linux-trace tracefs/for-next
head: e8368d1f4bedbb0cce4cfe33a1d2664bb0fd4f27
commit: 4d9b262031ffef203243e53577a90ae6e1090e67 [1/3] eventfs: Simplify code using guard()s
config: x86_64-kexec (https://download.01.org/0day-ci/archive/20260410/202604102117.N77kyKGR-lkp@intel.com/config)
compiler: clang version 20.1.8 (https://github.com/llvm/llvm-project 87f0227cb60147a26a1eeb4fb06e3b505e9c7261)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260410/202604102117.N77kyKGR-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/202604102117.N77kyKGR-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> fs/tracefs/event_inode.c:577:6: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
577 | int ret = -EINVAL;
| ^
1 warning generated.
vim +/ret +577 fs/tracefs/event_inode.c
63940449555e799 Ajay Kaher 2023-07-28 563
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 564) /*
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 565) * Walk the children of a eventfs_inode to fill in getdents().
63940449555e799 Ajay Kaher 2023-07-28 566 */
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 567) static int eventfs_iterate(struct file *file, struct dir_context *ctx)
63940449555e799 Ajay Kaher 2023-07-28 568 {
5790b1fb3d672d9 Steven Rostedt (Google 2023-10-04 569) const struct file_operations *fops;
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 570) struct inode *f_inode = file_inode(file);
5790b1fb3d672d9 Steven Rostedt (Google 2023-10-04 571) const struct eventfs_entry *entry;
5790b1fb3d672d9 Steven Rostedt (Google 2023-10-04 572) struct eventfs_inode *ei_child;
63940449555e799 Ajay Kaher 2023-07-28 573 struct tracefs_inode *ti;
63940449555e799 Ajay Kaher 2023-07-28 574 struct eventfs_inode *ei;
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 575) const char *name;
5790b1fb3d672d9 Steven Rostedt (Google 2023-10-04 576) umode_t mode;
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 @577) int ret = -EINVAL;
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 578) int ino;
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 579) int i, r, c;
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 580)
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 581) if (!dir_emit_dots(file, ctx))
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 582) return 0;
63940449555e799 Ajay Kaher 2023-07-28 583
63940449555e799 Ajay Kaher 2023-07-28 584 ti = get_tracefs(f_inode);
63940449555e799 Ajay Kaher 2023-07-28 585 if (!(ti->flags & TRACEFS_EVENT_INODE))
63940449555e799 Ajay Kaher 2023-07-28 586 return -EINVAL;
63940449555e799 Ajay Kaher 2023-07-28 587
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 588) c = ctx->pos - 2;
ef36b4f92868d66 Steven Rostedt (Google 2023-09-22 589)
4d9b262031ffef2 Steven Rostedt 2025-06-04 590 guard(srcu)(&eventfs_srcu);
5790b1fb3d672d9 Steven Rostedt (Google 2023-10-04 591)
4d9b262031ffef2 Steven Rostedt 2025-06-04 592 scoped_guard(mutex, &eventfs_mutex) {
5790b1fb3d672d9 Steven Rostedt (Google 2023-10-04 593) ei = READ_ONCE(ti->private);
4d9b262031ffef2 Steven Rostedt 2025-06-04 594 if (!ei || ei->is_freed)
4d9b262031ffef2 Steven Rostedt 2025-06-04 595 return -EINVAL;
4d9b262031ffef2 Steven Rostedt 2025-06-04 596 }
5790b1fb3d672d9 Steven Rostedt (Google 2023-10-04 597)
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 598) /*
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 599) * Need to create the dentries and inodes to have a consistent
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 600) * inode number.
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 601) */
1de94b52d5e8d8b Steven Rostedt (Google 2024-01-04 602) ret = 0;
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 603)
1de94b52d5e8d8b Steven Rostedt (Google 2024-01-04 604) /* Start at 'c' to jump over already read entries */
1de94b52d5e8d8b Steven Rostedt (Google 2024-01-04 605) for (i = c; i < ei->nr_entries; i++, ctx->pos++) {
1de94b52d5e8d8b Steven Rostedt (Google 2024-01-04 606) void *cdata = ei->data;
1e4624eb5a0ecaa Steven Rostedt (Google 2024-01-04 607)
704f960dbee2f16 Steven Rostedt (Google 2024-01-04 608) entry = &ei->entries[i];
704f960dbee2f16 Steven Rostedt (Google 2024-01-04 609) name = entry->name;
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 610)
704f960dbee2f16 Steven Rostedt (Google 2024-01-04 611) /* If ei->is_freed then just bail here, nothing more to do */
4d9b262031ffef2 Steven Rostedt 2025-06-04 612 scoped_guard(mutex, &eventfs_mutex) {
4d9b262031ffef2 Steven Rostedt 2025-06-04 613 if (ei->is_freed)
4d9b262031ffef2 Steven Rostedt 2025-06-04 614 return -EINVAL;
704f960dbee2f16 Steven Rostedt (Google 2024-01-04 615) r = entry->callback(name, &mode, &cdata, &fops);
4d9b262031ffef2 Steven Rostedt 2025-06-04 616 }
704f960dbee2f16 Steven Rostedt (Google 2024-01-04 617) if (r <= 0)
704f960dbee2f16 Steven Rostedt (Google 2024-01-04 618) continue;
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 619)
852e46e239ee6db Steven Rostedt (Google 2024-01-16 620) ino = EVENTFS_FILE_INODE_INO;
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 621)
704f960dbee2f16 Steven Rostedt (Google 2024-01-04 622) if (!dir_emit(ctx, name, strlen(name), ino, DT_REG))
4d9b262031ffef2 Steven Rostedt 2025-06-04 623 return -EINVAL;
5790b1fb3d672d9 Steven Rostedt (Google 2023-10-04 624) }
ef36b4f92868d66 Steven Rostedt (Google 2023-09-22 625)
1de94b52d5e8d8b Steven Rostedt (Google 2024-01-04 626) /* Subtract the skipped entries above */
1de94b52d5e8d8b Steven Rostedt (Google 2024-01-04 627) c -= min((unsigned int)c, (unsigned int)ei->nr_entries);
1de94b52d5e8d8b Steven Rostedt (Google 2024-01-04 628)
704f960dbee2f16 Steven Rostedt (Google 2024-01-04 629) list_for_each_entry_srcu(ei_child, &ei->children, list,
704f960dbee2f16 Steven Rostedt (Google 2024-01-04 630) srcu_read_lock_held(&eventfs_srcu)) {
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 631)
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 632) if (c > 0) {
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 633) c--;
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 634) continue;
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 635) }
493ec81a8fb8e4a Steven Rostedt (Google 2024-01-03 636)
1e4624eb5a0ecaa Steven Rostedt (Google 2024-01-04 637) ctx->pos++;
1e4624eb5a0ecaa Steven Rostedt (Google 2024-01-04 638)
704f960dbee2f16 Steven Rostedt (Google 2024-01-04 639) if (ei_child->is_freed)
5790b1fb3d672d9 Steven Rostedt (Google 2023-10-04 640) continue;
ef36b4f92868d66 Steven Rostedt (Google 2023-09-22 641)
704f960dbee2f16 Steven Rostedt (Google 2024-01-04 642) name = ei_child->name;
704f960dbee2f16 Steven Rostedt (Google 2024-01-04 643)
834bf76add3e616 Steven Rostedt (Google 2024-01-22 644) ino = eventfs_dir_ino(ei_child);
ef36b4f92868d66 Steven Rostedt (Google 2023-09-22 645)
4d9b262031ffef2 Steven Rostedt 2025-06-04 646 if (!dir_emit(ctx, name, strlen(name), ino, DT_DIR)) {
1e4624eb5a0ecaa Steven Rostedt (Google 2024-01-04 647) /* Incremented ctx->pos without adding something, reset it */
1e4624eb5a0ecaa Steven Rostedt (Google 2024-01-04 648) ctx->pos--;
4d9b262031ffef2 Steven Rostedt 2025-06-04 649 return -EINVAL;
4d9b262031ffef2 Steven Rostedt 2025-06-04 650 }
4d9b262031ffef2 Steven Rostedt 2025-06-04 651 }
4d9b262031ffef2 Steven Rostedt 2025-06-04 652 return 1;
63940449555e799 Ajay Kaher 2023-07-28 653 }
63940449555e799 Ajay Kaher 2023-07-28 654
:::::: The code at line 577 was first introduced by commit
:::::: 493ec81a8fb8e4ada6f223b8b73791a1280d4774 eventfs: Stop using dcache_readdir() for getdents()
:::::: TO: Steven Rostedt (Google) <rostedt@goodmis.org>
:::::: CC: Steven Rostedt (Google) <rostedt@goodmis.org>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
reply other threads:[~2026-04-10 13:18 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=202604102117.N77kyKGR-lkp@intel.com \
--to=lkp@intel.com \
--cc=llvm@lists.linux.dev \
--cc=mhiramat@kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=rostedt@goodmis.org \
/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