Received: by 2002:a25:1985:0:0:0:0:0 with SMTP id 127csp3647197ybz; Mon, 4 May 2020 07:07:17 -0700 (PDT) X-Google-Smtp-Source: APiQypI68sHhIxoTqZ1FJidgeTFGQOb4DVhE7MFn7F75NJvDfwsNnMyjcznI4a/8aRf9D1zgNvGl X-Received: by 2002:a17:906:1dce:: with SMTP id v14mr15395525ejh.244.1588601236911; Mon, 04 May 2020 07:07:16 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1588601236; cv=none; d=google.com; s=arc-20160816; b=N68Gw1a9kUVC+Y5xnnInvhN0waiZqfC64tqf0QWIp8oNtk0kt+4hnbZCfSftJU33R3 /jBX9gCnLhBu+nD0yIdfbL/yD4RZ2llKoo2YPucwvlvakAgTANIRpNDe2hF9rOAdVuKQ j1iSfBRyhIhtpSl1ZQLpyGQAHBPzwLxxf6kVg+wRj12bcmwqfIvwLU4ZaRnCfzMwWLmi 4gSpH52wY/Adjtzp5ijRKyZDI5r33syQCcDcK1vfFqeHaHs+LjWYqdeNhE5toHuWEWIv uF0wP9WV94chV4UXCbVQSO6p6iOvo1BTgKPNSrcos5YxvxI53Asj4Ia5KSEGkhgaKMI+ nTLw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=tCHoUYZDTjX4vaPAyNBL+EYEDPRlfwHwHf5FTbB/Lh4=; b=FMtoBR/iIdCpTF70fgG+MiBriddl44ysR+vSQm7LBrITzUf2FSqG4VVBWxicJlPUs8 xaMPqiT2PjHy7uhs4UGh2OwiBAyqPqkcAdBAQ/CNInByX/p8b8pqO82CVOO+sF6+7hle 6gOHNBTt5VzpGXlSVc/jeWfMGIUlk6BjKCIsnxqIrW68JQHfhF86eSyZF3JXqv8/KGsj QorxtInUPStCTMAUeVIBuNiU8JW7QVW2PXCGDIVAX3K6McP4YTeddVDB1/24WSgWwH5w 5dI4Nd6sIUQ7a8B3z3ZU/EIAvppWTAimHBH71dh9sw+2HNVYr6BgE50y047sta+k35fk lP/Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=KF937lRK; 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=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id g15si6928559edr.14.2020.05.04.07.06.44; Mon, 04 May 2020 07:07:16 -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; dkim=pass header.i=@gmail.com header.s=20161025 header.b=KF937lRK; 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=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728990AbgEDODy (ORCPT + 99 others); Mon, 4 May 2020 10:03:54 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48166 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728187AbgEDODy (ORCPT ); Mon, 4 May 2020 10:03:54 -0400 Received: from mail-ej1-x644.google.com (mail-ej1-x644.google.com [IPv6:2a00:1450:4864:20::644]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B4E83C061A0E; Mon, 4 May 2020 07:03:53 -0700 (PDT) Received: by mail-ej1-x644.google.com with SMTP id s9so13991955eju.1; Mon, 04 May 2020 07:03:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=tCHoUYZDTjX4vaPAyNBL+EYEDPRlfwHwHf5FTbB/Lh4=; b=KF937lRK0nNhSOVgWpMcj6ZuKQ7C+cWdzULHvmIN8/tI2pgvbinqmSxs9O7Uis2hZM Rp/YST7OxmY4ZCpDeEoX9rM4PETetPd4g1YjCX9U/iokjzZ/U558GihHLyAPaoiGBWFe 7ASSJsYTBefWrcak4ajMy5czb8VLh4sdAb7Ztsv9hNEp1MkAllURAMdI95wxD4oGFJKD 45o3L29FCXEj/Szid66lvzJHfI6AnKlPqfKYtAT6dEPz4HJAd60i8PceLYvjsDqsFRZt H1nmqCk+Wmsf86jhvtew6IYS+UIOVjqszXtyfSPo0EbnHFMNAQ8Qsqh1SiF7t+4aoIRS N3Dg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=tCHoUYZDTjX4vaPAyNBL+EYEDPRlfwHwHf5FTbB/Lh4=; b=Y9AwG4f4yYjlcoejH16dXusSI1hRLhQoV/q63uM29U21Q7OHSA9AqihND1MDV0mj3g HjNZyqH3rxGEftiPbsw/kIafSMkVU+dpWiyFD3ayEBDew5BOHxCnJhiz+2bq1uKjmF76 ApMRTQE7aeU3YwWMcYYq95tg3BsPPQsxHdFwfRufBHkoeHvWZwjWt0dkap54eRrmHJos ZiraRYltEhh797i8foF6gnsldq5hcpvxHMGG1VWAeayoCEIr7+DUDzme1tXy19kwQxQO VlXRSTojI9NrhYeJxbbSfn2f1rSrEScmXlFBCCMo1wZuX+SJyxv4ZgQr7NyGKZlYO+ls y2Vg== X-Gm-Message-State: AGi0PuZqvQnyY8D1aiFZE25NudtadTh0BM6N+CekJbAthHwJclTaaf5r Yziy7sDyNSQ2Ceq7bblfamzWOjJM5N6POCLd/ZM= X-Received: by 2002:a17:906:78c:: with SMTP id l12mr14074138ejc.189.1588601032384; Mon, 04 May 2020 07:03:52 -0700 (PDT) MIME-Version: 1.0 References: <20200504120127.4482-1-zhengdejin5@gmail.com> In-Reply-To: <20200504120127.4482-1-zhengdejin5@gmail.com> From: Vladimir Oltean Date: Mon, 4 May 2020 17:03:41 +0300 Message-ID: Subject: Re: [PATCH net v1] net: enetc: fix an issue about leak system resources To: Dejin Zheng Cc: Claudiu Manoil , "David S. Miller" , Vladimir Oltean , netdev , lkml , Andy Shevchenko Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 4 May 2020 at 15:04, Dejin Zheng wrote: > > the related system resources were not released when enetc_hw_alloc() > return error in the enetc_pci_mdio_probe(), add iounmap() for error > handling label "err_hw_alloc" to fix it. > > Fixes: 6517798dd3432a ("enetc: Make MDIO accessors more generic and export to include/linux/fsl") > Cc: Andy Shevchenko > Signed-off-by: Dejin Zheng > --- Reviewed-by: Vladimir Oltean > drivers/net/ethernet/freescale/enetc/enetc_pci_mdio.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/freescale/enetc/enetc_pci_mdio.c b/drivers/net/ethernet/freescale/enetc/enetc_pci_mdio.c > index ebc635f8a4cc..15f37c5b8dc1 100644 > --- a/drivers/net/ethernet/freescale/enetc/enetc_pci_mdio.c > +++ b/drivers/net/ethernet/freescale/enetc/enetc_pci_mdio.c > @@ -74,8 +74,8 @@ static int enetc_pci_mdio_probe(struct pci_dev *pdev, > pci_disable_device(pdev); > err_pci_enable: > err_mdiobus_alloc: > - iounmap(port_regs); > err_hw_alloc: > + iounmap(port_regs); > err_ioremap: > return err; > } > -- > 2.25.0 > Thanks! -Vladimir