Received: by 2002:a25:e74b:0:0:0:0:0 with SMTP id e72csp1049521ybh; Mon, 13 Jul 2020 08:06:31 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwIM/kxlEppzqjVrCsaN1VLfG+cLz96pSUCscltO1cpaKHA48gkcHdx5bZdYIRLlEiGRUd6 X-Received: by 2002:a50:e801:: with SMTP id e1mr89750126edn.251.1594652791774; Mon, 13 Jul 2020 08:06:31 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1594652791; cv=none; d=google.com; s=arc-20160816; b=qOOqkHS46bzrHb6L/UnvTHXrK4PYl5Ld4hmCFavQ1VDf5ts+zAIf5toZWd5r9PnSY/ U0c6Aq1jwfSar01ia6rvrZhiSz+wr/5+gd86RMy1txHxfJfFh0F1nTn6kkya9CSn/UL5 NLAwnM546DJo4Y6qc9BPtdFo2cZatw8SCKXrSAjKzDvqs8183jaJmh8Z+rHgSj1NEUT7 otOoBKgcr+yZ0yzzfsxICfvcLnv0PRlyeHga3fhpTM5Qtju8wqF52+i0q/ZXppzHSHGp 8+39EJhmx7miITpoap+xEtX6eoItH361H50WZHHcpBz68Ills9CQhFFtcV54YElMcjo9 KYeA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:message-id:date:to:cc:from :subject:references:in-reply-to:content-transfer-encoding :mime-version; bh=bf/vWU12KtrrXLvPassFN4Tcf71C+6f5omz0MMQgeoA=; b=sJs7Ct1MstWqUlOnrNVlY3dWlg7eqd8sxKo/JMHuZMaaAurnq1iyLhDj3vene5C9n3 rLmXBTC9ww2KMfwAdLtZ9/W8/HMmcU2c+DAiDxAIVmgXetdetgAOeDHbqCOxfXza2AO2 9OJY0JrRRnN1HscyUM5HuV8Bg887U544RHhXBIY1+X1AzEo3SFr6R0gc33NHikpQjeiD QoozWZHp1mRwJgCVoQ+MiqSR0+++WNyNulNKRKx+hrxPUoe1l4VbWLzsMNof6ZdCwnkc idu4lIzNjnwmuKnzqS9+mfIL66gHmwJtwR4GZnHGrcM2sdTo4QEFr0bZvX5e80w355Xp /Ssw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id i26si9057291ejk.576.2020.07.13.08.06.08; Mon, 13 Jul 2020 08:06:31 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729851AbgGMPDX convert rfc822-to-8bit (ORCPT + 99 others); Mon, 13 Jul 2020 11:03:23 -0400 Received: from mail.fireflyinternet.com ([77.68.26.236]:62706 "EHLO fireflyinternet.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1729593AbgGMPDX (ORCPT ); Mon, 13 Jul 2020 11:03:23 -0400 X-Greylist: delayed 962 seconds by postgrey-1.27 at vger.kernel.org; Mon, 13 Jul 2020 11:03:22 EDT X-Default-Received-SPF: pass (skip=forwardok (res=PASS)) x-ip-name=78.156.65.138; Received: from localhost (unverified [78.156.65.138]) by fireflyinternet.com (Firefly Internet (M1)) with ESMTP (TLS) id 21804978-1500050 for multiple; Mon, 13 Jul 2020 15:46:35 +0100 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT In-Reply-To: <20200713142551.423649-1-colin.king@canonical.com> References: <20200713142551.423649-1-colin.king@canonical.com> Subject: Re: [PATCH][next] drm/i915/selftest: fix an error return path where err is not being set From: Chris Wilson Cc: kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org To: Colin King , Daniel Vetter , David Airlie , Jani Nikula , Joonas Lahtinen , Rodrigo Vivi , dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org Date: Mon, 13 Jul 2020 15:46:35 +0100 Message-ID: <159465159599.23097.17150763550392763778@build.alporthouse.com> User-Agent: alot/0.9 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Quoting Colin King (2020-07-13 15:25:51) > From: Colin Ian King > > There is an error condition where err is not being set and an uninitialized > garbage value in err is being returned. Fix this by assigning err to an > appropriate error return value before taking the error exit path. > > Addresses-Coverity: ("Uninitialized scalar value") > Fixes: ed2690a9ca89 ("drm/i915/selftest: Check that GPR are restored across noa_wait") > Signed-off-by: Colin Ian King Thanks, pushed. -Chris