mirror of
https://github.com/torvalds/linux.git
synced 2026-03-08 08:24:47 +01:00
Since we started running selftests in NIPA we have been seeing
tc_actions.sh generate a soft lockup warning on ~20% of the runs.
On the pre-netdev foundation setup it was actually a missed irq
splat from the console. Now it's either that or a lockup.
I initially suspected a socket locking issue since the test
is exercising local loopback with act_mirred.
After hours of staring at this I noticed in strace that ncat
when -o $file is specified _both_ saves the output to the file
and still prints it to stdout. Because the file being sent
is constructed with:
dd conv=sparse status=none if=/dev/zero bs=1M count=2 of=$mirred
^^^^^^^^^
the data printed is all \0. Most terminals don't display nul
characters (and neither does vng output capture save them).
But QEMU's serial console still has to poke them thru which
is very slow and causes the lockup (if the file is >600kB).
Replace the '-o $file' with '> $file'. This speeds the test up
from 2m20s to 18s on debug kernels, and prevents the warnings.
Fixes:
|
||
|---|---|---|
| .. | ||
| .gitignore | ||
| bridge_activity_notify.sh | ||
| bridge_fdb_learning_limit.sh | ||
| bridge_fdb_local_vlan_0.sh | ||
| bridge_igmp.sh | ||
| bridge_locked_port.sh | ||
| bridge_mdb.sh | ||
| bridge_mdb_host.sh | ||
| bridge_mdb_max.sh | ||
| bridge_mdb_port_down.sh | ||
| bridge_mld.sh | ||
| bridge_port_isolation.sh | ||
| bridge_sticky_fdb.sh | ||
| bridge_vlan_aware.sh | ||
| bridge_vlan_mcast.sh | ||
| bridge_vlan_unaware.sh | ||
| config | ||
| custom_multipath_hash.sh | ||
| devlink_lib.sh | ||
| dual_vxlan_bridge.sh | ||
| fib_offload_lib.sh | ||
| forwarding.config.sample | ||
| gre_custom_multipath_hash.sh | ||
| gre_inner_v4_multipath.sh | ||
| gre_inner_v6_multipath.sh | ||
| gre_multipath.sh | ||
| gre_multipath_nh.sh | ||
| gre_multipath_nh_res.sh | ||
| ip6_forward_instats_vrf.sh | ||
| ip6gre_custom_multipath_hash.sh | ||
| ip6gre_flat.sh | ||
| ip6gre_flat_key.sh | ||
| ip6gre_flat_keys.sh | ||
| ip6gre_hier.sh | ||
| ip6gre_hier_key.sh | ||
| ip6gre_hier_keys.sh | ||
| ip6gre_inner_v4_multipath.sh | ||
| ip6gre_inner_v6_multipath.sh | ||
| ip6gre_lib.sh | ||
| ipip_flat_gre.sh | ||
| ipip_flat_gre_key.sh | ||
| ipip_flat_gre_keys.sh | ||
| ipip_hier_gre.sh | ||
| ipip_hier_gre_key.sh | ||
| ipip_hier_gre_keys.sh | ||
| ipip_lib.sh | ||
| lib.sh | ||
| lib_sh_test.sh | ||
| local_termination.sh | ||
| Makefile | ||
| min_max_mtu.sh | ||
| mirror_gre.sh | ||
| mirror_gre_bound.sh | ||
| mirror_gre_bridge_1d.sh | ||
| mirror_gre_bridge_1d_vlan.sh | ||
| mirror_gre_bridge_1q.sh | ||
| mirror_gre_bridge_1q_lag.sh | ||
| mirror_gre_changes.sh | ||
| mirror_gre_flower.sh | ||
| mirror_gre_lag_lacp.sh | ||
| mirror_gre_lib.sh | ||
| mirror_gre_neigh.sh | ||
| mirror_gre_nh.sh | ||
| mirror_gre_topo_lib.sh | ||
| mirror_gre_vlan.sh | ||
| mirror_gre_vlan_bridge_1q.sh | ||
| mirror_lib.sh | ||
| mirror_topo_lib.sh | ||
| mirror_vlan.sh | ||
| no_forwarding.sh | ||
| pedit_dsfield.sh | ||
| pedit_ip.sh | ||
| pedit_l4port.sh | ||
| q_in_vni.sh | ||
| q_in_vni_ipv6.sh | ||
| README | ||
| router.sh | ||
| router_bridge.sh | ||
| router_bridge_1d.sh | ||
| router_bridge_1d_lag.sh | ||
| router_bridge_lag.sh | ||
| router_bridge_pvid_vlan_upper.sh | ||
| router_bridge_vlan.sh | ||
| router_bridge_vlan_upper.sh | ||
| router_bridge_vlan_upper_pvid.sh | ||
| router_broadcast.sh | ||
| router_mpath_nh.sh | ||
| router_mpath_nh_lib.sh | ||
| router_mpath_nh_res.sh | ||
| router_mpath_seed.sh | ||
| router_multicast.sh | ||
| router_multipath.sh | ||
| router_nh.sh | ||
| router_vid_1.sh | ||
| sch_ets.sh | ||
| sch_ets_core.sh | ||
| sch_ets_tests.sh | ||
| sch_red.sh | ||
| sch_tbf_core.sh | ||
| sch_tbf_ets.sh | ||
| sch_tbf_etsprio.sh | ||
| sch_tbf_prio.sh | ||
| sch_tbf_root.sh | ||
| settings | ||
| skbedit_priority.sh | ||
| tc_actions.sh | ||
| tc_chains.sh | ||
| tc_common.sh | ||
| tc_flower.sh | ||
| tc_flower_cfm.sh | ||
| tc_flower_l2_miss.sh | ||
| tc_flower_port_range.sh | ||
| tc_flower_router.sh | ||
| tc_mpls_l2vpn.sh | ||
| tc_police.sh | ||
| tc_shblocks.sh | ||
| tc_taprio.sh | ||
| tc_tunnel_key.sh | ||
| tc_vlan_modify.sh | ||
| tsn_lib.sh | ||
| vxlan_asymmetric.sh | ||
| vxlan_asymmetric_ipv6.sh | ||
| vxlan_bridge_1d.sh | ||
| vxlan_bridge_1d_ipv6.sh | ||
| vxlan_bridge_1d_port_8472.sh | ||
| vxlan_bridge_1d_port_8472_ipv6.sh | ||
| vxlan_bridge_1q.sh | ||
| vxlan_bridge_1q_ipv6.sh | ||
| vxlan_bridge_1q_mc_ul.sh | ||
| vxlan_bridge_1q_port_8472.sh | ||
| vxlan_bridge_1q_port_8472_ipv6.sh | ||
| vxlan_reserved.sh | ||
| vxlan_symmetric.sh | ||
| vxlan_symmetric_ipv6.sh | ||
Motivation
==========
One of the nice things about network namespaces is that they allow one
to easily create and test complex environments.
Unfortunately, these namespaces can not be used with actual switching
ASICs, as their ports can not be migrated to other network namespaces
(dev->netns_immutable) and most of them probably do not support the
L1-separation provided by namespaces.
However, a similar kind of flexibility can be achieved by using VRFs and
by looping the switch ports together. For example:
br0
+
vrf-h1 | vrf-h2
+ +---+----+ +
| | | |
192.0.2.1/24 + + + + 192.0.2.2/24
swp1 swp2 swp3 swp4
+ + + +
| | | |
+--------+ +--------+
The VRFs act as lightweight namespaces representing hosts connected to
the switch.
This approach for testing switch ASICs has several advantages over the
traditional method that requires multiple physical machines, to name a
few:
1. Only the device under test (DUT) is being tested without noise from
other system.
2. Ability to easily provision complex topologies. Testing bridging
between 4-ports LAGs or 8-way ECMP requires many physical links that are
not always available. With the VRF-based approach one merely needs to
loopback more ports.
These tests are written with switch ASICs in mind, but they can be run
on any Linux box using veth pairs to emulate physical loopbacks.
Guidelines for Writing Tests
============================
o Where possible, reuse an existing topology for different tests instead
of recreating the same topology.
o Tests that use anything but the most trivial topologies should include
an ASCII art showing the topology.
o Where possible, IPv6 and IPv4 addresses shall conform to RFC 3849 and
RFC 5737, respectively.
o Where possible, tests shall be written so that they can be reused by
multiple topologies and added to lib.sh.
o Checks shall be added to lib.sh for any external dependencies.
o Code shall be checked using ShellCheck [1] prior to submission.
1. https://www.shellcheck.net/
Cleanups
--------
o lib.sh brings in defer.sh (by way of ../lib.sh) by default. Consider
making use of the defer primitive to schedule automatic cleanups. This
makes it harder to forget to remove a temporary netdevice, kill a running
process or perform other cleanup when the test script is interrupted.
o When adding a helper that dirties the environment, but schedules all
necessary cleanups through defer, consider prefixing it adf_ for
consistency with lib.sh and ../lib.sh helpers. This serves as an
immediately visible bit of documentation about the helper API.
o Definitely do the above for any new code in lib.sh, if practical.
Customization
=============
The forwarding selftests framework uses a number of variables that
influence its behavior and tools it invokes, and how it invokes them, in
various ways. A number of these variables can be overridden. The way these
overridable variables are specified is typically one of the following two
syntaxes:
: "${VARIABLE:=default_value}"
VARIABLE=${VARIABLE:=default_value}
Any of these variables can be overridden. Notably net/forwarding/lib.sh and
net/lib.sh contain a number of overridable variables.
One way of overriding these variables is through the environment:
PAUSE_ON_FAIL=yes ./some_test.sh
The variable NETIFS is special. Since it is an array variable, there is no
way to pass it through the environment. Its value can instead be given as
consecutive arguments to the selftest:
./some_test.sh swp{1..8}
A way to customize variables in a persistent fashion is to create a file
named forwarding.config in this directory. lib.sh sources the file if
present, so it can contain any shell code. Typically it will contain
assignments of variables whose value should be overridden.
forwarding.config.sample is available in the directory as an example of
how forwarding.config might look.