diff options
author | Steven <sluong@cisco.com> | 2018-05-11 11:06:23 -0700 |
---|---|---|
committer | Damjan Marion <dmarion.lists@gmail.com> | 2018-05-25 11:46:05 +0000 |
commit | 0d88301a576191a0e330e539cf1dcb3837ee1bf6 (patch) | |
tree | 2bf42dd2935161d6cdb982371015e94a52d74e62 /test/test_bond.py | |
parent | 0053de63ec4bf8b9bce7817f1b61c9791baf6c26 (diff) |
bond: performance harvesting
- hash is great. But it is a bit too slow for the DP. Use direct array indexing
to quickly retrieve the slave interface.
- the algorithm used by flow hash is great. But it is a bit too slow for the DP.
Use l2_hash_hash() extracted from lb_hash.h which ECMP is using. It makes use
of intrinsic crc32 instruction set.
- shortcut modulo arithmetic when the operand is 2**x (where x up to 4) to
avoid division instruction.
- special case for link count == 1 in bond_tx_fn()
- use clib_mem_unaligned to access data for the packet to avoid alignment error
- Fix some typos for packet tracing.
Change-Id: I8eae3ad497061c5473aa675ba894ee0211120d25
Signed-off-by: Steven <sluong@cisco.com>
Diffstat (limited to 'test/test_bond.py')
-rw-r--r-- | test/test_bond.py | 14 |
1 files changed, 3 insertions, 11 deletions
diff --git a/test/test_bond.py b/test/test_bond.py index b54a1f1deb5..e354601ca91 100644 --- a/test/test_bond.py +++ b/test/test_bond.py @@ -136,19 +136,11 @@ class TestBondInterface(VppTestCase): found = 1 self.assertEqual(found, 1) - # pg0 tx bytes = 142 - intfs = self.vapi.cli("show interface pg0").split("\n") - found = 0 - for intf in intfs: - if "tx bytes" in intf and "142" in intf: - found = 1 - self.assertEqual(found, 1) - - # pg0 tx bytes = 142 - intfs = self.vapi.cli("show interface pg1").split("\n") + # BondEthernet0 tx bytes = 284 + intfs = self.vapi.cli("show interface BondEthernet0").split("\n") found = 0 for intf in intfs: - if "tx bytes" in intf and "142" in intf: + if "tx bytes" in intf and "284" in intf: found = 1 self.assertEqual(found, 1) |