Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New tests, minor improvements and possibility to dump registers just after start and before stop #12

Closed
wants to merge 13 commits into from

Conversation

ol-andrewr
Copy link
Contributor

@ol-andrewr ol-andrewr commented Jun 10, 2024

Add new tests on:

  • stats reset
  • Rx VLAN filter
  • Tx PVID
  • flow control set

Expectations are filled in for virtio and i40e.

Add possibility to dump device registers just after device start and before device stop. Requires TE v1.34.0 or higher.

ol-andrewr and others added 13 commits May 29, 2024 10:25
Use wrapper to always do various actions just before or after
Ethernet device stop.

Set environment variable TE_ENV_DPDK_DUMP_REGS to any value to
get and dump in logs registers just before the device stop.

Signed-off-by: Andrew Rybchenko <[email protected]>
Reviewed-by: Daniil Byshenko <[email protected]>
Add general and extended statistics reset test.

Signed-off-by: Daniil Byshenko <[email protected]>
Reviewed-by: Andrew Rybchenko <[email protected]>
Signed-off-by: Daniil Byshenko <[email protected]>
Reviewed-by: Andrew Rybchenko <[email protected]>
Add test to check port-based VLAN (PVID) insertion on transmit
configured via Tx mode and/or using rte_eth_dev_set_vlan_pvid() API.

Signed-off-by: Daniil Byshenko <[email protected]>
Reviewed-by: Andrew Rybchenko <[email protected]>
net/virtio silently ignores PVID configuration in Tx mode and does
not support runtime API to set PVID.

Signed-off-by: Andrew Rybchenko <[email protected]>
Increase Rx rings on Tester to maximum to make it likely to
receive everything sent from IUT.

If Rx ring is smaller than number of sent packets anyway, do not
check mismatch and received packets number since some packet in
the middle may be lost.

Signed-off-by: Andrew Rybchenko <[email protected]>
Update to the latest release.

Signed-off-by: Andrew Rybchenko <[email protected]>
i40e may reorder non-IP tagged and untagged packets on Rx.

Signed-off-by: Andrew Rybchenko <[email protected]>
If peer is X710 (i40e driver), its maximum Rx ring size is too
small to reliably receive that many packets.

Signed-off-by: Andrew Rybchenko <[email protected]>
Signed-off-by: Daniil Byshenko <[email protected]>
Reviewed-by: Andrew Rybchenko <[email protected]>
A number of tested PMDs do not support flow control operations.

Signed-off-by: Andrew Rybchenko <[email protected]>
i40e returns current mode on get which may differ from specified on set.

Signed-off-by: Andrew Rybchenko <[email protected]>
i40e has limitations on high/low watermark values.

Signed-off-by: Andrew Rybchenko <[email protected]>
@okt-galaktionov
Copy link
Contributor

Applied, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants