| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
| |
It's pretty common to want to omit statistics from output, to make it
easier to read. This commit adds an ovs-ofctl option to make that easy.
A lot of the OVS internal tests could use this, too, in place of
ofctl_strip. This commit adopts it for a subset.
Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Aaron Conole <aconole@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
HRW algorithm uses hash function which is dependent from the build
environment. This leads to constant fails of the testsuite
with CFLAGS='-march=native' if CPU supports hashing instructions:
[-------------------------------------------------------------------]
./bundle.at:233: ovs-appctl ofproto/trace br0 \
'in_port=LOCAL,dl_src=50:54:00:00:00:05,dl_dst=50:54:00:00:00:06'
stdout:
Bridge: br0
Flow: in_port=LOCAL,vlan_tci=0x0000,dl_src=50:54:00:00:00:05,\
dl_dst=50:54:00:00:00:06,dl_type=0x0000
Rule: table=0 cookie=0
OpenFlow actions=bundle(eth_src,50,hrw,ofport,slaves:1,2)
Final flow: unchanged
Megaflow: recirc_id=0,in_port=LOCAL,dl_src=50:54:00:00:00:05, \
dl_type=0x0000
Datapath actions: 2
./bundle.at:234: tail -1 stdout
--- -
+++ /testsuite.dir/at-groups/85/stdout
@@ -1,2 +1,2 @@
-Datapath actions: 1
+Datapath actions: 2
[-------------------------------------------------------------------]
Using of 'active_backup' algorithm will help to avoid such issues.
CC: Thadeu Lima de Souza Cascardo <cascardo@redhat.com>
Fixes: 63460a30c53e ("tests/bundle: test bundle action with ports up and down")
Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also, add the keyword bundle_action to all the tests in bundle.at,
distinguishing it from OF bundles.
It came to my attention recently that bundle_load will load 0xFFFF in case all
the slaves are down, as bundle_execute will return OFPP_NONE.
As I noticed this was not explicitly tested, not even for the bundle action, I
thought it would be nice to do it as a way of documenting this behavior.
Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@redhat.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
|
|
|
|
|
|
|
|
|
| |
Explain what this test is doing, and check that the decoded action can
be re-encoded and dumped back out of OVS.
Suggested-by: Ben Pfaff <blp@ovn.org>
Signed-off-by: Joe Stringer <joe@ovn.org>
Acked-by: Ben Pfaff <blp@ovn.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a test which causes internal reallocation of the ofpacts buffer,
followed by a large bundle action which should cause a subsequent
reallocation while decoding slave ports. Running this test under
valgrind reveals the issue below, which is fixed in the following
commit.
Invalid read of size 4
at 0x4CED87: decode_bundle (ofp-actions.c:1253)
by 0x4CEDFC: decode_NXAST_RAW_BUNDLE (ofp-actions.c:1272)
by 0x4DBDE6: ofpact_decode (ofp-actions.inc2:3765)
by 0x4D6914: ofpacts_decode (ofp-actions.c:5735)
by 0x4D6A3D: ofpacts_pull_openflow_actions__ (ofp-actions.c:5772)
by 0x4D74F3: ofpacts_pull_openflow_instructions (ofp-actions.c:6352)
by 0x4F59FA: ofputil_decode_flow_mod (ofp-util.c:1704)
by 0x4EAD18: ofp_print_flow_mod (ofp-print.c:786)
by 0x4F0711: ofp_to_string__ (ofp-print.c:3220)
by 0x4F0D98: ofp_to_string (ofp-print.c:3453)
by 0x5486B3: do_recv (vconn.c:644)
by 0x548498: vconn_recv (vconn.c:598)
by 0x524582: rconn_recv (rconn.c:703)
by 0x45DA61: ofconn_run (connmgr.c:1370)
by 0x45B3B4: connmgr_run (connmgr.c:323)
by 0x41D1E8: ofproto_run (ofproto.c:1762)
by 0x40CEE0: bridge_run__ (bridge.c:2885)
by 0x40D093: bridge_run (bridge.c:2940)
by 0x412F7E: main (ovs-vswitchd.c:120)
Address 0x66aa460 is 1,152 bytes inside a block of size 1,184 free'd
at 0x4C2AF2E: realloc (vg_replace_malloc.c:692)
by 0x543D27: xrealloc (util.c:123)
by 0x5089EF: ofpbuf_resize__ (ofpbuf.c:243)
by 0x508B81: ofpbuf_prealloc_tailroom (ofpbuf.c:290)
by 0x508D5C: ofpbuf_put_uninit (ofpbuf.c:364)
by 0x508DEF: ofpbuf_put (ofpbuf.c:387)
by 0x4CED7D: decode_bundle (ofp-actions.c:1255)
by 0x4CEDFC: decode_NXAST_RAW_BUNDLE (ofp-actions.c:1272)
by 0x4DBDE6: ofpact_decode (ofp-actions.inc2:3765)
by 0x4D6914: ofpacts_decode (ofp-actions.c:5735)
by 0x4D6A3D: ofpacts_pull_openflow_actions__ (ofp-actions.c:5772)
by 0x4D74F3: ofpacts_pull_openflow_instructions (ofp-actions.c:6352)
by 0x4F59FA: ofputil_decode_flow_mod (ofp-util.c:1704)
by 0x4EAD18: ofp_print_flow_mod (ofp-print.c:786)
by 0x4F0711: ofp_to_string__ (ofp-print.c:3220)
by 0x4F0D98: ofp_to_string (ofp-print.c:3453)
by 0x5486B3: do_recv (vconn.c:644)
by 0x548498: vconn_recv (vconn.c:598)
by 0x524582: rconn_recv (rconn.c:703)
by 0x45DA61: ofconn_run (connmgr.c:1370)
Signed-off-by: Joe Stringer <joe@ovn.org>
Acked-by: Jarno Rajahalme <jarno@ovn.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Improve link speed by linking 29 test programs into ovstest.
On my machine, running the following command against a fully
built tree:
$ touch lib/random.c; time make
Improve the overall build time from 7 seconds to 3.5 seconds.
Signed-off-by: Andy Zhou <azhou@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
|
|
|
|
|
| |
The bundle_load action behaves the same as the bundle action,
except instead of outputting, it writes its result to a register.
|
|
|
|
|
|
|
| |
I don't expect this bundling algorithm to be particularly useful in
software switches. However, hardware switches will probably only
support this bundling algorithm, so it's implemented here as an
example and a reference.
|
|
This patch creates a new action called "bundle". Bundles are a way
to implement a simple form of multipath in OpenFlow by grouping
several ports in a single output-like action.
|