diff options
author | Tamir Duberstein <tamird@google.com> | 2021-10-01 03:58:23 -0700 |
---|---|---|
committer | gVisor bot <gvisor-bot@google.com> | 2021-10-01 04:01:20 -0700 |
commit | 0db2ab9329b17a66b3dd4f49999325a4b69726bf (patch) | |
tree | 3cf85b2db0783ad80ba7083d78c379efaaa42241 /test | |
parent | 84f1578e9eae732dcff77e05dab46fdb98e508c0 (diff) |
Move pendingEndpoints to acceptQueue
This obsoletes the need for the pendingMu and pending, since they are redundant
with acceptMu and pendingAccepted.
Fixes #6671.
PiperOrigin-RevId: 400162391
Diffstat (limited to 'test')
-rw-r--r-- | test/packetimpact/tests/tcp_listen_backlog_test.go | 16 |
1 files changed, 2 insertions, 14 deletions
diff --git a/test/packetimpact/tests/tcp_listen_backlog_test.go b/test/packetimpact/tests/tcp_listen_backlog_test.go index e4e12a9b5..c5fd37845 100644 --- a/test/packetimpact/tests/tcp_listen_backlog_test.go +++ b/test/packetimpact/tests/tcp_listen_backlog_test.go @@ -285,20 +285,8 @@ func TestTCPListenBacklog(t *testing.T) { go func(i int) { defer wg.Done() - // The pending connection in the SYN queue is now a zombie on gVisor. - // - // TODO(https://gvisor.dev/issues/6671): Stop retransmitting the SYN-ACK. - if i == 0 && !dut.Uname.IsLinux() { - seqNum := uint32(*conn.RemoteSeqNum(t) - 1) - if got, err := conn.Expect(t, testbench.TCP{SeqNum: &seqNum}, time.Second); err != nil { - t.Errorf("%d: expected TCP frame: %s", i, err) - } else if got, want := *got.Flags, header.TCPFlagSyn|header.TCPFlagAck; got != want { - t.Errorf("%d: got %s, want %s", i, got, want) - } - } else { - if got, err := conn.Expect(t, testbench.TCP{}, time.Second); err == nil { - t.Errorf("%d: expected no TCP frame, got %s", i, got) - } + if got, err := conn.Expect(t, testbench.TCP{}, time.Second); err == nil { + t.Errorf("%d: expected no TCP frame, got %s", i, got) } }(i) } |