[PATCH 2/3] tests: Insert sleep after DATA_TEST_FRAME where needed

Jouni Malinen j at w1.fi
Thu Dec 19 15:35:11 PST 2024


On Fri, Sep 27, 2024 at 11:36:21AM +0200, Benjamin Berg wrote:
> If the frame is not fully processed the neighbor entries of the bridge
> will not yet be updated. Add a sleep to ensure that the tests are not
> racy.

Thanks, I applied all three patches, but dropping this reordering of the
validation steps since it is not really related to this commit message
and does not seem to make any real difference in the end:

> -    if len(matches) != 3:
> -        raise Exception("Unexpected number of neighbor entries after connect")
>      if 'aaaa:bbbb:cccc::2 dev ap-br0 lladdr 02:00:00:00:00:00 PERMANENT' not in matches:
>          raise Exception("dev0 addr missing")
>      if 'aaaa:bbbb:dddd::2 dev ap-br0 lladdr 02:00:00:00:01:00 PERMANENT' not in matches:
>          raise Exception("dev1 addr(1) missing")
>      if 'aaaa:bbbb:eeee::2 dev ap-br0 lladdr 02:00:00:00:01:00 PERMANENT' not in matches:
>          raise Exception("dev1 addr(2) missing")
> +    if len(matches) != 3:

-- 
Jouni Malinen                                            PGP id EFC895FA



More information about the Hostap mailing list