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 21015A053C for ; Fri, 31 Jul 2020 17:46:45 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id D9B891BFDC; Fri, 31 Jul 2020 17:46:44 +0200 (CEST) Received: from mail-pf1-f196.google.com (mail-pf1-f196.google.com [209.85.210.196]) by dpdk.org (Postfix) with ESMTP id 83653262E for ; Fri, 31 Jul 2020 17:46:42 +0200 (CEST) Received: by mail-pf1-f196.google.com with SMTP id u185so14688591pfu.1 for ; Fri, 31 Jul 2020 08:46:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=z4z3TlioFW0hWI8WXV3usenyNGtDfyUlxTwtFS79y7A=; b=k56yp+OfBnl/svr+IMc4uKrBE68D6Xg0JhPJF2ooHn8SzHXcimfkoNMNFlVaKg9XIa WG2lS7/T9nWa8Vyr0Bm2QQFUvKvfi1IUhhZVFhOK4le+JKkjvHCrwrWpazHt5V52uo8R y6WJksXQ7Ak0PKmFpcpJU4DAimtTO+z61RjXGfi8nsmfbnvDhUtFl3c0cRsdmFlGtcLJ Wsg0hHcUcTF64G47OQlkBOCP1ozGSNuLTp5mJF46jEgGEWW4ySlvW3n7qD1Yz9zqkoz+ gWhwnD38SVhrOvGFozAj1BbTDegg0Ubyj5ND2K3v8tV2ArcED9cU4oM6v7A+k+Y3ns1P SL3Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=z4z3TlioFW0hWI8WXV3usenyNGtDfyUlxTwtFS79y7A=; b=fIv+uplftcW3P3MJp1BVa8HGcG6yMVykGNjKLzALosSaCu1mJ+QtSO4CRsu0THK680 SAC0dxSxACQGW/A518GAnO0g19XVAVDTuccZ//G2DL47pQTP/SV0/aTCHq1fv/OYhqhr 9x3IXKVsf+K2d11lUL5JJlpFWVOIC5l1Azz1xdVuPs3+6zBuhcJG9mkDYjPMx/8oshZn vZLo9ZJhkUlecbqkILHjJ/eaow+EGxHgfOIde9W9GFA2ZlNLvlR6GVtV9qJ7FE09qdv9 ladmLKPf3YUJkByixzAjhRc5LjLyxtVfpYuImbwo7INX6O6P23ltaYATJsyF3/VdbaTu fjvg== X-Gm-Message-State: AOAM533AFQtMR/Zt+BlCd1R2jZi+g2RSTudknMLedCVIy9g/SGUrfGfG yr5jh+ecFJAuNzhRwbqFwWqjWw== X-Google-Smtp-Source: ABdhPJwSE1MmLYr/zZNHiyGFZjpu1d7KON+NdD/xsnbctB1ffYXAvWdAcg/A0obkDzJ6MO6wQHk70w== X-Received: by 2002:a65:4502:: with SMTP id n2mr4290868pgq.132.1596210401461; Fri, 31 Jul 2020 08:46:41 -0700 (PDT) Received: from hermes.lan (204-195-22-127.wavecable.com. [204.195.22.127]) by smtp.gmail.com with ESMTPSA id f29sm11083537pga.59.2020.07.31.08.46.40 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 31 Jul 2020 08:46:41 -0700 (PDT) Date: Fri, 31 Jul 2020 08:46:38 -0700 From: Stephen Hemminger To: Olivier Matz Cc: David Marchand , Wisam Jaddo , dev , Raslan , Thomas Monjalon , "Iremonger, Bernard" , dpdk stable Message-ID: <20200731084638.304ef20c@hermes.lan> In-Reply-To: <20200731125516.GG5869@platinum> References: <1582204709-7992-1-git-send-email-wisamm@mellanox.com> <1582210386-27331-1-git-send-email-wisamm@mellanox.com> <20200731125516.GG5869@platinum> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-stable] [dpdk-dev] [PATCH v3] cmdline: increase maximum line length X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" On Fri, 31 Jul 2020 14:55:16 +0200 Olivier Matz wrote: > Hi, > > Ressurecting this old thread. > > On Sat, Feb 22, 2020 at 04:28:15PM +0100, David Marchand wrote: > > This patch is flagged as an ABI breakage: > > https://travis-ci.com/ovsrobot/dpdk/jobs/289313318#L2273 > > > > In case we want this fix for 20.11, should we do a deprecation notice > in 20.08? > > > Olivier > > > > > > On Thu, Feb 20, 2020 at 3:53 PM Wisam Jaddo wrote: > > > > > > This increase due to the usage of cmdline in dpdk applications > > > as config commands such as testpmd do for rte_flow rules creation. > > > > > > The current size of buffer is not enough to fill > > > many cases of rte_flow commands validation/creation. > > > > > > rte_flow now can have outer items, inner items, modify > > > actions, meta data actions, duplicate action, fate action and > > > more in one single rte flow, thus 512 char will not be enough > > > to validate such rte flow rules. > > > > > > Such change shouldn't affect the memory since the cmdline > > > reading again using the same buffer. > > > > I don't get your point here. The cmdline is a awkward user API. Thomas wanted to replace it but it seems to have gotten nowhere. Agree that having something dynamic would be best, Something based of getline() or editline (readline).