Received: by 2002:a25:8b12:0:0:0:0:0 with SMTP id i18csp63640ybl; Thu, 29 Aug 2019 18:41:37 -0700 (PDT) X-Google-Smtp-Source: APXvYqxX1R/eWmWMOkl9Fuc6hfrry7CWpmgR062QolYhDYoXmfRlCltUSmvdbkOTMpx0eNZSN0Gq X-Received: by 2002:a17:90a:cc13:: with SMTP id b19mr13089007pju.117.1567129296988; Thu, 29 Aug 2019 18:41:36 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1567129296; cv=none; d=google.com; s=arc-20160816; b=RJ6E+uriJic2z+1T4ge3Hkx3+4+OBx6et5pkVNR8ullqXMSvru3sBZtlcDRC9Spd1N WJCdnXh7EHPe/buugLIJSSDbe/s5jWmYQWDblkLPFJ6p+Lkgqn0pX0Erc8NIqSdwGVKf Hio4Jiw27kYJNfnKQQ1XWeLddttPSFl0TrvVjU/pny3EBKZTQZ9QiP8UY7G9hjs9WJ72 qr5iuACJm8oFXI+3hFKOQm6l3EDz5Jhta3mZ0suYaNvIY+oQBBFLjHwHlKoD/K29p3qW SLNJUQIKXtOUkVnbuXIOHvP6y2Ayl/cKPda+neFAt4MGz/pmXWULDAozwaYDtIxDGDC+ ii5w== 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:in-reply-to :mime-version:user-agent:date:message-id:from:cc:references:to :subject; bh=EHnzCkrK/hDXIeo9ASxmR8Opu9Lf8IlG41JDIE7Qza0=; b=nWl83X66bIY2RsvOmo+kPR2PMnuxvJsilWIx8V8Yl8Sk1crBujN/0F3U9r2Cdp5IjL N8h5nCFljb+Vc+XO6dp3oMl2L9hfaZVw7Kjef1YhfLjc78rqViBEciKxCPCguTk1PVyA sv+oL/hNXcqt4ovVqq7TzqM+TwZhdxB/H8wM2wS/QHQfeddfrq1rdYfsobnsb74aiVWd 5W4SiZUY0CeHHhi9fNYtJQi8WZvLI7DB3b3gQd1m7VpW9ce/1QX12pYTS3ikGMiEn3lG 6pI4b2z9TgRNsMVnbT/BRIWPe1u95vsvTh07mdUyrBrSr7gzjmHFV+wlCfbF60U8sv8x CsNQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 126si3337452pgb.47.2019.08.29.18.41.20; Thu, 29 Aug 2019 18:41:36 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727739AbfH3BkU (ORCPT + 99 others); Thu, 29 Aug 2019 21:40:20 -0400 Received: from szxga04-in.huawei.com ([45.249.212.190]:5695 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726825AbfH3BkU (ORCPT ); Thu, 29 Aug 2019 21:40:20 -0400 Received: from DGGEMS406-HUB.china.huawei.com (unknown [172.30.72.58]) by Forcepoint Email with ESMTP id CE00A2F40678C9E7A30A; Fri, 30 Aug 2019 09:40:16 +0800 (CST) Received: from [127.0.0.1] (10.133.213.239) by DGGEMS406-HUB.china.huawei.com (10.3.19.206) with Microsoft SMTP Server id 14.3.439.0; Fri, 30 Aug 2019 09:40:12 +0800 Subject: Re: [PATCH] amd-xgbe: Fix error path in xgbe_mod_init() To: Jakub Kicinski References: <20190829024600.16052-1-yuehaibing@huawei.com> <20190829105237.196722f9@cakuba.netronome.com> CC: , , , From: Yuehaibing Message-ID: <8b480542-4ec4-03b2-4426-348ac65aa4d6@huawei.com> Date: Fri, 30 Aug 2019 09:40:11 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <20190829105237.196722f9@cakuba.netronome.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.133.213.239] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2019/8/30 1:52, Jakub Kicinski wrote: > On Thu, 29 Aug 2019 10:46:00 +0800, YueHaibing wrote: >> In xgbe_mod_init(), we should do cleanup if some error occurs >> >> Reported-by: Hulk Robot >> Fixes: efbaa828330a ("amd-xgbe: Add support to handle device renaming") >> Fixes: 47f164deab22 ("amd-xgbe: Add PCI device support") >> Signed-off-by: YueHaibing > > Looks correct. Thanks! > > For networking fixes please try to use [PATCH net] as a tag ([PATCH > net-next] for normal, non-fix patches). Ok. > >