Skip to content

Commit

Permalink
selftests: mptcp: join: no RST when rm subflow/addr
Browse files Browse the repository at this point in the history
Recently, we noticed that some RST were wrongly generated when removing
the initial subflow.

This patch makes sure RST are not sent when removing any subflows or any
addresses.

Signed-off-by: Matthieu Baerts <[email protected]>
  • Loading branch information
matttbe authored and intel-lab-lkp committed Oct 10, 2023
1 parent 4cb87d2 commit 2c3dd89
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions tools/testing/selftests/net/mptcp/mptcp_join.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2343,6 +2343,7 @@ remove_tests()
chk_join_nr 1 1 1
chk_rm_tx_nr 1
chk_rm_nr 1 1
chk_rst_nr 0 0
fi

# multiple subflows, remove
Expand All @@ -2355,6 +2356,7 @@ remove_tests()
run_tests $ns1 $ns2 10.0.1.1
chk_join_nr 2 2 2
chk_rm_nr 2 2
chk_rst_nr 0 0
fi

# single address, remove
Expand All @@ -2367,6 +2369,7 @@ remove_tests()
chk_join_nr 1 1 1
chk_add_nr 1 1
chk_rm_nr 1 1 invert
chk_rst_nr 0 0
fi

# subflow and signal, remove
Expand All @@ -2380,6 +2383,7 @@ remove_tests()
chk_join_nr 2 2 2
chk_add_nr 1 1
chk_rm_nr 1 1
chk_rst_nr 0 0
fi

# subflows and signal, remove
Expand All @@ -2394,6 +2398,7 @@ remove_tests()
chk_join_nr 3 3 3
chk_add_nr 1 1
chk_rm_nr 2 2
chk_rst_nr 0 0
fi

# addresses remove
Expand All @@ -2408,6 +2413,7 @@ remove_tests()
chk_join_nr 3 3 3
chk_add_nr 3 3
chk_rm_nr 3 3 invert
chk_rst_nr 0 0
fi

# invalid addresses remove
Expand All @@ -2422,6 +2428,7 @@ remove_tests()
chk_join_nr 1 1 1
chk_add_nr 3 3
chk_rm_nr 3 1 invert
chk_rst_nr 0 0
fi

# subflows and signal, flush
Expand All @@ -2436,6 +2443,7 @@ remove_tests()
chk_join_nr 3 3 3
chk_add_nr 1 1
chk_rm_nr 1 3 invert simult
chk_rst_nr 0 0
fi

# subflows flush
Expand All @@ -2455,6 +2463,7 @@ remove_tests()
else
chk_rm_nr 3 3
fi
chk_rst_nr 0 0
fi

# addresses flush
Expand All @@ -2469,6 +2478,7 @@ remove_tests()
chk_join_nr 3 3 3
chk_add_nr 3 3
chk_rm_nr 3 3 invert simult
chk_rst_nr 0 0
fi

# invalid addresses flush
Expand All @@ -2483,6 +2493,7 @@ remove_tests()
chk_join_nr 1 1 1
chk_add_nr 3 3
chk_rm_nr 3 1 invert
chk_rst_nr 0 0
fi

# remove id 0 subflow
Expand All @@ -2494,6 +2505,7 @@ remove_tests()
run_tests $ns1 $ns2 10.0.1.1
chk_join_nr 1 1 1
chk_rm_nr 1 1
chk_rst_nr 0 0 ## TODO: currently failing, Geliang is working on a fix
fi

# remove id 0 address
Expand All @@ -2506,6 +2518,7 @@ remove_tests()
chk_join_nr 1 1 1
chk_add_nr 1 1
chk_rm_nr 1 1 invert
chk_rst_nr 0 0 invert ## TODO: currently failing, Geliang is working on a fix
fi
}

Expand Down

0 comments on commit 2c3dd89

Please sign in to comment.