[dpdk-dev] [PATCH] testpmd: check return value of rte_eth_dev_vlan_filter()
Thomas Monjalon
thomas.monjalon at 6wind.com
Tue Jan 27 11:33:20 CET 2015
Hi Michal,
2015-01-23 11:43, Michal Jastrzebski:
> This patch modifies testpmd behavior when setting:
> rx_vlan add all vf_port (enabling all vlanids
> to be passed thru rx filter on VF).
> Rx_vlan_all_filter_set() function,
> checks if the next vlanid can be enabled by the driver.
> Number of vlanids is limited by the NIC and thus the NIC
> do not allow to enable more vlanids than it can allocate
> in VFTA table.
>
> Signed-off by: Michal Jastrzebski <michalx.k.jastrzebski at intel.com>
checkpatch is not happy because you forgot an hyphen.
> @@ -1667,8 +1668,9 @@ rx_vlan_all_filter_set(portid_t port_id, int on)
>
> if (port_id_is_invalid(port_id))
> return;
> - for (vlan_id = 0; vlan_id < 4096; vlan_id++)
> - rx_vft_set(port_id, vlan_id, on);
> + for (vlan_id = 0; vlan_id < 4096; vlan_id++) {
> + if ( rx_vft_set(port_id, vlan_id, on) ) break;
Again, checkpatch does not like this line.
And more importantly, you make it clear that sometimes we cannot enable all
vlans and return no error.
So I wonder how is it documented in the testpmd help?
Thanks
--
Thomas
More information about the dev
mailing list