From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ed1-f48.google.com (mail-ed1-f48.google.com [209.85.208.48]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2560B1DE881 for ; Wed, 18 Dec 2024 17:51:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.48 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1734544320; cv=none; b=O+ZCV+IdjXLmK5jvkGxPb7dCKugaOp+bLw7hLuYE+g4HsPjubbIJGDNfKeAvt+bkRHxJmw8ePn9X3frbKNUvY4lez3N5R+ckCXaeXnE2Lu34JIhfhZhBwvgvmzS33mjKgSWzF+1Sr1UN4EaIPBu2MSzFyDwZ9o99PZk4qgY7AQo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1734544320; c=relaxed/simple; bh=2So0nul1mz0/2K4Ol25NHSdvjvCJf1yUkrkJLZzp1jk=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=JD/NR42nIuI6mwuRIUmWmoNaiLeVYtrWjJWAo0OIaa27qpdgwLofQlzpQjj5q+ZlAsh1BAFuENQ2xVzJSgGLL+qMUY7RKWhgyAusBWJgPX/EIVZ3fRnQSL6Upr4r8XKIFLmPIFSdRRo5pGwFqjOtzZk7Y2UP7Pn28WzlkNhgTdk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=eVM2C8Oe; arc=none smtp.client-ip=209.85.208.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="eVM2C8Oe" Received: by mail-ed1-f48.google.com with SMTP id 4fb4d7f45d1cf-5d3f28a4fccso10124701a12.2 for ; Wed, 18 Dec 2024 09:51:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1734544317; x=1735149117; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=WuTI8JBt/EZ+l8PFms5uxro4CHQ/QT9ZzGcBaY71A3k=; b=eVM2C8OeHoEjet1QRZ2YNkAD/OewghUWzeX+AWuzumsyaBz+5Xvvn3/KfTbiBVWxin 6vzP6NXiX4taQC1GFnLKb/982MWXadCAyerVBa0MdkMKN5NeiUTOdlWk6lzJtzl0800K gZwfFNH71MAGSwFVWXWyVamphZW+RcUVY+bPlGPJDwlnc/D65286cJ+TvcqJMIOv5699 IL7AbPJWew/kUrnC/W1hVt9k/xg3MGySrdKqX5lxXB3ywI9nWn77NWaBmKBO6/+r5o0b oeTfRhH4zlYqJ/tvi3TR+4/AFrz8sFUaD9CJW7n9i680u2VQaVG6jRM1w2tk/bsIjReZ PCrA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1734544317; x=1735149117; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=WuTI8JBt/EZ+l8PFms5uxro4CHQ/QT9ZzGcBaY71A3k=; b=kE7KXCBmg8+aK/do7Hr906XjPtNERUb9bmhVmsO3lQ0lZMkVc24kiQXqRSLz/2K8zj 2UXjO5at6zSiqoVUH5pJGMG4vMHirGAWvHx79S5ZGtXoDaMDlMqfyp90BaMXx/MlaRgq HT7gyGR6HheE4gk8CR/urlMoJ8FKYKaVn5p+GcZqojqwOgd6QSNTiG1BiI9ZXjmlnLup uic7ch9/+Ke5YCtcm3DyEK+4iQV+mAHiRnFN7ObWUmVFUAh+Yaya/YQyQpMiUBuBmu+3 FdnyDhzMWUdS8PUm32nGASZ+i5J+bi+DiFAS966Qz1EGNTfoG6wc4XsGwuTmYuvlUZSX 6VUg== X-Forwarded-Encrypted: i=1; AJvYcCViGv5tF+wTzE41WDzXx0DYJOi7126+rSSMkk9g4TCQQNpMhF/D3BOHCCjyN8e9x8yhA9FjFOCeUgfSekg=@vger.kernel.org X-Gm-Message-State: AOJu0Yzi5EpJF8qI75iz3eqZNLrllZhytXrelVqcyUluo+wJ8FcP+ZGW d3Of6lHwTlOW0Qd/5p7UqQJI5nu4u0xGOstQJUPs0J450sSa79LWMX8ozhWUTRE= X-Gm-Gg: ASbGncu/cm3Fj8OQZgt9PY3sFpGQBt41hlFummE2fOikjlJzxgHMv+XvzBIOiyGVVBm oiU/axKnSmCQOXmGru9Nrtriy06/kQDEEnsTbXAPgZAihy7f2BGCFxdM5aAMEu4qxVnnDTE5ePp z1MEAmtq/MhicUObQJC1dOidhEvSZNVWwQq5hTmi1DHXpfRHnPffFhbAHg8H1T4Kscfy6B21U0P wzMDmFWqx9OoFxEsLjQBRbgzB5hBZ0YSfulZ7NGvMvKbo9xLZ80J6y4IDzTOA== X-Google-Smtp-Source: AGHT+IH1c0SiqJ9S0vW3GjpTrilhfirwlgVe7LxPXAmcmvdcFWW1srbLC4xqZpvUu0l/1LZk2/U+3g== X-Received: by 2002:a05:6402:3581:b0:5d7:ea25:c72f with SMTP id 4fb4d7f45d1cf-5d7ee3ff3acmr4017187a12.25.1734544317452; Wed, 18 Dec 2024 09:51:57 -0800 (PST) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-5d652ad1b31sm5553348a12.33.2024.12.18.09.51.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 18 Dec 2024 09:51:57 -0800 (PST) Date: Wed, 18 Dec 2024 20:51:53 +0300 From: Dan Carpenter To: Markus Elfring Cc: Harshit Mogalapalli , netdev@vger.kernel.org, Andrew Lunn , Bharat Bhushan , "David S. Miller" , Eric Dumazet , Geethasowjanya Akula , Hariprasad Kelam , Jakub Kicinski , Paolo Abeni , Simon Horman , Subbaraya Sundeep Bhatta , Sunil Goutham , linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org, Dan Carpenter , Przemek Kitszel Subject: Re: [PATCH net v2 1/2] octeontx2-pf: fix netdev memory leak in rvu_rep_create() Message-ID: <116fc5cb-cc46-4e0f-9990-499ae7ef90ee@stanley.mountain> References: <20241217052326.1086191-1-harshit.m.mogalapalli@oracle.com> <8d54b21b-7ca9-4126-ba13-bbd333d6ba0c@web.de> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <8d54b21b-7ca9-4126-ba13-bbd333d6ba0c@web.de> On Wed, Dec 18, 2024 at 06:38:25PM +0100, Markus Elfring wrote: > > When rvu_rep_devlink_port_register() fails, free_netdev(ndev) for this > > incomplete iteration before going to "exit:" label. > > > … > > +++ b/drivers/net/ethernet/marvell/octeontx2/nic/rep.c > > @@ -680,8 +680,10 @@ int rvu_rep_create(struct otx2_nic *priv, struct netlink_ext_ack *extack) > > ndev->features |= ndev->hw_features; > > eth_hw_addr_random(ndev); > > err = rvu_rep_devlink_port_register(rep); > > - if (err) > > + if (err) { > > + free_netdev(ndev); > > goto exit; > > + } > > > > SET_NETDEV_DEVLINK_PORT(ndev, &rep->dl_port); > … > > I suggest to add another jump target instead so that a bit of exception handling > can be better reused at the end of this function implementation. > When you're cleaning up from inside a loop, then the best practices is to clean up partial iterations before the goto and then clean up whole iterations in the unwind ladder. So this patch is better the way that Harshit his written it. regards, dan carpenter