[dpdk-dev] [PATCH v3] testpmd: fix wrong prompt in tx_vlan set command handler

2016-02-04 Thread Wang Xiao W
When using testpmd, sometimes we forget the right order of port_id and vid in "tx_vlan set (port_id) vlan_id[, vlan_id_outer]\n" command, and input "tx_vlan set 51 0", we'll get a strange prompt saying "Error, as QinQ has been enabled.". In cmd_tx_vlan_set_parsed function, the first thing we do is

[dpdk-dev] [PATCH v3] testpmd: fix wrong prompt in tx_vlan set command handler

2016-02-04 Thread Wang Xiao W
v3: * Reworded the patch subject and commit message, added detail for reproducing the bug. * Reordered the definitions of cmd_tx_vlan_set_portid and cmd_tx_vlan_set_vlanid to keep in consistency with cmd_tx_vlan_set.tokens. v2: * Removed the bug fix unrelated code change to make this patch a

[dpdk-dev] [PATCH v3] testpmd: fix wrong prompt in tx_vlan set command handler

2016-02-04 Thread Bruce Richardson
On Thu, Feb 04, 2016 at 04:27:32PM +0800, Wang Xiao W wrote: > When using testpmd, sometimes we forget the right order of port_id and > vid in "tx_vlan set (port_id) vlan_id[, vlan_id_outer]\n" command, and > input "tx_vlan set 51 0", we'll get a strange prompt saying "Error, as > QinQ has been ena