Merge branch 'selftests-fib_tests-simplifications-verbosity-and-a-race'
authorDavid S. Miller <davem@davemloft.net>
Tue, 13 Feb 2018 19:34:29 +0000 (14:34 -0500)
committerDavid S. Miller <davem@davemloft.net>
Tue, 13 Feb 2018 19:34:29 +0000 (14:34 -0500)
commitb95367e3bd8eaf2d88d7b7a7ab2847f5a6e70321
tree621fd561ed3ec2ffe9c94dc33f7abee560662e91
parent8c2ceabe99e04005cadba739856eed6953a8a3af
parente2ba732a1681d907a6dffbaf9802aebcac233099
Merge branch 'selftests-fib_tests-simplifications-verbosity-and-a-race'

David Ahern says:

====================
selftests: fib_tests: simplifications, verbosity and a race

Improve efficiency of fib_tests.sh and make the test result more verbose,
from this summary:
$ fib_tests.sh is failing in a VM:
    $ fib_tests.sh
    Running netdev unregister tests
    PASS: unicast route test
    PASS: multipath route test
    Running netdev down tests
    PASS: unicast route test
    PASS: multipath route test
    Running netdev carrier change tests
    PASS: local route carrier test
    FAIL: unicast route carrier test

where a single entry actually corresponds to many checks to a much more
verbse output that clarifies test cases:
$fib_tests.sh
Single path route carrier test
    ....
    Carrier down
        IPv4 fibmatch                                         [ OK ]
        IPv6 fibmatch                                         [ OK ]
        IPv4 linkdown flag set                                [FAIL]
        IPv6 linkdown flag set                                [FAIL]
    Second address added with carrier down
        IPv4 fibmatch                                         [ OK ]
        IPv6 fibmatch                                         [ OK ]
        IPv4 linkdown flag set                                [FAIL]
        IPv6 linkdown flag set                                [ OK ]

And then fix the race in changing carrier down on dummy device to checking
the corresponding routes.
====================

Signed-off-by: David S. Miller <davem@davemloft.net>