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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8B31BC04A95 for ; Tue, 25 Oct 2022 08:21:38 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id E4CE08E0003; Tue, 25 Oct 2022 04:21:37 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id DFAA08E0001; Tue, 25 Oct 2022 04:21:37 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id CE9958E0003; Tue, 25 Oct 2022 04:21:37 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id BE1D28E0001 for ; Tue, 25 Oct 2022 04:21:37 -0400 (EDT) Received: from smtpin12.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id 7D2B81A02AB for ; Tue, 25 Oct 2022 08:21:37 +0000 (UTC) X-FDA: 80058777834.12.B527AF9 Received: from pandora.armlinux.org.uk (pandora.armlinux.org.uk [78.32.30.218]) by imf18.hostedemail.com (Postfix) with ESMTP id 221F41C0005 for ; Tue, 25 Oct 2022 08:21:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc: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-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=EtmojnUqnCVKWOh8C7J92DZrsq9/z0GK3KV0rT9mLpw=; b=BiKMTB/c8/919DvH6LZjZyf2T7 Lk+Oc0WQgSHrU043Y+cnUTrcCAWFFIRwbP/gwShZJOH7Ph6ZjNkE4YzQLwl+rZ6FQMYOZVZ1KaAAv JrHnpzzBk58E6Bw1q1q6MogLgb0j7g2H4c3zZg9FWX17eaRq/RM83PQCbfJMvNXNS9KzKq0aINVsM KaygbGcRwn6F39U9DxcVpgeF99Nlne7F879RubTg63DmK7uq37EnDvG5KYZFfCKozoHb2sgk7eO4l lwO6xacAFy+7uJjlKmYCqjbyjVIn9IsBwGV8SHQv4hpDF3zC3DsR7nT9nOHu1R4lpXKIo3O5UJycU fdMcQzOw==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:34940) by pandora.armlinux.org.uk with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1onFBQ-0004Dt-7i; Tue, 25 Oct 2022 09:21:12 +0100 Received: from linux by shell.armlinux.org.uk with local (Exim 4.94.2) (envelope-from ) id 1onFBL-0007xV-IX; Tue, 25 Oct 2022 09:21:07 +0100 Date: Tue, 25 Oct 2022 09:21:07 +0100 From: "Russell King (Oracle)" To: Jakub Kicinski Cc: kernel test robot , Andrew Morton , ntfs3@lists.linux.dev, netdev@vger.kernel.org, linux-parisc@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, amd-gfx@lists.freedesktop.org, Linux Memory Management List Subject: Re: [linux-next:master] BUILD SUCCESS WITH WARNING 76cf65d1377f733af1e2a55233e3353ffa577f54 Message-ID: References: <6356c451.pwLIF+9EvDUrDjTY%lkp@intel.com> <20221024145527.0eff7844@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20221024145527.0eff7844@kernel.org> ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1666686096; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=EtmojnUqnCVKWOh8C7J92DZrsq9/z0GK3KV0rT9mLpw=; b=JWET5/XMMLYgtsFzKANpLRaSLMonSkY4h89cAKcVKB6SI0GroZrWbGwYNO77yxE/9Td9kD ZHKwXa0TjPcKjA19nbsgR/SYlqfu+tS59PYRM9XhxXFUHsr2taMkFNFEnvUWX9Uoq0xjOf sXqoMjClK1dzurXIi+2ukXoOI1k6oOw= ARC-Authentication-Results: i=1; imf18.hostedemail.com; dkim=pass header.d=armlinux.org.uk header.s=pandora-2019 header.b="BiKMTB/c"; spf=none (imf18.hostedemail.com: domain of "linux+linux-mm=kvack.org@armlinux.org.uk" has no SPF policy when checking 78.32.30.218) smtp.mailfrom="linux+linux-mm=kvack.org@armlinux.org.uk"; dmarc=pass (policy=none) header.from=armlinux.org.uk ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1666686096; a=rsa-sha256; cv=none; b=fUrBeTpYVVn3TBrYqxt5Ewvd1Nnpi7z+BSLSvOdYUHBMg7D7d+b4JRiEVP8ze100Qf4rJs K3xt413Qx6noSc18zW/+nuXvJLCIVo4wjuYIiOPW2+X14N2WHaFIDBnUtcCeCydQoWuh04 ljtejBVjrahsOZGZvTJNY0gLHoMEnMw= X-Rspamd-Queue-Id: 221F41C0005 Authentication-Results: imf18.hostedemail.com; dkim=pass header.d=armlinux.org.uk header.s=pandora-2019 header.b="BiKMTB/c"; spf=none (imf18.hostedemail.com: domain of "linux+linux-mm=kvack.org@armlinux.org.uk" has no SPF policy when checking 78.32.30.218) smtp.mailfrom="linux+linux-mm=kvack.org@armlinux.org.uk"; dmarc=pass (policy=none) header.from=armlinux.org.uk X-Rspam-User: X-Rspamd-Server: rspam10 X-Stat-Signature: x3ofi649ctiuw87fqb4j73bm6cc7ucwr X-HE-Tag: 1666686095-229554 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000001, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Mon, Oct 24, 2022 at 02:55:27PM -0700, Jakub Kicinski wrote: > On Tue, 25 Oct 2022 00:58:57 +0800 kernel test robot wrote: > > drivers/net/phy/phylink.c:588 phylink_validate_mask_caps() warn: variable dereferenced before check 'state' (see line 583) > > Hi Russell, I think the warning is semi-legit. Your commit f392a1846489 > ("net: phylink: provide phylink_validate_mask_caps() helper") added an > if (state) before defer'ing state but it's already deref'ed higher up > so can't be null. Not me, Sean. My original implementation of phylink_validate_mask_caps() doesn't know anything about rate matching, so my version didn't have this issue. Sean's version of my patch (which is what was submitted) added the dereference that causes this, so, it's up to Sean to figure out a fix - but he reading his follow up to the build bot's message, he seems to be passing it over to me to fix! I've got other issues to be worked on right now, and have no time to spare to fix other people's mistakes. Sorry. You can't always rely on the apparent author mentioned in the commit to be the actual person responsible for the changes in a patch. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!