app/testpmd: disable Rx VLAN offloads by default
authorMoti Haimovsky <motih@mellanox.com>
Wed, 31 Jan 2018 17:32:03 +0000 (19:32 +0200)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 31 Jan 2018 20:45:33 +0000 (21:45 +0100)
commit8b9bd0efe0b6920a08e28eebacf2bb916bdf5653
tree67b2835a480339af10e5ac6f5e855f4f79c69919
parent0e45c64d27977aa5e4f8745563aec17ee0e9f89b
app/testpmd: disable Rx VLAN offloads by default

Removed the hardcoded preconfigured Rx VLAN offload configuration
from testpmd and changed the Rx offload command line parameters from
disable to enable.

It has been decided by the Technical Board that testers who wish to
use these offloads will now have to explicitly write them in the
command-line when running testpmd.
The agreement is to keep two exceptions enabled by default in 18.02:
Rx CRC strip and Tx fast free.

Motivation:
Some PMDs such at the mlx4 may not implement all the offloads.
After the offload API rework assuming no offload is enabled by default,
  commit ce17eddefc20 ("ethdev: introduce Rx queue offloads API")
  commit cba7f53b717d ("ethdev: introduce Tx queue offloads API") trying
to enable a not supported offload is clearly an error which will cause
configuration failing.

Considering that testpmd is an application to test the PMD, it should
not fail on a configuration which was not explicitly requested.
The behavior of this test application is then turned to an opt-in
model.

Signed-off-by: Moti Haimovsky <motih@mellanox.com>
Acked-by: Thomas Monjalon <thomas@monjalon.net>
12 files changed:
app/test-pmd/parameters.c
app/test-pmd/testpmd.c
doc/guides/contributing/documentation.rst
doc/guides/howto/lm_virtio_vhost_user.rst
doc/guides/howto/pvp_reference_benchmark.rst
doc/guides/howto/virtio_user_as_exceptional_path.rst
doc/guides/nics/mrvl.rst
doc/guides/nics/octeontx.rst
doc/guides/nics/thunderx.rst
doc/guides/nics/virtio.rst
doc/guides/testpmd_app_ug/run_app.rst
doc/guides/testpmd_app_ug/testpmd_funcs.rst