diff options
| author |   Florin Coras <fcoras@cisco.com> | 2019-08-31 09:46:24 -0700 |
|---|---|---|
| committer | Florin Coras <fcoras@cisco.com> | 2019-08-31 09:46:24 -0700 |
| commit | 4e1fcf49057b6bd9b4a3a32a8da20a721536c247 (patch) | |
| tree | f05dcb7b9f81b914cfef05c109bc00bcb658a148 /src/vnet/tcp | |
| parent | 5c89fbf28665d13a0052fc88288c1524e0e93918 (diff) | |
tcp: fix rx min/max fifo size parsing
Type: fix
Change-Id: Iff4a13da654dbdf7f08bb6d0743b466a1e6be01b
Signed-off-by: Florin Coras <fcoras@cisco.com>
Diffstat (limited to 'src/vnet/tcp')
| -rw-r--r-- | src/vnet/tcp/tcp.c | 27 |
1 files changed, 14 insertions, 13 deletions
diff --git a/src/vnet/tcp/tcp.c b/src/vnet/tcp/tcp.c index cd7b9e80c7a..bd3a4c1160a 100644 --- a/src/vnet/tcp/tcp.c +++ b/src/vnet/tcp/tcp.c @@ -1678,6 +1678,7 @@ unformat_tcp_cc_algo_cfg (unformat_input_t * input, va_list * va) static clib_error_t * tcp_config_fn (vlib_main_t * vm, unformat_input_t * input) { + uword memory_size; while (unformat_check_input (input) != UNFORMAT_END_OF_INPUT) { if (unformat (input, "preallocated-connections %d", @@ -1690,14 +1691,14 @@ tcp_config_fn (vlib_main_t * vm, unformat_input_t * input) &tcp_cfg.buffer_fail_fraction)) ; else if (unformat (input, "max-rx-fifo %U", unformat_memory_size, - &tcp_cfg.max_rx_fifo)) - ; + &memory_size)) + tcp_cfg.max_rx_fifo = memory_size; else if (unformat (input, "min-rx-fifo %U", unformat_memory_size, - &tcp_cfg.min_rx_fifo)) - ; - else if (unformat (input, "mtu %d", &tcp_cfg.default_mtu)) + &memory_size)) + tcp_cfg.min_rx_fifo = memory_size; + else if (unformat (input, "mtu %u", &tcp_cfg.default_mtu)) ; - else if (unformat (input, "initial-cwnd-multiplier %d", + else if (unformat (input, "initial-cwnd-multiplier %u", &tcp_cfg.initial_cwnd_multiplier)) ; else if (unformat (input, "no-tx-pacing")) @@ -1707,19 +1708,19 @@ tcp_config_fn (vlib_main_t * vm, unformat_input_t * input) ; else if (unformat (input, "%U", unformat_tcp_cc_algo_cfg)) ; - else if (unformat (input, "closewait-time %d", &tcp_cfg.closewait_time)) + else if (unformat (input, "closewait-time %u", &tcp_cfg.closewait_time)) tcp_cfg.closewait_time /= TCP_TIMER_TICK; - else if (unformat (input, "timewait-time %d", &tcp_cfg.timewait_time)) + else if (unformat (input, "timewait-time %u", &tcp_cfg.timewait_time)) tcp_cfg.timewait_time /= TCP_TIMER_TICK; - else if (unformat (input, "finwait1-time %d", &tcp_cfg.finwait1_time)) + else if (unformat (input, "finwait1-time %u", &tcp_cfg.finwait1_time)) tcp_cfg.finwait1_time /= TCP_TIMER_TICK; - else if (unformat (input, "finwait2-time %d", &tcp_cfg.finwait2_time)) + else if (unformat (input, "finwait2-time %u", &tcp_cfg.finwait2_time)) tcp_cfg.finwait2_time /= TCP_TIMER_TICK; - else if (unformat (input, "lastack-time %d", &tcp_cfg.lastack_time)) + else if (unformat (input, "lastack-time %u", &tcp_cfg.lastack_time)) tcp_cfg.lastack_time /= TCP_TIMER_TICK; - else if (unformat (input, "closing-time %d", &tcp_cfg.closing_time)) + else if (unformat (input, "closing-time %u", &tcp_cfg.closing_time)) tcp_cfg.closing_time /= TCP_TIMER_TICK; - else if (unformat (input, "cleanup-time %d", &tcp_cfg.cleanup_time)) + else if (unformat (input, "cleanup-time %u", &tcp_cfg.cleanup_time)) tcp_cfg.cleanup_time /= TCP_TIMER_TICK; else return clib_error_return (0, "unknown input `%U'", |