[dpdk-dev] [PATCH] cmdline: increase commandline buffer size to 512
Nélio Laranjeiro
nelio.laranjeiro at 6wind.com
Thu Nov 5 17:50:44 CET 2015
On Thu, Nov 05, 2015 at 09:58:36AM +0000, John McNamara wrote:
> Increase commandline buffer size to 512 to allow for longer
> commandlines required by testpmd when dealing with flow
> director and IPv6 addresses.
>
> Signed-off-by: John McNamara <john.mcnamara at intel.com>
> ---
> lib/librte_cmdline/cmdline_rdline.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/librte_cmdline/cmdline_rdline.h b/lib/librte_cmdline/cmdline_rdline.h
> index b9aad9b..72e2dad 100644
> --- a/lib/librte_cmdline/cmdline_rdline.h
> +++ b/lib/librte_cmdline/cmdline_rdline.h
> @@ -93,7 +93,7 @@ extern "C" {
> #endif
>
> /* configuration */
> -#define RDLINE_BUF_SIZE 256
> +#define RDLINE_BUF_SIZE 512
> #define RDLINE_PROMPT_SIZE 32
> #define RDLINE_VT100_BUF_SIZE 8
> #define RDLINE_HISTORY_BUF_SIZE BUFSIZ
> --
> 1.8.1.4
>
I have faced the same issue. For some commands this value is still too
low. For example, RETA update can not be used, the command line limits
he configuration to 34 entries.
See http://dpdk.org/ml/archives/dev/2015-October/024681.html
and particularly
http://dpdk.org/ml/archives/dev/2015-October/024682.html
It was too late to announce an ABI change so we left out the cmdline
size update.
Olivier, Thomas, what do you think about this? It seems this is really
blocking.
--
Nélio Laranjeiro
6WIND
More information about the dev
mailing list