From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id D865AA00C5; Tue, 7 Jul 2020 01:09:35 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id A86131DBD5; Tue, 7 Jul 2020 01:09:34 +0200 (CEST) Received: from out2-smtp.messagingengine.com (out2-smtp.messagingengine.com [66.111.4.26]) by dpdk.org (Postfix) with ESMTP id 800201D9DE for ; Tue, 7 Jul 2020 01:09:33 +0200 (CEST) Received: from compute7.internal (compute7.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id 0C5CE5C01F8; Mon, 6 Jul 2020 19:09:33 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute7.internal (MEProxy); Mon, 06 Jul 2020 19:09:33 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s=fm1; bh= DO7O8kgZ2aaC7uryhHRCRaLBxOC39aeVlLTmoZj16vk=; b=oo40j9p+iYyW3A1a mzKE4hOeTddH7J3b8OGJUHmVj2r/GzS07FPAlb4vgGn64R5Gp9R6jONHxKyB/7zK 7wsp9SZINUGGNfdvjSg/OP3zpPREt29q2NsbLOOBt7mLn8HmUMOfPyKKpRgnB5H0 6XRGF6/PkO5uxKkYthqV0/ZrczVid4NxFicSSIaJm8mbudJKpAgFvJhiJSaNnl0q Aq3we+/0rqpS58wrzX2PWcYoesyBZ8kNHhUF7OcMRH0GduEiHuq5aftAlIHP1xLj fL/Om3rFKfxHFWcPdlhNW/1WkBWnN06/RW93OSL/ScrvobWzHyYzWTGyXfj4l+TQ 9HK2yA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm3; bh=DO7O8kgZ2aaC7uryhHRCRaLBxOC39aeVlLTmoZj16 vk=; b=phDMy5rVrCk49IYulnoLdUNI2CZ6UGNdzfvYsTXg0mobIKzUxw0GHhzes S2mRjSLZMMehTGIIh457zpLJ+G0hjhK6B4QpuAEROhfXti/a4caCJ4++TUJbEnBf Cou+JQrilRCjup7z6c6DuDO6WEfgltEA3aYtel/CqIYKxDfYMjVCUrzjgv98dNSw d3B5ltKj4//SmMv6r/EDlSPad8eA8yBzc/8rF86zDM4rq/zfbqpfoP1obVm2IklU P+ALv8jJZTXNDPToqALdVNVihEV9DUvEQav8To+45yIp/ATeoNgXgMKVEp9yZmW5 143Aa5a/QSKJxMhPl/GnCo9PQj5tA== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduiedrudeggddulecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpefhvffufffkjghfggfgtgesthfuredttddtvdenucfhrhhomhepvfhhohhmrghs ucfoohhnjhgrlhhonhcuoehthhhomhgrshesmhhonhhjrghlohhnrdhnvghtqeenucggtf frrghtthgvrhhnpeffvdffjeeuteelfeeileduudeugfetjeelveefkeejfeeigeehteff vdekfeegudenucffohhmrghinhepughpughkrdhorhhgnecukfhppeejjedrudefgedrvd dtfedrudekgeenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhr ohhmpehthhhomhgrshesmhhonhhjrghlohhnrdhnvght X-ME-Proxy: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id 4875D328005A; Mon, 6 Jul 2020 19:09:31 -0400 (EDT) From: Thomas Monjalon To: Alan Dewar , Alan Dewar Cc: dev@dpdk.org, "Yigit, Ferruh" , "Kantecki, Tomasz" , Stephen Hemminger , "dev@dpdk.org" , "Dumitrescu, Cristian" , jasvinder.singh@intel.com, david.marchand@redhat.com, bruce.richardson@intel.com Date: Tue, 07 Jul 2020 01:09:30 +0200 Message-ID: <1642305.bAa2EqY5vZ@thomas> In-Reply-To: <3EB4FA525960D640B5BDFFD6A3D891268E87276A@IRSMSX108.ger.corp.intel.com> References: <1516032969-23894-1-git-send-email-alan.dewar@att.com> <2596922.ozHIldrUZQ@xps> <3EB4FA525960D640B5BDFFD6A3D891268E87276A@IRSMSX108.ger.corp.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH v7] sched: make RED scaling configurable X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 08/04/2019 15:29, Dumitrescu, Cristian: > From: Thomas Monjalon [mailto:thomas@monjalon.net] > > 08/04/2019 10:24, Alan Dewar: > > > On Fri, Apr 5, 2019 at 4:36 PM Ferruh Yigit wrote: > > > > On 1/16/2018 4:07 PM, alangordondewar@gmail.com wrote: > > > > > From: Alan Dewar > > > > > > > > > > The RED code stores the weighted moving average in a 32-bit integer as > > > > > a pseudo fixed-point floating number with 10 fractional bits. Twelve > > > > > other bits are used to encode the filter weight, leaving just 10 bits > > > > > for the queue length. This limits the maximum queue length supported > > > > > by RED queues to 1024 packets. > > > > > > > > > > Introduce a new API to allow the RED scaling factor to be configured > > > > > based upon maximum queue length. If this API is not called, the RED > > > > > scaling factor remains at its default value. > > > > > > > > > > Added some new RED scaling unit-tests to test with RED queue-lengths > > > > > up to 8192 packets long. > > > > > > > > > > Signed-off-by: Alan Dewar > > > > > > > > Hi Cristian, Alan, > > > > > > > > The v7 of this patch is sting without any comment for more than a year. > > > > What is the status of this patch? Is it still valid? What is blocking it? > > > > > > > > For reference patch: > > > > https://patches.dpdk.org/patch/33837/ > > > > > > We are still using this patch against DPDK 17.11 and 18.11 as part of > > > the AT&T Vyatta NOS. It is needed to make WRED queues longer than > > > 1024 packets work correctly. I'm afraid that I have no idea what is > > > holding it up from being merged. > > > > It will be in a release when it will be merged in the git tree > > dpdk-next-qos, managed by Cristian. > > I was hoping to get a review & ACK from Tomasz Kantecki, the author of the WRED code in DPDK, hence the lack of progress on this patch. It seems nobody was able to provide an feedback after two years, and it was never merged in the QoS git tree. The handling of this patch is really a shame. Alan, please rebase this patch. If nothing is wrong in CI (including ABI check), I will merge the next version.