Received: by 2002:a25:ef43:0:0:0:0:0 with SMTP id w3csp947048ybm; Wed, 27 May 2020 11:53:35 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzhsibGigoA7VMJSLPC+CIVRfficnIwpLqm6jcQXgeMGywyOJxg4lQXHmLYEuSQfDzMyJqj X-Received: by 2002:a17:907:1002:: with SMTP id ox2mr7127558ejb.189.1590605615352; Wed, 27 May 2020 11:53:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1590605615; cv=none; d=google.com; s=arc-20160816; b=mSbIrdwyYB2ZEeO/NoYTmkBTGl9DXPuihqEB4ZmtmtPebYec/s2LMQcrmiv5VGn2lp 6g9vyv3JzrlvvTTqQ/AGRK0VNjSWEJBDoB2PWgPlesfu/qIdQuWJLV7IcJ1QOgjZ7CEs eohir/6Odtwbp/CKVaZenxtSRNT9UoKxWS0K01qhTsg471ZdckJYIlvVTxQ/HDEXtw1b qyeitW6CTU9gBGrwUJozktYCseoMnFBxq4msQO0n4NrR1npSXqHHEXn02hvDU0t/QtSn E+Jume6GK0ALGFEvC81ok5fBtXN+pciolQqa6sQSCz+hHq8NRFBS5SdQ/eiZqCzHL2wY jviQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:date:user-agent:message-id:subject:cc:to :from:references:in-reply-to:content-transfer-encoding:mime-version; bh=e3YHmRufcrqSlzQgHC6cz+DlHU9q6CHdm7bFCDfOl8c=; b=i6QaN63UpU45N9jNGFvMM2i7grAut5PJfjlXKLMH4ODQPv1HNT3IspjKbTvDC15oMT 5M8M8HFi/7xaE+6WCAuepMm3mtUYFKccoCnZ1Jx83B6pK5YN/kkqANJzvo22uULwR7D2 oIYL2wGyfMt+eF7lomJhH21LSqiCPJdlHZPTqg6dF+vYzQL1Q6k/NH3gdza210CqdPYM RtRmtjdQVmBsxtV/brJju64Ro8k2rXoNcTefNQ0vngnatxyoff7nn3HYk39KCRbARN04 gl7pOll+URUQnVaVwc3Cd80jSjqV99FTzZzW00C0djSPG9stT6r9zIuFCgUvstrA2SuC jmKA== 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 ch4si2226814edb.65.2020.05.27.11.53.12; Wed, 27 May 2020 11:53:35 -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 S1730413AbgE0Pn6 convert rfc822-to-8bit (ORCPT + 99 others); Wed, 27 May 2020 11:43:58 -0400 Received: from mail.fireflyinternet.com ([109.228.58.192]:59692 "EHLO fireflyinternet.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1730386AbgE0Pn6 (ORCPT ); Wed, 27 May 2020 11:43:58 -0400 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 21311296-1500050 for multiple; Wed, 27 May 2020 16:43:53 +0100 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT In-Reply-To: <20200527140526.1458215-2-arnd@arndb.de> References: <20200527140526.1458215-1-arnd@arndb.de> <20200527140526.1458215-2-arnd@arndb.de> From: Chris Wilson To: Arnd Bergmann , Daniel Vetter , David Airlie , Jani Nikula , John Harrison , Joonas Lahtinen , Rodrigo Vivi , Tvrtko Ursulin Cc: Arnd Bergmann , Matthew Auld , Mika Kuoppala , Daniele Ceraolo Spurio , Lionel Landwerlin , "Robert M. Fosha" , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/3] drm/i915: work around false-positive maybe-uninitialized warning Message-ID: <159059423232.30979.11698525690770673738@build.alporthouse.com> User-Agent: alot/0.8.1 Date: Wed, 27 May 2020 16:43:52 +0100 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Quoting Arnd Bergmann (2020-05-27 15:05:09) > gcc-9 gets confused by the code flow in check_dirty_whitelist: > > drivers/gpu/drm/i915/gt/selftest_workarounds.c: In function 'check_dirty_whitelist': > drivers/gpu/drm/i915/gt/selftest_workarounds.c:492:17: error: 'rsvd' may be used uninitialized in this function [-Werror=maybe-uninitialized] > > I could not figure out a good way to do this in a way that gcc > understands better, so initialize the variable to zero, as last > resort. Does it look neater if we initialise it as a local? No. Reviewed-by: Chris Wilson -Chris