From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 35557A0093; Mon, 7 Mar 2022 19:35:12 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id ECB1640688; Mon, 7 Mar 2022 19:35:11 +0100 (CET) Received: from out5-smtp.messagingengine.com (out5-smtp.messagingengine.com [66.111.4.29]) by mails.dpdk.org (Postfix) with ESMTP id 50BF94014E for ; Mon, 7 Mar 2022 19:35:11 +0100 (CET) Received: from compute2.internal (compute2.nyi.internal [10.202.2.46]) by mailout.nyi.internal (Postfix) with ESMTP id B5B995C0185; Mon, 7 Mar 2022 13:35:10 -0500 (EST) Received: from mailfrontend1 ([10.202.2.162]) by compute2.internal (MEProxy); Mon, 07 Mar 2022 13:35:10 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= cc:cc:content-transfer-encoding:content-type:date:date:from:from :in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:sender:subject:subject:to:to; s=fm1; bh=voMNr47WoJDxjn rTyaFKMm5b2iuIc9b0mUU1arE0q8U=; b=lnE2Wkg+piZ8gdkHO/4gPCyKVV3Mzj E/UexOp/yYIXkc1/gIkvVeS0blT8gfoBtT5i6J+WcBpFHnAwWQWB9lubEPcpzKsJ QYEehCSB1oAMVpoiN0oICKBhk7WieDcR7atKSPuxgrn8W9VPLW9gFNZDws4jpjgC PXtZ258lOuAkPUN6K0z+0Ew3ow9wsmHw8Nggi1IiRLXiNcCSt5zqtaSdJ05KBMUr 4M3yz0txfa2QyNH+Er4g2axlVcA2/JxvMg6kM5nuRbawobzlkb+b0ZQWt7GIK1sD 3C07ILwAt+Xqmoyv+sx70Gf09sLcV6JCDD/1O21vU4OzvMMvr1i2zd+Q== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:date:date:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm2; bh=voMNr47WoJDxjnrTyaFKMm5b2iuIc9b0mUU1arE0q 8U=; b=G8VETr4uEGbtJ3MWlAuLZBv0H7Fkr+M1QoVgT6kmZuglnTAVa5y2l2vRm yvF2OczVFmIN4/Qbi8EzgmL4qVL5uUuhSgn5xf/vPFomUKBJ8LNe9i8FX3JS8c8L 0C09Uj0iapi6dCa3sDDrQwq0WsLG/FwxzhjTFYF1Arz2gcYzHCl93H8Rff3YMmkF 6TXVbWcf73mj0LAa8u/Ng1dakMeSM1tSwolT4Y0aSlzhcIm3voCFwjNffKIrrZe1 RpjXck5ZFcsIOumBDoKqUAcxHjcRzYw3aIFnKF5VQPAh9p8VF5K0zoWknv78fPCt mo4+z5fpJWMArbGxi3DYoFdplnCrQ== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvvddruddugedguddufecutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfgh necuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd enucfjughrpefhvffufffkjghfggfgtgesthfuredttddtvdenucfhrhhomhepvfhhohhm rghsucfoohhnjhgrlhhonhcuoehthhhomhgrshesmhhonhhjrghlohhnrdhnvghtqeenuc ggtffrrghtthgvrhhnpedugefgvdefudfftdefgeelgffhueekgfffhfeujedtteeutdej ueeiiedvffegheenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfh hrohhmpehthhhomhgrshesmhhonhhjrghlohhnrdhnvght X-ME-Proxy: Received: by mail.messagingengine.com (Postfix) with ESMTPA; Mon, 7 Mar 2022 13:35:08 -0500 (EST) From: Thomas Monjalon To: Weiguo Li Cc: cristian.dumitrescu@intel.com, dev@dpdk.org, jasvinder.singh@intel.com, Stephen Hemminger Subject: Re: [PATCH v5] sched: remove useless malloc in pie data init Date: Mon, 07 Mar 2022 19:35:07 +0100 Message-ID: <39592950.J2Yia2DhmK@thomas> In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org 01/03/2022 21:32, Weiguo Li: > 'rte_pie_rt_data_init(NULL)' is not expected, and it's ought to > fail when this happen. The malloc inside the function didn't work. > So remove the malloc otherwise will lead to a memory leak. > > Fixes: 44c730b0e37971 ("sched: add PIE based congestion management") > > Signed-off-by: Weiguo Li > Acked-by: Stephen Hemminger > Acked-by: Cristian Dumitrescu > --- > v5: > * fix compile error > v4: > * replace all assignments to zero by a memset. > v3: > * fix typo > v2: > * revise according to Stephen's suggestion. Applied, thanks.