Received: by 2002:a25:e74b:0:0:0:0:0 with SMTP id e72csp1024428ybh; Mon, 13 Jul 2020 07:28:45 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz9cSJBvy5aeopEwXO3w5rjGwxT4e85MUx+vnhmZFS+Xly0CkEe6mFXMPRcNLPyJMQLrL7Y X-Received: by 2002:a17:906:c187:: with SMTP id g7mr98047ejz.108.1594650525705; Mon, 13 Jul 2020 07:28:45 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1594650525; cv=none; d=google.com; s=arc-20160816; b=B1ap0JtA4GqUV4czbhpzoB4GzbQmOjE8QmCkUuQ5WrYdH3q4Cx+FInu79gX4+xwXYM lgdKIxW6O6o98Zqy0sb3nZTB+H0SLPxRHVHteLLVN4uGT8exXbmCJCJ3fYmwgCMVQN2T p4HZNaykxWQHASMubbbA0dfZcT2GrAMh1vtVM736H/aX1REt0dbJ5WtTe1gfVzA2GqkS 6eG/o1883MsODXuBkGwFhmRzhyniRQ1sIXtIEn+EyUoNmD41FNWREBnAHV75m9hgMIf1 RZJX6pEICkQAIyfSrHEPr8nbm3ZowawcJb2RSHIuHA1cQjiOzVKzDX8Rlhn4gInjeGgr Qrog== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=mskRoX/YZfZ3knoMmLUUWSbP6vcQzuVH6EPjo6LcugY=; b=LbFnWfCPOmXi4GcEBeDc0L84ZNCqsZiEMPJofYUUPaVrwdCU4uevHlEIVHW+Y0oHYJ pNSlY73vG8TKD+EeavUIzxj95AU8LqxwugKl/AgjSP1JtOj40aDzTfMGN9ajeyHXTE6P Y+hlGjyOAZmZ4vYSbWn7d/RbXTvc7whLH5OBDO65TtdAYgc3Fj/I6z5wqA/SypFf//p7 TEC4qOrOKdF6rUWe1WvXDEa5UaEmKzZdw6DGrSOHzcCZyG2Ng/U1Rer2vE46NkCJ49a2 Mk5DgfYcwXLuWMMml1Py8sjJvUdNg0dol9/5ABZ3KzdDjHF4GLo7GIJDaCVvnZE6g10/ cMmg== 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=canonical.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id rv2si9195218ejb.428.2020.07.13.07.28.21; Mon, 13 Jul 2020 07:28:45 -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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=canonical.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729979AbgGMOZ6 (ORCPT + 99 others); Mon, 13 Jul 2020 10:25:58 -0400 Received: from youngberry.canonical.com ([91.189.89.112]:57301 "EHLO youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729863AbgGMOZ6 (ORCPT ); Mon, 13 Jul 2020 10:25:58 -0400 Received: from 1.general.cking.uk.vpn ([10.172.193.212] helo=localhost) by youngberry.canonical.com with esmtpsa (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.86_2) (envelope-from ) id 1juzOy-0000pJ-0Y; Mon, 13 Jul 2020 14:25:52 +0000 From: Colin King To: Jani Nikula , Joonas Lahtinen , Rodrigo Vivi , David Airlie , Daniel Vetter , Chris Wilson , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org Cc: kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH][next] drm/i915/selftest: fix an error return path where err is not being set Date: Mon, 13 Jul 2020 15:25:51 +0100 Message-Id: <20200713142551.423649-1-colin.king@canonical.com> X-Mailer: git-send-email 2.27.0 MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 --- drivers/gpu/drm/i915/selftests/i915_perf.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/selftests/i915_perf.c b/drivers/gpu/drm/i915/selftests/i915_perf.c index deb6dec1b5ab..7aa73bb03381 100644 --- a/drivers/gpu/drm/i915/selftests/i915_perf.c +++ b/drivers/gpu/drm/i915/selftests/i915_perf.c @@ -329,6 +329,7 @@ static int live_noa_gpr(void *arg) cs = intel_ring_begin(rq, 2 * 32 + 2); if (IS_ERR(cs)) { i915_request_add(rq); + err = PTR_ERR(cs); goto out_rq; } -- 2.27.0