Message ID | 20230509090919.1100329-1-liali@redhat.com (mailing list archive) |
---|---|
State | Accepted |
Commit | 2f0f556713f08515f3017fd35309b1f00fbc8932 |
Delegated to: | Netdev Maintainers |
Headers | show |
Series | [net-next] selftests: bonding: delete unnecessary line | expand |
On 09/05/2023 12:09, Liang Li wrote: > "ip link set dev "$devbond1" nomaster" > This line code in bond-eth-type-change.sh is unnecessary. > Because $devbond1 was not added to any master device. > > Signed-off-by: Liang Li <liali@redhat.com> > Acked-by: Hangbin Liu <liuhangbin@gmail.com> > --- > .../selftests/drivers/net/bonding/bond-eth-type-change.sh | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/tools/testing/selftests/drivers/net/bonding/bond-eth-type-change.sh b/tools/testing/selftests/drivers/net/bonding/bond-eth-type-change.sh > index 5cdd22048ba7..862e947e17c7 100755 > --- a/tools/testing/selftests/drivers/net/bonding/bond-eth-type-change.sh > +++ b/tools/testing/selftests/drivers/net/bonding/bond-eth-type-change.sh > @@ -53,7 +53,6 @@ bond_test_enslave_type_change() > # restore ARPHRD_ETHER type by enslaving such device > ip link set dev "$devbond2" master "$devbond0" > check_err $? "could not enslave $devbond2 to $devbond0" > - ip link set dev "$devbond1" nomaster > > bond_check_flags "$devbond0" > Thanks, Reviewed-by: Nikolay Aleksandrov <razor@blackwall.org>
Hello: This patch was applied to netdev/net-next.git (main) by David S. Miller <davem@davemloft.net>: On Tue, 9 May 2023 09:09:19 +0000 you wrote: > "ip link set dev "$devbond1" nomaster" > This line code in bond-eth-type-change.sh is unnecessary. > Because $devbond1 was not added to any master device. > > Signed-off-by: Liang Li <liali@redhat.com> > Acked-by: Hangbin Liu <liuhangbin@gmail.com> > > [...] Here is the summary with links: - [net-next] selftests: bonding: delete unnecessary line https://git.kernel.org/netdev/net-next/c/2f0f556713f0 You are awesome, thank you!
diff --git a/tools/testing/selftests/drivers/net/bonding/bond-eth-type-change.sh b/tools/testing/selftests/drivers/net/bonding/bond-eth-type-change.sh index 5cdd22048ba7..862e947e17c7 100755 --- a/tools/testing/selftests/drivers/net/bonding/bond-eth-type-change.sh +++ b/tools/testing/selftests/drivers/net/bonding/bond-eth-type-change.sh @@ -53,7 +53,6 @@ bond_test_enslave_type_change() # restore ARPHRD_ETHER type by enslaving such device ip link set dev "$devbond2" master "$devbond0" check_err $? "could not enslave $devbond2 to $devbond0" - ip link set dev "$devbond1" nomaster bond_check_flags "$devbond0"