Attention is currently required from: laforge, pespin.

View Change

1 comment:

• File stp/STP_Tests_M3UA.ttcn:

  • Patch Set #4, Line 1229: f_sleep(1.0);

    can you explain this?

    In the case when `ASP[idx_b]` is configured as the client (i.e. osmo-stp connects to the testsuite; `idx_b >= NR_M3UA`), in `f_M3UA_CLNT_asp_up_act()` we're expecting to receive `ASPACT` and then sending `ASPACT_ACK` to it. Right after that, we're sending some random data via `ASP[idx_a]`, which we then expect to receive via `ASP[idx_b]` (see `f_test_traffic()`).

    There is a chance that the random data sent via `ASP[idx_a]` would reach osmo-stp earlier than the `ASPUP_ACK` we sent for `ASP[idx_b]` -- this is what I was seeing when running `TC_m3ua_tcp_cli_srv`. In this test scenario only `ASP[idx_b]` is using TCP, and there is some delay likely caused by Nagle's algorithm.

    It's not my invention (you can find `f_sleep(1.0)` in the code above), but it does help to avoid the race condition.

To view, visit change 36150. To unsubscribe, or for help writing mail filters, visit settings.