lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <202211220541.Ck9NkitJ-lkp@intel.com>
Date:   Tue, 22 Nov 2022 10:11:05 +0300
From:   Dan Carpenter <error27@...il.com>
To:     oe-kbuild@...ts.linux.dev, Randy Dunlap <rdunlap@...radead.org>
Cc:     lkp@...el.com, oe-kbuild-all@...ts.linux.dev,
        linux-kernel@...r.kernel.org, Palmer Dabbelt <palmer@...osinc.com>,
        Anup Patel <anup@...infault.org>
Subject: drivers/cpuidle/cpuidle-riscv-sbi.c:506 sbi_genpd_probe() warn:
 missing error code 'ret'

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   eb7081409f94a9a8608593d0fb63a1aa3d6f95d8
commit: f81f7861ee2aaa6f652f18e8f622547bdd379724 cpuidle: riscv: support non-SMP config
date:   7 months ago
config: riscv-randconfig-m031-20221121
compiler: riscv64-linux-gcc (GCC) 12.1.0

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@...el.com>
| Reported-by: Dan Carpenter <error27@...il.com>

smatch warnings:
drivers/cpuidle/cpuidle-riscv-sbi.c:506 sbi_genpd_probe() warn: missing error code 'ret'

vim +/ret +506 drivers/cpuidle/cpuidle-riscv-sbi.c

6abf32f1d9c500 Anup Patel 2022-02-10  481  static int sbi_genpd_probe(struct device_node *np)
6abf32f1d9c500 Anup Patel 2022-02-10  482  {
6abf32f1d9c500 Anup Patel 2022-02-10  483  	struct device_node *node;
6abf32f1d9c500 Anup Patel 2022-02-10  484  	int ret = 0, pd_count = 0;
6abf32f1d9c500 Anup Patel 2022-02-10  485  
6abf32f1d9c500 Anup Patel 2022-02-10  486  	if (!np)
6abf32f1d9c500 Anup Patel 2022-02-10  487  		return -ENODEV;
6abf32f1d9c500 Anup Patel 2022-02-10  488  
6abf32f1d9c500 Anup Patel 2022-02-10  489  	/*
6abf32f1d9c500 Anup Patel 2022-02-10  490  	 * Parse child nodes for the "#power-domain-cells" property and
6abf32f1d9c500 Anup Patel 2022-02-10  491  	 * initialize a genpd/genpd-of-provider pair when it's found.
6abf32f1d9c500 Anup Patel 2022-02-10  492  	 */
6abf32f1d9c500 Anup Patel 2022-02-10  493  	for_each_child_of_node(np, node) {
6abf32f1d9c500 Anup Patel 2022-02-10  494  		if (!of_find_property(node, "#power-domain-cells", NULL))
6abf32f1d9c500 Anup Patel 2022-02-10  495  			continue;
6abf32f1d9c500 Anup Patel 2022-02-10  496  
6abf32f1d9c500 Anup Patel 2022-02-10  497  		ret = sbi_pd_init(node);
6abf32f1d9c500 Anup Patel 2022-02-10  498  		if (ret)
6abf32f1d9c500 Anup Patel 2022-02-10  499  			goto put_node;
6abf32f1d9c500 Anup Patel 2022-02-10  500  
6abf32f1d9c500 Anup Patel 2022-02-10  501  		pd_count++;
6abf32f1d9c500 Anup Patel 2022-02-10  502  	}
6abf32f1d9c500 Anup Patel 2022-02-10  503  
6abf32f1d9c500 Anup Patel 2022-02-10  504  	/* Bail out if not using the hierarchical CPU topology. */
6abf32f1d9c500 Anup Patel 2022-02-10  505  	if (!pd_count)
6abf32f1d9c500 Anup Patel 2022-02-10 @506  		goto no_pd;

Error code?

6abf32f1d9c500 Anup Patel 2022-02-10  507  
6abf32f1d9c500 Anup Patel 2022-02-10  508  	/* Link genpd masters/subdomains to model the CPU topology. */
6abf32f1d9c500 Anup Patel 2022-02-10  509  	ret = dt_idle_pd_init_topology(np);
6abf32f1d9c500 Anup Patel 2022-02-10  510  	if (ret)
6abf32f1d9c500 Anup Patel 2022-02-10  511  		goto remove_pd;
6abf32f1d9c500 Anup Patel 2022-02-10  512  
6abf32f1d9c500 Anup Patel 2022-02-10  513  	return 0;
6abf32f1d9c500 Anup Patel 2022-02-10  514  
6abf32f1d9c500 Anup Patel 2022-02-10  515  put_node:
6abf32f1d9c500 Anup Patel 2022-02-10  516  	of_node_put(node);
6abf32f1d9c500 Anup Patel 2022-02-10  517  remove_pd:
6abf32f1d9c500 Anup Patel 2022-02-10  518  	sbi_pd_remove();
6abf32f1d9c500 Anup Patel 2022-02-10  519  	pr_err("failed to create CPU PM domains ret=%d\n", ret);
6abf32f1d9c500 Anup Patel 2022-02-10  520  no_pd:
6abf32f1d9c500 Anup Patel 2022-02-10  521  	return ret;
6abf32f1d9c500 Anup Patel 2022-02-10  522  }

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ