From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.5 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C6C2BC04EB9 for ; Fri, 30 Nov 2018 02:12:32 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 95A512086B for ; Fri, 30 Nov 2018 02:12:32 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="SxRiJSly"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=canb.auug.org.au header.i=@canb.auug.org.au header.b="r/vvtPsj" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 95A512086B Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=canb.auug.org.au Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: MIME-Version:References:In-Reply-To:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=lNHKlvgdko8OoG9o4uhrn8BUl0zm9/F4cbwP7+kuCeQ=; b=SxRiJSlyR4IJYAaYi7X92Vcs1 EAa5BRHgO3iT0eZ4a04g1DM/M5yLdP1natrUWNO4FGh3K7TWs/vSIBURqbibrikH8OXl5tOdv0xO+ iiMtEcpU/nVoVOUBJeoBGB/G63Oy6nWZhAd9S9ZVxrNydgEUpH/BFUuBrcLF8JCObwcymqybww6cz VPh9TSq/bpGC0o6YoVtI8xN4ClV3L0kSJMVztAlUhbOj+PDMr1SdOCyu/dRiwPo0+bp7Gqsp6dJbk Ft7s9mhk3Mrw3ki8Ut+shDx6n/Mrdl1lf+Y8UqkS+xYPr8XPr1OwYO0iZdmp/i3av0p6bFBHRvBi1 cu4ExoJtQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1gSYI1-0006sf-Nf; Fri, 30 Nov 2018 02:12:21 +0000 Received: from ozlabs.org ([203.11.71.1]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1gSYHx-0006rr-8H for linux-arm-kernel@lists.infradead.org; Fri, 30 Nov 2018 02:12:19 +0000 Received: from authenticated.ozlabs.org (localhost [127.0.0.1]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPSA id 435dDy0SsCz9s8F; Fri, 30 Nov 2018 13:12:01 +1100 (AEDT) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=canb.auug.org.au DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=canb.auug.org.au; s=201702; t=1543543922; bh=Z1fZ2Crm3Ouc+ihJ2dxfkNmVWcS/1CQm7fPkFhSuP4Q=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=r/vvtPsjr+2r0Z2Awhe2iQKjpEvD8ZGreKvyaCDN1PRNCJnLJq9/Z17b3F8hnsNxc MVV3+MDnEq0NWw0agJU34MKDt2lKg0XS/OTyQqf37KwvyguyLYfxe862vnupppTOr5 YcmLoEdiWz0k/+8m3W3ztgeRn+msjUdwKI+sE/w4OmaRT1zst8aclE03AMa1zwXKyR PkUMK/CAv7JhPLmDjFidvR6rUVdBXg8gSCAMTYiS+GVGZKyLwxIDRh2A0B4HGYYOSD TAoVfZBIA1vLgkFo/sLoMX0O2gKD7bc6RJWXVMy2bV7AfHRBnhk3WIOq0xKrHxPhwW 8spDm/JfyINyw== Date: Fri, 30 Nov 2018 13:12:01 +1100 From: Stephen Rothwell To: Andy Gross Subject: Re: [PATCH] drm: msm: Check cmd_db_read_aux_data() for failure Message-ID: <20181130131201.6fc91dcd@canb.auug.org.au> In-Reply-To: <20181129224234.GD32306@hector.attlocal.net> References: <20181129163826.178670-1-swboyd@chromium.org> <20181129223600.GC32306@hector.attlocal.net> <20181129224234.GD32306@hector.attlocal.net> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20181129_181218_090826_B45DD34C X-CRM114-Status: GOOD ( 12.81 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Rob Clark , kbuild test robot , linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, Evan Green , Stephen Boyd , Jordan Crouse , Bjorn Andersson , linux-arm-kernel@lists.infradead.org Content-Type: multipart/mixed; boundary="===============7933059520780426553==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============7933059520780426553== Content-Type: multipart/signed; micalg=pgp-sha256; boundary="Sig_/uTnMOcHV6Tw7mxJCfUrnTQC"; protocol="application/pgp-signature" --Sig_/uTnMOcHV6Tw7mxJCfUrnTQC Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Hi Andy, On Thu, 29 Nov 2018 16:42:34 -0600 Andy Gross wrote: > > On Thu, Nov 29, 2018 at 04:36:00PM -0600, Andy Gross wrote: > > On Thu, Nov 29, 2018 at 08:38:26AM -0800, Stephen Boyd wrote: =20 > > > We need to check the call to cmd_db_read_aux_data() for the error cas= e, > > > so that we don't continue and use potentially uninitialized values for > > > 'pri_count' and 'sec_count'. Otherwise, we get the following compiler > > > warnings: > > >=20 > > > drivers/gpu/drm/msm/adreno/a6xx_gmu.c: In function 'a6xx_gmu_rpmh_= arc_votes_init.isra.12': > > > drivers/gpu/drm/msm/adreno/a6xx_gmu.c:943:12: warning: 'pri_count'= is used uninitialized in this function [-Wuninitialized] > > > pri_count >>=3D 1; > > > ^~~ > > > drivers/gpu/drm/msm/adreno/a6xx_gmu.c:948:12: warning: 'sec_count'= may be used uninitialized in this function > > > [-Wmaybe-uninitialized] > > > sec_count >>=3D 1; > > > ^~~ > > >=20 > > > Reported-by: Stephen Rothwell > > > Reported-by: kbuild test robot > > > Cc: Jordan Crouse > > > Cc: Bjorn Andersson > > > Cc: Evan Green > > > Cc: Jordan Crouse > > > Cc: Rob Clark > > > Fixes: ed3cafa79ea7 ("soc: qcom: cmd-db: Stop memcpy()ing in cmd_db_r= ead_aux_data()") > > > Signed-off-by: Stephen Boyd =20 >=20 > On second thought, this needs to go through drm tree. Why? The commit that introduces the warnings is only in the qcom tree ... --=20 Cheers, Stephen Rothwell --Sig_/uTnMOcHV6Tw7mxJCfUrnTQC Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEENIC96giZ81tWdLgKAVBC80lX0GwFAlwAnHEACgkQAVBC80lX 0Gzm5QgAoS5hbEm2DH6OlupYdISzBShylvu98CN3kkwvnYnWUaqc8FZ0RcVRRbdN T9E2kznxCpp0dyan/ZEw+5w/RtbKqurg33bGnVEGljZ9XdskEtc3bv/wVqGIvNrN wWN0pss6sjSi9r/SezK46NTq/9deR6Dm41xvRv6M62yesTSjp1/BssvGPQGm6SLQ GIjXIR1wYy1jFKZGL3UgruwCHg1wb3TuvSsZmWhoIXjSdjR0Q1zhg27YWeDSit70 sihng+14RHSXHYbOYbPD0EdVcl+TLqlR8uMoyE8bVUPucJkhgCTVz2ba3wqPTw0O EbJdRFtVUaohn2apbPGjpxY9CGEhCw== =SDX4 -----END PGP SIGNATURE----- --Sig_/uTnMOcHV6Tw7mxJCfUrnTQC-- --===============7933059520780426553== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============7933059520780426553==--