Module Name:    src
Committed By:   ozaki-r
Date:           Fri Mar  4 04:18:44 UTC 2016

Modified Files:
        src/tests/net/arp: t_arp.sh

Log Message:
Improve tests of proxy arp

The tests make it clear how it behaves though, I don't know if the current
behavior is what it should be.


To generate a diff of this commit:
cvs rdiff -u -r1.12 -r1.13 src/tests/net/arp/t_arp.sh

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/tests/net/arp/t_arp.sh
diff -u src/tests/net/arp/t_arp.sh:1.12 src/tests/net/arp/t_arp.sh:1.13
--- src/tests/net/arp/t_arp.sh:1.12	Mon Feb 29 09:35:16 2016
+++ src/tests/net/arp/t_arp.sh	Fri Mar  4 04:18:44 2016
@@ -1,4 +1,4 @@
-#	$NetBSD: t_arp.sh,v 1.12 2016/02/29 09:35:16 ozaki-r Exp $
+#	$NetBSD: t_arp.sh,v 1.13 2016/03/04 04:18:44 ozaki-r Exp $
 #
 # Copyright (c) 2015 The NetBSD Foundation, Inc.
 # All rights reserved.
@@ -26,16 +26,15 @@
 #
 
 inetserver="rump_server -lrumpnet -lrumpnet_net -lrumpnet_netinet -lrumpnet_shmif"
+inetserver="$inetserver -lrumpdev -lrumpnet_tap"
 HIJACKING="env LD_PRELOAD=/usr/lib/librumphijack.so RUMPHIJACK=sysctl=yes"
 
 SOCKSRC=unix://commsock1
 SOCKDST=unix://commsock2
 IP4SRC=10.0.1.1
 IP4DST=10.0.1.2
-IP4DST_PUB=10.0.1.3
-MACDST_PUB=b2:a1:00:00:00:01
-IP4DST_PUBPROXY=10.0.1.4
-MACDST_PUBPROXY=b2:a1:00:00:00:02
+IP4DST_PROXYARP1=10.0.1.3
+IP4DST_PROXYARP2=10.0.1.4
 
 DEBUG=false
 TIMEOUT=1
@@ -45,7 +44,8 @@ atf_test_case cache_expiration_10s clean
 atf_test_case command cleanup
 atf_test_case garp cleanup
 atf_test_case cache_overwriting cleanup
-atf_test_case pubproxy_arp cleanup
+atf_test_case proxy_arp_pub cleanup
+atf_test_case proxy_arp_pubproxy cleanup
 atf_test_case link_activation cleanup
 
 cache_expiration_5s_head()
@@ -78,9 +78,15 @@ cache_overwriting_head()
 	atf_set "require.progs" "rump_server"
 }
 
-pubproxy_arp_head()
+proxy_arp_pub_head()
 {
-	atf_set "descr" "Tests for Proxy ARP"
+	atf_set "descr" "Tests for Proxy ARP (pub)"
+	atf_set "require.progs" "rump_server"
+}
+
+proxy_arp_pubproxy_head()
+{
+	atf_set "descr" "Tests for Proxy ARP (pub proxy)"
 	atf_set "require.progs" "rump_server"
 }
 
@@ -360,9 +366,12 @@ check_entry_flags()
 	    "rump.netstat -rn -f inet | grep ^'$ip'"
 }
 
-pubproxy_arp_body()
+
+test_proxy_arp()
 {
 	local arp_keep=5
+	local opts= title= flags=
+	local type=$1
 
 	atf_check -s exit:0 ${inetserver} $SOCKSRC
 	atf_check -s exit:0 ${inetserver} $SOCKDST
@@ -371,48 +380,116 @@ pubproxy_arp_body()
 	setup_src_server $arp_keep
 
 	export RUMP_SERVER=$SOCKDST
+	atf_check -s exit:0 -o ignore rump.sysctl -w net.inet.ip.forwarding=1
+	macaddr_dst=$(rump.ifconfig shmif0 |awk '/address/ {print $2;}')
 
-	atf_check -s exit:0 -o ignore rump.arp -s $IP4DST_PUB \
-	    $MACDST_PUB pub
-	atf_check -s exit:0 -o match:'permanent published' \
-	    rump.arp -n $IP4DST_PUB
-	check_entry_flags $IP4DST_PUB ULSp
+	if [ "$type" = "pub" ]; then
+		opts="pub"
+		title="permanent published"
+		flags="ULSp"
+	else
+		opts="pub proxy"
+		title='permanent published \(proxy only\)'
+		flags="UHLSp"
+	fi
 
-	$DEBUG && rump.arp -n -a
-	$DEBUG && rump.netstat -nr -f inet
+	#
+	# Test#1: First setup an endpoint then create proxy arp entry
+	#
+	export RUMP_SERVER=$SOCKDST
+	atf_check -s exit:0 rump.ifconfig tap1 create
+	atf_check -s exit:0 rump.ifconfig tap1 $IP4DST_PROXYARP1/24 up
+	atf_check -s exit:0 rump.ifconfig -w 10
 
-	atf_check -s exit:0 -o ignore rump.arp -s $IP4DST_PUBPROXY \
-	    $MACDST_PUBPROXY pub proxy
-	atf_check -s exit:0 -o match:'permanent published \(proxy only\)' \
-	    rump.arp -n $IP4DST_PUBPROXY
-	check_entry_flags $IP4DST_PUBPROXY UHLSp
+	# Try to ping (should fail w/o proxy arp)
+	export RUMP_SERVER=$SOCKSRC
+	atf_check -s not-exit:0 -o ignore -e ignore \
+	    rump.ping -n -w 1 -c 1 $IP4DST_PROXYARP1
 
-	$DEBUG && rump.arp -n -a
-	$DEBUG && rump.netstat -nr -f inet
+	# Flushing
+	extract_new_packets > ./out
 
-	export RUMP_SERVER=$SOCKSRC
+	# Set up proxy ARP entry
+	export RUMP_SERVER=$SOCKDST
+	atf_check -s exit:0 -o ignore \
+	    rump.arp -s $IP4DST_PROXYARP1 $macaddr_dst $opts
+	atf_check -s exit:0 -o match:"$title" rump.arp -n $IP4DST_PROXYARP1
+	if [ "$type" = "pub" ]; then
+		# XXX local? Is it correct?
+		check_entry_flags $IP4DST_PROXYARP1 ${flags}l
+	else
+		check_entry_flags $IP4DST_PROXYARP1 $flags
+	fi
 
-	atf_check -s not-exit:0 -o ignore -e ignore \
-	    rump.ping -n -w 1 -c 1 $IP4DST_PUB
+	# Try to ping
+	export RUMP_SERVER=$SOCKSRC
+	if [ "$type" = "pub" ]; then
+		# XXX fails
+		atf_check -s not-exit:0 -o ignore -e ignore \
+		    rump.ping -n -w 1 -c 1 $IP4DST_PROXYARP1
+	else
+		atf_check -s exit:0 -o ignore \
+		    rump.ping -n -w 1 -c 1 $IP4DST_PROXYARP1
+	fi
 
-	atf_check -s exit:0 sleep 1
 	extract_new_packets > ./out
 	$DEBUG && cat ./out
 
-	pkt=$(make_pkt_str_arprep $IP4DST_PUB $MACDST_PUB)
-	atf_check -s exit:0 -x "cat ./out |grep -q '$pkt'"
+	pkt=$(make_pkt_str_arprep $IP4DST_PROXYARP1 $macaddr_dst)
+	if [ "$type" = "pub" ]; then
+		atf_check -s not-exit:0 -x "cat ./out |grep -q '$pkt'"
+	else
+		atf_check -s exit:0 -x "cat ./out |grep -q '$pkt'"
+	fi
 
+	#
+	# Test#2: Create proxy arp entry then set up an endpoint
+	#
+	export RUMP_SERVER=$SOCKDST
+	atf_check -s exit:0 -o ignore \
+	    rump.arp -s $IP4DST_PROXYARP2 $macaddr_dst $opts
+	atf_check -s exit:0 -o match:"$title" rump.arp -n $IP4DST_PROXYARP2
+	check_entry_flags $IP4DST_PROXYARP2 $flags
+
+	# Try to ping (should fail because no endpoint exists)
+	export RUMP_SERVER=$SOCKSRC
 	atf_check -s not-exit:0 -o ignore -e ignore \
-	    rump.ping -n -w 1 -c 1 $IP4DST_PUBPROXY
+	    rump.ping -n -w 1 -c 1 $IP4DST_PROXYARP2
 
-	atf_check -s exit:0 sleep 1
 	extract_new_packets > ./out
 	$DEBUG && cat ./out
 
-	pkt=$(make_pkt_str_arprep $IP4DST_PUBPROXY $MACDST_PUBPROXY)
+	# ARP reply should be sent
+	pkt=$(make_pkt_str_arprep $IP4DST_PROXYARP2 $macaddr_dst)
 	atf_check -s exit:0 -x "cat ./out |grep -q '$pkt'"
 
-	return 0
+	export RUMP_SERVER=$SOCKDST
+	atf_check -s exit:0 rump.ifconfig tap2 create
+	atf_check -s exit:0 rump.ifconfig tap2 $IP4DST_PROXYARP2/24 up
+	atf_check -s exit:0 rump.ifconfig -w 10
+
+	# Try to ping
+	export RUMP_SERVER=$SOCKSRC
+	if [ "$type" = "pub" ]; then
+		atf_check -s exit:0 -o ignore \
+		    rump.ping -n -w 1 -c 1 $IP4DST_PROXYARP2
+	else
+		# XXX fails
+		atf_check -s not-exit:0 -o ignore -e ignore \
+		    rump.ping -n -w 1 -c 1 $IP4DST_PROXYARP2
+	fi
+}
+
+proxy_arp_pub_body()
+{
+
+	test_proxy_arp pub
+}
+
+proxy_arp_pubproxy_body()
+{
+
+	test_proxy_arp pubproxy
 }
 
 link_activation_body()
@@ -515,7 +592,13 @@ cache_overwriting_cleanup()
 	cleanup
 }
 
-pubproxy_arp_cleanup()
+proxy_arp_pub_cleanup()
+{
+	$DEBUG && dump
+	cleanup
+}
+
+proxy_arp_pubproxy_cleanup()
 {
 	$DEBUG && dump
 	cleanup
@@ -534,6 +617,7 @@ atf_init_test_cases()
 	atf_add_test_case command
 	atf_add_test_case garp
 	atf_add_test_case cache_overwriting
-	atf_add_test_case pubproxy_arp
+	atf_add_test_case proxy_arp_pub
+	atf_add_test_case proxy_arp_pubproxy
 	atf_add_test_case link_activation
 }

Reply via email to