[akpm-mm:mm-unstable 26/98] fs/libfs.c:1231:47: warning: Local variable 'anon_aops' shadows outer variable [shadowVariable]
From: kernel test robot
Date: Mon Aug 15 2022 - 11:37:49 EST
tree: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-unstable
head: d2af7b221349ff6241e25fa8c67bcfae2b360700
commit: e4ab315810850b93cac381d6f4efd532f45a790e [26/98] procfs: add 'path' to /proc/<pid>/fdinfo/
compiler: or1k-linux-gcc (GCC) 12.1.0
reproduce (cppcheck warning):
# apt-get install cppcheck
git checkout e4ab315810850b93cac381d6f4efd532f45a790e
cppcheck --quiet --enable=style,performance,portability --template=gcc FILE
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@xxxxxxxxx>
cppcheck warnings: (new ones prefixed by >>)
>> init/main.c:333:18: warning: Pointer addition with NULL pointer. [nullPointerArithmetic]
char *end = buf + size;
^
init/main.c:376:28: note: Calling function 'xbc_snprint_cmdline', 1st argument 'NULL' value is 0
len = xbc_snprint_cmdline(NULL, 0, root);
^
init/main.c:333:18: note: Null pointer addition
char *end = buf + size;
^
>> fs/libfs.c:1231:47: warning: Local variable 'anon_aops' shadows outer variable [shadowVariable]
static const struct address_space_operations anon_aops = {
^
fs/libfs.c:1220:46: note: Shadowed declaration
static const struct address_space_operations anon_aops = {
^
fs/libfs.c:1231:47: note: Shadow variable
static const struct address_space_operations anon_aops = {
^
cppcheck possible warnings: (new ones prefixed by >>, may not real problems)
>> init/main.c:736:28: warning: Comparing pointers that point to different objects [comparePointers]
for (p = __setup_start; p < __setup_end; p++) {
^
init/main.c:197:38: note: Variable declared here.
extern const struct obs_kernel_param __setup_start[], __setup_end[];
^
init/main.c:736:11: note: Array decayed to pointer here.
for (p = __setup_start; p < __setup_end; p++) {
^
init/main.c:197:55: note: Variable declared here.
extern const struct obs_kernel_param __setup_start[], __setup_end[];
^
init/main.c:736:30: note: Array decayed to pointer here.
for (p = __setup_start; p < __setup_end; p++) {
^
init/main.c:736:28: note: Comparing pointers that point to different objects
for (p = __setup_start; p < __setup_end; p++) {
^
init/main.c:1412:33: warning: Comparing pointers that point to different objects [comparePointers]
for (fn = __initcall_start; fn < __initcall0_start; fn++)
^
init/main.c:1316:25: note: Variable declared here.
extern initcall_entry_t __initcall_start[];
^
init/main.c:1412:12: note: Array decayed to pointer here.
for (fn = __initcall_start; fn < __initcall0_start; fn++)
^
init/main.c:1317:25: note: Variable declared here.
extern initcall_entry_t __initcall0_start[];
^
init/main.c:1412:35: note: Array decayed to pointer here.
for (fn = __initcall_start; fn < __initcall0_start; fn++)
^
init/main.c:1412:33: note: Comparing pointers that point to different objects
for (fn = __initcall_start; fn < __initcall0_start; fn++)
^
>> init/main.c:1215:31: warning: Uninitialized variable: entry->buf [uninitvar]
if (!strcmp(fn_name, entry->buf)) {
^
init/main.c:1202:16: note: Assuming condition is false
if (list_empty(&blacklisted_initcalls))
^
init/main.c:1215:31: note: Uninitialized variable: entry->buf
if (!strcmp(fn_name, entry->buf)) {
^
--
>> kernel/sysctl.c:224:52: warning: Parameter 'ppos' can be declared with const [constParameter]
static bool proc_first_pos_non_zero_ignore(loff_t *ppos,
^
>> kernel/sysctl.c:571:19: warning: Parameter 'lenp' can be declared with const [constParameter]
size_t *lenp, loff_t *ppos,
^
>> kernel/sysctl.c:787:41: warning: Parameter 'table' can be declared with const [constParameter]
static int proc_taint(struct ctl_table *table, int write,
^
>> kernel/sysctl.c:857:36: warning: Uninitialized variable: tmp [uninitvar]
if ((param->min && *param->min > tmp) ||
^
kernel/sysctl.c:853:6: note: Assuming condition is false
if (ret)
^
kernel/sysctl.c:857:36: note: Uninitialized variable: tmp
if ((param->min && *param->min > tmp) ||
^
kernel/sysctl.c:922:36: warning: Uninitialized variable: tmp [uninitvar]
if ((param->min && *param->min > tmp) ||
^
kernel/sysctl.c:918:6: note: Assuming condition is false
if (ret)
^
kernel/sysctl.c:922:36: note: Uninitialized variable: tmp
if ((param->min && *param->min > tmp) ||
^
kernel/sysctl.c:1259:36: warning: Uninitialized variable: tmp [uninitvar]
if ((param->min && *param->min > tmp) ||
^
kernel/sysctl.c:1255:6: note: Assuming condition is false
if (ret)
^
kernel/sysctl.c:1259:36: note: Uninitialized variable: tmp
if ((param->min && *param->min > tmp) ||
^
>> mm/vmstat.c:1420:53: warning: Parameter 'pos' can be declared with const [constParameter]
static void *frag_start(struct seq_file *m, loff_t *pos)
^
>> mm/vmstat.c:1661:68: warning: Parameter 'zone' can be declared with const [constParameter]
static bool is_zone_first_populated(pg_data_t *pgdat, struct zone *zone)
^
mm/vmstat.c:1791:55: warning: Parameter 'pos' can be declared with const [constParameter]
static void *vmstat_start(struct seq_file *m, loff_t *pos)
^
>> mm/dmapool.c:479:65: warning: Parameter 'match_data' can be declared with const [constParameter]
static int dmam_pool_match(struct device *dev, void *res, void *match_data)
^
>> mm/dmapool.c:88:20: warning: Uninitialized variable: page->in_use [uninitvar]
blocks += page->in_use;
^
>> mm/dmapool.c:95:21: warning: Uninitialized variable: pool->allocation [uninitvar]
pages * (pool->allocation / pool->size),
^
>> mm/dmapool.c:326:13: warning: Uninitialized variable: page->offset [uninitvar]
if (page->offset < pool->allocation)
^
>> mm/dmapool.c:387:19: warning: Uninitialized variable: page->dma [uninitvar]
if (dma < page->dma)
^
>> mm/madvise.c:1485:6: warning: Redundant assignment of 'ret' to itself. [selfAssignment]
ret = (total_len - iov_iter_count(&iter)) ? : ret;
^
>> mm/madvise.c:125:28: warning: Parameter 'anon_name' can be declared with const [constParameter]
struct anon_vma_name *anon_name)
^
--
>> fs/proc/task_mmu.c:127:50: warning: Parameter 'ppos' can be declared with const [constParameter]
static void *m_start(struct seq_file *m, loff_t *ppos)
^
>> fs/proc/task_mmu.c:1349:64: warning: Parameter 'pme' can be declared with const [constParameter]
static int add_to_pagemap(unsigned long addr, pagemap_entry_t *pme,
^
>> kernel/sched/fair.c:11575:62: warning: Same value in both branches of ternary operator. [duplicateValueTernary]
update_load_avg(cfs_rq, se, sched_feat(ATTACH_AGE_LOAD) ? 0 : SKIP_AGE_LOAD);
^
>> kernel/sched/fair.c:7461:1: warning: Label 'done' is not used. There is #if in function body so the label might be used in code that is removed by the preprocessor. [unusedLabelConfiguration]
done: __maybe_unused;
^
>> kernel/sched/sched.h:2058:67: warning: Parameter 'p' can be declared with const [constParameter]
static inline int task_current(struct rq *rq, struct task_struct *p)
^
>> kernel/sched/fair.c:5149:25: warning: Uninitialized variables: cfs_rq.load, cfs_rq.nr_running, cfs_rq.h_nr_running, cfs_rq.idle_nr_running, cfs_rq.idle_h_nr_running, cfs_rq.exec_clock, cfs_rq.min_vruntime, cfs_rq.min_vruntime_copy, cfs_rq.tasks_timeline, cfs_rq.curr, cfs_rq.next, cfs_rq.last, cfs_rq.skip [uninitvar]
struct rq *rq = rq_of(cfs_rq);
^
kernel/sched/fair.c:5149:25: warning: Uninitialized variables: cfs_rq.load, cfs_rq.nr_running, cfs_rq.h_nr_running, cfs_rq.idle_nr_running, cfs_rq.idle_h_nr_running, cfs_rq.exec_clock, cfs_rq.min_vruntime, cfs_rq.min_vruntime_copy, cfs_rq.tasks_timeline, cfs_rq.curr, cfs_rq.next, cfs_rq.last, cfs_rq.skip, cfs_rq.rq, cfs_rq.on_list, cfs_rq.leaf_cfs_rq_list, cfs_rq.tg, cfs_rq.idle, cfs_rq.runtime_enabled, cfs_rq.runtime_remaining, cfs_rq.throttled_pelt_idle, cfs_rq.throttled_pelt_idle_copy, cfs_rq.throttled_clock, cfs_rq.throttled_clock_pelt, cfs_rq.throttled_clock_pelt_time, cfs_rq.throttled, cfs_rq.throttle_count, cfs_rq.throttled_list [uninitvar]
struct rq *rq = rq_of(cfs_rq);
^
>> kernel/sched/fair.c:6481:16: warning: Local variable 'task_util' shadows outer function [shadowFunction]
unsigned long task_util, best_cap = 0;
^
kernel/sched/fair.c:4093:29: note: Shadowed declaration
static inline unsigned long task_util(struct task_struct *p)
^
kernel/sched/fair.c:6481:16: note: Shadow variable
unsigned long task_util, best_cap = 0;
^
kernel/sched/fair.c:6522:16: warning: Local variable 'task_util' shadows outer function [shadowFunction]
unsigned long task_util;
^
kernel/sched/fair.c:4093:29: note: Shadowed declaration
static inline unsigned long task_util(struct task_struct *p)
^
kernel/sched/fair.c:6522:16: note: Shadow variable
unsigned long task_util;
^
>> kernel/sched/fair.c:10528:6: warning: Local variable 'update_next_balance' shadows outer function [shadowFunction]
int update_next_balance = 0;
^
kernel/sched/fair.c:10382:1: note: Shadowed declaration
update_next_balance(struct sched_domain *sd, unsigned long *next_balance)
^
kernel/sched/fair.c:10528:6: note: Shadow variable
int update_next_balance = 0;
^
kernel/sched/fair.c:10926:6: warning: Local variable 'update_next_balance' shadows outer function [shadowFunction]
int update_next_balance = 0;
^
kernel/sched/fair.c:10382:1: note: Shadowed declaration
update_next_balance(struct sched_domain *sd, unsigned long *next_balance)
^
kernel/sched/fair.c:10926:6: note: Shadow variable
int update_next_balance = 0;
^
>> kernel/sched/fair.c:9079:58: warning: Parameter 'p' can be declared with const [constParameter]
static int idle_cpu_without(int cpu, struct task_struct *p)
^
kernel/sched/fair.c:5149:25: warning: Uninitialized variables: cfs_rq.load, cfs_rq.nr_running, cfs_rq.h_nr_running, cfs_rq.idle_nr_running, cfs_rq.idle_h_nr_running, cfs_rq.exec_clock, cfs_rq.min_vruntime, cfs_rq.min_vruntime_copy, cfs_rq.tasks_timeline, cfs_rq.curr, cfs_rq.next, cfs_rq.last, cfs_rq.skip, cfs_rq.avg, cfs_rq.last_update_time_copy, cfs_rq.removed [uninitvar]
struct rq *rq = rq_of(cfs_rq);
^
>> kernel/sched/fair.c:6540:25: warning: Uninitialized variable: task_util [uninitvar]
asym_fits_capacity(task_util, target))
^
kernel/sched/fair.c:6529:28: note: Assuming condition is false
if (static_branch_unlikely(&sched_asym_cpucapacity)) {
^
kernel/sched/fair.c:6540:25: note: Uninitialized variable: task_util
asym_fits_capacity(task_util, target))
^
>> mm/migrate.c:916:7: warning: Redundant initialization for 'rc'. The initialized value is overwritten before it is read. [redundantInitialization]
rc = migrate_folio(mapping, dst, src, mode);
^
mm/migrate.c:906:9: note: rc is initialized
int rc = -EAGAIN;
^
mm/migrate.c:916:7: note: rc is overwritten
rc = migrate_folio(mapping, dst, src, mode);
^
>> mm/migrate.c:354:54: warning: Parameter 'mapping' can be declared with const [constParameter]
static int folio_expected_refs(struct address_space *mapping,
^
>> kernel/sched/core.c:5820:2: warning: Comparing pointers that point to different objects [comparePointers]
for_each_class(class) {
^
kernel/sched/sched.h:2243:27: note: Variable declared here.
extern struct sched_class __sched_class_highest[];
^
kernel/sched/core.c:5820:2: note: Array decayed to pointer here.
for_each_class(class) {
^
kernel/sched/sched.h:2244:27: note: Variable declared here.
extern struct sched_class __sched_class_lowest[];
^
kernel/sched/core.c:5820:2: note: Array decayed to pointer here.
for_each_class(class) {
^
kernel/sched/core.c:5820:2: note: Comparing pointers that point to different objects
for_each_class(class) {
^
>> kernel/sched/sched.h:2058:67: warning: Parameter 'p' can be declared with const [constParameter]
static inline int task_current(struct rq *rq, struct task_struct *p)
^
>> kernel/sched/core.c:2748:4: warning: Address of local auto-variable assigned to a function parameter. [autoVariables]
p->migration_pending = &my_pending;
^
kernel/sched/core.c:9268:3: warning: Comparing pointers that point to different objects [comparePointers]
for_each_class(class) {
^
kernel/sched/sched.h:2243:27: note: Variable declared here.
extern struct sched_class __sched_class_highest[];
^
kernel/sched/core.c:9268:3: note: Array decayed to pointer here.
for_each_class(class) {
^
kernel/sched/sched.h:2244:27: note: Variable declared here.
extern struct sched_class __sched_class_lowest[];
^
kernel/sched/core.c:9268:3: note: Array decayed to pointer here.
for_each_class(class) {
^
kernel/sched/core.c:9268:3: note: Comparing pointers that point to different objects
for_each_class(class) {
^
kernel/sched/core.c:9280:3: warning: Comparing pointers that point to different objects [comparePointers]
for_each_class(class) {
^
kernel/sched/sched.h:2243:27: note: Variable declared here.
extern struct sched_class __sched_class_highest[];
^
kernel/sched/core.c:9280:3: note: Array decayed to pointer here.
for_each_class(class) {
^
kernel/sched/sched.h:2244:27: note: Variable declared here.
extern struct sched_class __sched_class_lowest[];
^
kernel/sched/core.c:9280:3: note: Array decayed to pointer here.
for_each_class(class) {
^
kernel/sched/core.c:9280:3: note: Comparing pointers that point to different objects
for_each_class(class) {
^
>> mm/shmem.c:411:25: warning: Parameter 'expected' can be declared with const [constParameter]
pgoff_t index, void *expected, void *replacement)
^
mm/shmem.c:701:29: warning: Parameter 'expected' can be declared with const [constParameter]
pgoff_t index, void *expected, gfp_t gfp,
^
vim +/anon_aops +1231 fs/libfs.c
e4ab315810850b9 Kalesh Singh 2022-06-23 1228
6987843ff7e836e Al Viro 2013-10-02 1229 struct inode *alloc_anon_inode(struct super_block *s)
6987843ff7e836e Al Viro 2013-10-02 1230 {
6987843ff7e836e Al Viro 2013-10-02 @1231 static const struct address_space_operations anon_aops = {
46de8b979492e13 Matthew Wilcox (Oracle 2022-02-09 1232) .dirty_folio = noop_dirty_folio,
6987843ff7e836e Al Viro 2013-10-02 1233 };
6987843ff7e836e Al Viro 2013-10-02 1234 struct inode *inode = new_inode_pseudo(s);
6987843ff7e836e Al Viro 2013-10-02 1235
6987843ff7e836e Al Viro 2013-10-02 1236 if (!inode)
6987843ff7e836e Al Viro 2013-10-02 1237 return ERR_PTR(-ENOMEM);
6987843ff7e836e Al Viro 2013-10-02 1238
6987843ff7e836e Al Viro 2013-10-02 1239 inode->i_ino = get_next_ino();
6987843ff7e836e Al Viro 2013-10-02 1240 inode->i_mapping->a_ops = &anon_aops;
6987843ff7e836e Al Viro 2013-10-02 1241
6987843ff7e836e Al Viro 2013-10-02 1242 /*
6987843ff7e836e Al Viro 2013-10-02 1243 * Mark the inode dirty from the very beginning,
6987843ff7e836e Al Viro 2013-10-02 1244 * that way it will never be moved to the dirty
6987843ff7e836e Al Viro 2013-10-02 1245 * list because mark_inode_dirty() will think
6987843ff7e836e Al Viro 2013-10-02 1246 * that it already _is_ on the dirty list.
6987843ff7e836e Al Viro 2013-10-02 1247 */
6987843ff7e836e Al Viro 2013-10-02 1248 inode->i_state = I_DIRTY;
6987843ff7e836e Al Viro 2013-10-02 1249 inode->i_mode = S_IRUSR | S_IWUSR;
6987843ff7e836e Al Viro 2013-10-02 1250 inode->i_uid = current_fsuid();
6987843ff7e836e Al Viro 2013-10-02 1251 inode->i_gid = current_fsgid();
6987843ff7e836e Al Viro 2013-10-02 1252 inode->i_flags |= S_PRIVATE;
078cd8279e65998 Deepa Dinamani 2016-09-14 1253 inode->i_atime = inode->i_mtime = inode->i_ctime = current_time(inode);
6987843ff7e836e Al Viro 2013-10-02 1254 return inode;
6987843ff7e836e Al Viro 2013-10-02 1255 }
6987843ff7e836e Al Viro 2013-10-02 1256 EXPORT_SYMBOL(alloc_anon_inode);
1c994a0909a5565 Jeff Layton 2014-08-27 1257
:::::: The code at line 1231 was first introduced by commit
:::::: 6987843ff7e836ea65b554905aec34d2fad05c94 take anon inode allocation to libfs.c
:::::: TO: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
:::::: CC: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
--
0-DAY CI Kernel Test Service
https://01.org/lkp