Linux XDP support #3849
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -64,6 +64,9 @@ elseif (APPLE) | |
| set(CX_PLATFORM "darwin") | ||
| elseif (UNIX) | ||
| set(CX_PLATFORM "linux") | ||
| file(STRINGS "/etc/lsb-release" LSB_RELEASE_CONTENT) | ||
|
There was a problem hiding this comment. This line broke our Alpine docker build, using /etc/os-release would be better as it is present in both distributions. |
||
| string(REGEX MATCH "DISTRIB_RELEASE=([0-9]+\\.[0-9]+)" _ ${LSB_RELEASE_CONTENT}) | ||
| set(UBUNTU_VERSION ${CMAKE_MATCH_1}) | ||
| endif() | ||
| message(STATUS "QUIC Platform: ${CX_PLATFORM}") | ||
|
|
||
|
|
@@ -434,13 +437,13 @@ if (NOT MSVC AND NOT APPLE AND NOT ANDROID) | |
| endif() | ||
| endif() | ||
|
|
||
| if (CMAKE_GENERATOR_PLATFORM STREQUAL "") | ||
| string(TOLOWER ${CMAKE_SYSTEM_PROCESSOR} SYSTEM_PROCESSOR) | ||
| else() | ||
| string(TOLOWER ${CMAKE_GENERATOR_PLATFORM} SYSTEM_PROCESSOR) | ||
| endif() | ||
|
|
||
| if(WIN32) | ||
| # Generate the MsQuicEtw header file. | ||
| file(MAKE_DIRECTORY ${QUIC_BUILD_DIR}/inc) | ||
|
|
||
|
|
||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,102 @@ | ||
| #!/bin/bash | ||
|
|
||
| # Set the number of NIC pairs | ||
| NumNicPairs=1 | ||
|
|
||
| if [ "$1" == "install" ]; then | ||
| # Configure each pair separately with its own hard-coded subnet, ie 192.168.x.0/24 and fc00::x/112 | ||
| for ((i=1; i<=NumNicPairs; i++)); do | ||
| echo "Plumbing IP config for pair $i" | ||
|
|
||
| # Generate the "ID" of the NICs, eg 1 and 2 for the first pair | ||
| nic1="duo$((i * 2 - 1))" | ||
| nic2="duo$((i * 2))" | ||
|
|
||
| # Create veth pair | ||
| sudo ip link add ${nic1} type veth peer name ${nic2} | ||
|
|
||
| # Set the veth interfaces up | ||
| sudo ip link set ${nic1} up | ||
| sudo ip link set ${nic2} up | ||
|
|
||
| # Assign IPv4 addresses | ||
| sudo ip addr add 192.168.${i}.11/24 dev ${nic1} | ||
| sudo ip addr add 192.168.${i}.12/24 dev ${nic2} | ||
|
|
||
| # Assign IPv6 addresses | ||
| sudo ip -6 addr add fc00::${i}:11/112 dev ${nic1} | ||
| sudo ip -6 addr add fc00::${i}:12/112 dev ${nic2} | ||
|
|
||
| # Add static neighbor entries (ARP) | ||
| sudo ip neigh add 192.168.${i}.12 lladdr 22:22:22:22:00:0$((i * 2)) dev ${nic1} nud permanent | ||
| sudo ip neigh add 192.168.${i}.11 lladdr 22:22:22:22:00:0$((i * 2 - 1)) dev ${nic2} nud permanent | ||
|
|
||
| # Add static neighbor entries (IPv6) | ||
| sudo ip -6 neigh add fc00::${i}:12 lladdr 22:22:22:22:00:0$((i * 2)) dev ${nic1} nud permanent | ||
| sudo ip -6 neigh add fc00::${i}:11 lladdr 22:22:22:22:00:0$((i * 2 - 1)) dev ${nic2} nud permanent | ||
|
|
||
| # Configure routing rules for IPv4 | ||
| sudo ip route add 192.168.${i}.12/32 dev ${nic1} metric 0 | ||
| sudo ip route add 192.168.${i}.11/32 dev ${nic2} metric 0 | ||
|
|
||
|
|
||
|
|
||
| # Configure routing rules for IPv6 | ||
| sudo ip -6 route add fc00::${i}:12/128 dev ${nic1} metric 0 | ||
| sudo ip -6 route add fc00::${i}:11/128 dev ${nic2} metric 0 | ||
|
|
||
| # Configure firewall rules for IPv4 | ||
| sudo iptables -A INPUT -p all -s 192.168.${i}.0/24 -i ${nic1} -j ACCEPT | ||
| sudo iptables -A INPUT -p all -s 192.168.${i}.0/24 -i ${nic2} -j ACCEPT | ||
|
|
||
| # Configure firewall rules for IPv6 | ||
| sudo ip6tables -A INPUT -p all -s fc00::${i}:0/112 -i ${nic1} -j ACCEPT | ||
| sudo ip6tables -A INPUT -p all -s fc00::${i}:0/112 -i ${nic2} -j ACCEPT | ||
|
|
||
| sleep 2 | ||
| sudo ip route change 192.168.${i}.12 dev duo1 src 192.168.${i}.11 | ||
| sudo ip route change 192.168.${i}.11 dev duo2 src 192.168.${i}.12 | ||
|
|
||
| sudo ip -6 route change fc00::${i}:12 dev duo1 src fc00::${i}:11 | ||
| sudo ip -6 route change fc00::${i}:11 dev duo2 src fc00::${i}:12 | ||
| done | ||
| elif [ "$1" == "uninstall" ]; then | ||
| # Cleanup each pair separately | ||
| for ((i=1; i<=NumNicPairs; i++)); do | ||
| echo "Cleaning up pair $i" | ||
|
|
||
| # Generate the "ID" of the NICs, eg 1 and 2 for the first pair | ||
| nic1="duo$((i * 2 - 1))" | ||
| nic2="duo$((i * 2))" | ||
|
|
||
| # Delete firewall rules for IPv4 | ||
| sudo iptables -D INPUT -p all -s 192.168.${i}.0/24 -i ${nic1} -j ACCEPT | ||
| sudo iptables -D INPUT -p all -s 192.168.${i}.0/24 -i ${nic2} -j ACCEPT | ||
|
|
||
| # Delete firewall rules for IPv6 | ||
| sudo ip6tables -D INPUT -p all -s fc00::${i}:0/112 -i ${nic1} -j ACCEPT | ||
| sudo ip6tables -D INPUT -p all -s fc00::${i}:0/112 -i ${nic2} -j ACCEPT | ||
|
|
||
| # Remove routing rules for IPv4 | ||
| sudo ip route del 192.168.${i}.12/32 dev ${nic1} | ||
| sudo ip route del 192.168.${i}.11/32 dev ${nic2} | ||
|
|
||
| # Remove routing rules for IPv6 | ||
| sudo ip -6 route del fc00::${i}:12/128 dev ${nic1} | ||
| sudo ip -6 route del fc00::${i}:11/128 dev ${nic2} | ||
|
|
||
| # Remove static neighbor entries (ARP) | ||
| sudo ip neigh del 192.168.${i}.12 lladdr 22:22:22:22:00:0$((i * 2)) dev ${nic1} | ||
| sudo ip neigh del 192.168.${i}.11 lladdr 22:22:22:22:00:0$((i * 2 - 1)) dev ${nic2} | ||
|
|
||
| # Remove static neighbor entries (IPv6) | ||
| sudo ip -6 neigh del fc00::${i}:12 lladdr 22:22:22:22:00:0$((i * 2)) dev ${nic1} | ||
| sudo ip -6 neigh del fc00::${i}:11 lladdr 22:22:22:22:00:0$((i * 2 - 1)) dev ${nic2} | ||
|
|
||
| # Remove the veth pair | ||
| sudo ip link delete ${nic1} | ||
| done | ||
| else | ||
| echo "Usage: $0 {install|uninsatll}" | ||
| exit 1 | ||
| fi |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
XDP doesn't work with sanitizer? Why?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I previously saw data alignment related error by test.yml.
Not sure stress test. I enabled stress test later than test.yml
I disabled sanitizer from functionality point of view
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should follow up (in a later PR) on the sanitizer.