From: kernel test robot <lkp@intel.com>
To: KP Singh <kpsingh@kernel.org>,
linux-security-module@vger.kernel.org, bpf@vger.kernel.org
Cc: oe-kbuild-all@lists.linux.dev, paul@paul-moore.com,
keescook@chromium.org, casey@schaufler-ca.com, song@kernel.org,
daniel@iogearbox.net, ast@kernel.org, kpsingh@kernel.org,
renauld@google.com
Subject: Re: [PATCH v4 3/5] security: Replace indirect LSM hook calls with static calls
Date: Sat, 23 Sep 2023 22:52:33 +0800 [thread overview]
Message-ID: <202309232244.uCfB7AMn-lkp@intel.com> (raw)
In-Reply-To: <20230922145505.4044003-4-kpsingh@kernel.org>
Hi KP,
kernel test robot noticed the following build warnings:
[auto build test WARNING on bpf-next/master]
[also build test WARNING on bpf/master pcmoore-selinux/next linus/master v6.6-rc2 next-20230921]
[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/KP-Singh/kernel-Add-helper-macros-for-loop-unrolling/20230922-225925
base: https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next.git master
patch link: https://lore.kernel.org/r/20230922145505.4044003-4-kpsingh%40kernel.org
patch subject: [PATCH v4 3/5] security: Replace indirect LSM hook calls with static calls
config: arm64-defconfig (https://download.01.org/0day-ci/archive/20230923/202309232244.uCfB7AMn-lkp@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20230923/202309232244.uCfB7AMn-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/202309232244.uCfB7AMn-lkp@intel.com/
All warnings (new ones prefixed by >>):
security/security.c: In function 'security_binder_set_context_mgr':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:872:16: note: in expansion of macro 'call_int_hook'
872 | return call_int_hook(binder_set_context_mgr, 0, mgr);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_binder_transaction':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:887:16: note: in expansion of macro 'call_int_hook'
887 | return call_int_hook(binder_transaction, 0, from, to);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_binder_transfer_binder':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:902:16: note: in expansion of macro 'call_int_hook'
902 | return call_int_hook(binder_transfer_binder, 0, from, to);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_binder_transfer_file':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:918:16: note: in expansion of macro 'call_int_hook'
918 | return call_int_hook(binder_transfer_file, 0, from, to, file);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_ptrace_access_check':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:937:16: note: in expansion of macro 'call_int_hook'
937 | return call_int_hook(ptrace_access_check, 0, child, mode);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_ptrace_traceme':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:952:16: note: in expansion of macro 'call_int_hook'
952 | return call_int_hook(ptrace_traceme, 0, parent);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_capget':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:974:16: note: in expansion of macro 'call_int_hook'
974 | return call_int_hook(capget, 0, target,
| ^~~~~~~~~~~~~
security/security.c: In function 'security_capset':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:996:16: note: in expansion of macro 'call_int_hook'
996 | return call_int_hook(capset, 0, new, old,
| ^~~~~~~~~~~~~
security/security.c: In function 'security_capable':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1018:16: note: in expansion of macro 'call_int_hook'
1018 | return call_int_hook(capable, 0, cred, ns, cap, opts);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_quotactl':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1034:16: note: in expansion of macro 'call_int_hook'
1034 | return call_int_hook(quotactl, 0, cmds, type, id, sb);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_quota_on':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1047:16: note: in expansion of macro 'call_int_hook'
1047 | return call_int_hook(quota_on, 0, dentry);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_syslog':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1062:16: note: in expansion of macro 'call_int_hook'
1062 | return call_int_hook(syslog, 0, type);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_settime64':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1077:16: note: in expansion of macro 'call_int_hook'
1077 | return call_int_hook(settime, 0, ts, tz);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_bprm_creds_for_exec':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1132:16: note: in expansion of macro 'call_int_hook'
1132 | return call_int_hook(bprm_creds_for_exec, 0, bprm);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_bprm_creds_from_file':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1156:16: note: in expansion of macro 'call_int_hook'
1156 | return call_int_hook(bprm_creds_from_file, 0, bprm, file);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_bprm_check':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1175:15: note: in expansion of macro 'call_int_hook'
1175 | ret = call_int_hook(bprm_check_security, 0, bprm);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_fs_context_submount':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1225:16: note: in expansion of macro 'call_int_hook'
1225 | return call_int_hook(fs_context_submount, 0, fc, reference);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_fs_context_dup':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1241:16: note: in expansion of macro 'call_int_hook'
1241 | return call_int_hook(fs_context_dup, 0, fc, src_fc);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_sb_alloc':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1289:14: note: in expansion of macro 'call_int_hook'
1289 | rc = call_int_hook(sb_alloc_security, 0, sb);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_sb_eat_lsm_opts':
>> security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1347:16: note: in expansion of macro 'call_int_hook'
1347 | return call_int_hook(sb_eat_lsm_opts, 0, options, mnt_opts);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_sb_mnt_opts_compat':
security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1364:16: note: in expansion of macro 'call_int_hook'
1364 | return call_int_hook(sb_mnt_opts_compat, 0, sb, mnt_opts);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_sb_remount':
security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1381:16: note: in expansion of macro 'call_int_hook'
1381 | return call_int_hook(sb_remount, 0, sb, mnt_opts);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_sb_kern_mount':
security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1395:16: note: in expansion of macro 'call_int_hook'
1395 | return call_int_hook(sb_kern_mount, 0, sb);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_sb_show_options':
security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1409:16: note: in expansion of macro 'call_int_hook'
1409 | return call_int_hook(sb_show_options, 0, m, sb);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_sb_statfs':
security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1423:16: note: in expansion of macro 'call_int_hook'
1423 | return call_int_hook(sb_statfs, 0, dentry);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_sb_mount':
security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1446:16: note: in expansion of macro 'call_int_hook'
1446 | return call_int_hook(sb_mount, 0, dev_name, path, type, flags, data);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_sb_umount':
security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1460:16: note: in expansion of macro 'call_int_hook'
1460 | return call_int_hook(sb_umount, 0, mnt, flags);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_sb_pivotroot':
security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1475:16: note: in expansion of macro 'call_int_hook'
1475 | return call_int_hook(sb_pivotroot, 0, old_path, new_path);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_sb_set_mnt_opts':
security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1494:16: note: in expansion of macro 'call_int_hook'
1494 | return call_int_hook(sb_set_mnt_opts,
| ^~~~~~~~~~~~~
security/security.c: In function 'security_sb_clone_mnt_opts':
security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1516:16: note: in expansion of macro 'call_int_hook'
1516 | return call_int_hook(sb_clone_mnt_opts, 0, oldsb, newsb,
| ^~~~~~~~~~~~~
security/security.c: In function 'security_move_mount':
security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1533:16: note: in expansion of macro 'call_int_hook'
1533 | return call_int_hook(move_mount, 0, from_path, to_path);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_path_notify':
security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1550:16: note: in expansion of macro 'call_int_hook'
1550 | return call_int_hook(path_notify, 0, path, mask, obj_type);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_inode_alloc':
security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
security/security.c:1569:14: note: in expansion of macro 'call_int_hook'
1569 | rc = call_int_hook(inode_alloc_security, 0, inode);
| ^~~~~~~~~~~~~
security/security.c: In function 'security_dentry_create_files_as':
security/security.c:851:1: warning: label 'out' defined but not used [-Wunused-label]
851 | out: \
| ^~~
vim +/out +851 security/security.c
845
846 #define call_int_hook(FUNC, IRC, ...) \
847 ({ \
848 __label__ out; \
849 int RC = IRC; \
850 LSM_LOOP_UNROLL(__CALL_STATIC_INT, RC, FUNC, out, __VA_ARGS__); \
> 851 out: \
852 RC; \
853 })
854
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2023-09-23 14:54 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-09-22 14:55 [PATCH v4 0/5] Reduce overhead of LSMs with static calls KP Singh
2023-09-22 14:55 ` [PATCH v4 1/5] kernel: Add helper macros for loop unrolling KP Singh
2023-09-22 14:55 ` [PATCH v4 2/5] security: Count the LSMs enabled at compile time KP Singh
2023-09-22 15:50 ` Kees Cook
2023-09-22 16:07 ` KP Singh
2023-09-27 22:37 ` KP Singh
2023-09-22 14:55 ` [PATCH v4 3/5] security: Replace indirect LSM hook calls with static calls KP Singh
2023-09-23 14:52 ` kernel test robot [this message]
2023-09-27 5:26 ` kernel test robot
2023-09-22 14:55 ` [PATCH v4 4/5] bpf: Only enable BPF LSM hooks when an LSM program is attached KP Singh
2023-09-22 14:55 ` [PATCH v4 5/5] security: Add CONFIG_SECURITY_HOOK_LIKELY KP Singh
2023-09-22 15:50 ` Kees Cook
2023-09-22 15:51 ` [PATCH v4 0/5] Reduce overhead of LSMs with static calls Kees Cook
2023-09-22 18:42 ` Mateusz Guzik
2023-09-23 16:16 ` KP Singh
2023-09-23 17:13 ` Mateusz Guzik
2023-09-23 17:15 ` Mateusz Guzik
2023-09-24 2:46 ` Kees Cook
2023-09-25 20:08 ` Mateusz Guzik
2023-09-25 22:02 ` Kees Cook
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=202309232244.uCfB7AMn-lkp@intel.com \
--to=lkp@intel.com \
--cc=ast@kernel.org \
--cc=bpf@vger.kernel.org \
--cc=casey@schaufler-ca.com \
--cc=daniel@iogearbox.net \
--cc=keescook@chromium.org \
--cc=kpsingh@kernel.org \
--cc=linux-security-module@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=paul@paul-moore.com \
--cc=renauld@google.com \
--cc=song@kernel.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;
as well as URLs for NNTP newsgroup(s).