From mboxrd@z Thu Jan  1 00:00:00 1970
Content-Type: multipart/mixed; boundary="===============3244805960554475945=="
MIME-Version: 1.0
From: Huang Rui <ray.huang@amd.com>
To: kbuild-all@lists.01.org
Subject: Re: [rui:linux-next 9/13] drivers/acpi/scan.c:2152 acpi_walk_dep_device_list() error: uninitialized symbol 'ret'.
Date: Tue, 29 Jun 2021 15:27:32 +0800
Message-ID: <20210629072732.GA1433809@hr-amd>
In-Reply-To: <9d52e69c-91d1-0f2a-d2ff-54ed03f8092e@gmail.com>
List-Id: <oe-kbuild-all.lists.linux.dev>

--===============3244805960554475945==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

On Mon, Jun 28, 2021 at 06:04:04PM +0800, Daniel Scally wrote:
> Hi Dan
> =

> On 28/06/2021 10:06, Dan Carpenter wrote:
> > tree:   https://nam11.safelinks.protection.outlook.com/?url=3Dhttps%3A%=
2F%2Fgit.kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Frui%2Flinux.git&am=
p;data=3D04%7C01%7Cray.huang%40amd.com%7C1d88cef62d9d441306c908d93a1c1180%7=
C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637604715076721040%7CUnknown%7C=
TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0=
%3D%7C2000&amp;sdata=3DntL36nVMHIl3%2Bb22kd5TrQ2RKfL%2F2DVYHi4t05iWbxU%3D&a=
mp;reserved=3D0 linux-next
> > head:   efd4c04742d30bb71022f50cd5451a0fbe52aa56
> > commit: 018f81b667154ad8ee504bc36f52dc26f4f1cb87 [9/13] ACPI: scan: Ext=
end acpi_walk_dep_device_list()
> > config: i386-randconfig-m021-20210627 (attached as .config)
> > compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
> >
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp@intel.com>
> > Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
> >
> > smatch warnings:
> > drivers/acpi/scan.c:2152 acpi_walk_dep_device_list() error: uninitializ=
ed symbol 'ret'.
> >
> > vim +/ret +2152 drivers/acpi/scan.c
> >
> > 018f81b667154a Daniel Scally 2021-06-03  2135  int acpi_walk_dep_device=
_list(acpi_handle handle,
> > 018f81b667154a Daniel Scally 2021-06-03  2136  			      int (*callback)=
(struct acpi_dep_data *, void *),
> > 018f81b667154a Daniel Scally 2021-06-03  2137  			      void *data)
> > 018f81b667154a Daniel Scally 2021-06-03  2138  {
> > 018f81b667154a Daniel Scally 2021-06-03  2139  	struct acpi_dep_data *d=
ep, *tmp;
> > 018f81b667154a Daniel Scally 2021-06-03  2140  	int ret;
> > 018f81b667154a Daniel Scally 2021-06-03  2141  =

> > 018f81b667154a Daniel Scally 2021-06-03  2142  	mutex_lock(&acpi_dep_li=
st_lock);
> > 018f81b667154a Daniel Scally 2021-06-03  2143  	list_for_each_entry_saf=
e(dep, tmp, &acpi_dep_list, node) {
> > 018f81b667154a Daniel Scally 2021-06-03  2144  		if (dep->supplier =3D=
=3D handle) {
> >
> > I haven't looked at the context, but it feels like we should to handle
> > the case where handle is not found.
> =

> =

> Agreed; Colin spotted this one and patched it already actually:
> =

> =

> https://nam11.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Flore.=
kernel.org%2Flkml%2F20210609173312.298414-1-colin.king%40canonical.com%2F&a=
mp;data=3D04%7C01%7Cray.huang%40amd.com%7C1d88cef62d9d441306c908d93a1c1180%=
7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637604715076721040%7CUnknown%7=
CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn=
0%3D%7C2000&amp;sdata=3DOQVHDDDhJ2H0388gcBEjsAXidc85BWK50xVDVz9eoPI%3D&amp;=
reserved=3D0

Thanks for the reporting, I synced up an old branch of linux-pm. :-)

Will update it.

Thanks,
Ray

> =

> >
> > 018f81b667154a Daniel Scally 2021-06-03  2145  			ret =3D callback(dep,=
 data);
> > 018f81b667154a Daniel Scally 2021-06-03  2146  			if (ret)
> > 018f81b667154a Daniel Scally 2021-06-03  2147  				break;
> > 40e7fcb19293cb Lan Tianyu    2014-11-23  2148  		}
> > 40e7fcb19293cb Lan Tianyu    2014-11-23  2149  	}
> > 40e7fcb19293cb Lan Tianyu    2014-11-23  2150  	mutex_unlock(&acpi_dep_=
list_lock);
> > 018f81b667154a Daniel Scally 2021-06-03  2151  =

> > 018f81b667154a Daniel Scally 2021-06-03 @2152  	return ret > 0 ? 0 : re=
t;
> > 40e7fcb19293cb Lan Tianyu    2014-11-23  2153  }
> >
> > ---
> > 0-DAY CI Kernel Test Service, Intel Corporation
> > https://nam11.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Flis=
ts.01.org%2Fhyperkitty%2Flist%2Fkbuild-all%40lists.01.org&amp;data=3D04%7C0=
1%7Cray.huang%40amd.com%7C1d88cef62d9d441306c908d93a1c1180%7C3dd8961fe4884e=
608e11a82d994e183d%7C0%7C0%7C637604715076721040%7CUnknown%7CTWFpbGZsb3d8eyJ=
WIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C2000&amp;=
sdata=3DEJ8dqlXrHCQ1PZVI3%2Behz5R8LQD2kDSsfLDtg%2BJqZU4%3D&amp;reserved=3D0
> >
--===============3244805960554475945==--