inetstack

inetstack copied to clipboard

[tcp] `dead_socket_tx` Causes Unit Test Regressions to Fail

3

Running `test_connect` currently fails with: ``` Failed to terminate connection: TrySendError { kind: Disconnected } thread 'protocols::tcp::tests::test_connect' panicked at 'Failed to terminate connection: TrySendError { kind: Disconnected }', src/protocols/tcp/established/background/mod.rs:58:14 stack...

gatoWololo

Description ========= `PassiveSocket`s are held inside the `TcpPeer`'s `passive` field. These are never removed from this field though. They should probably be removed once `close` is called on the corresponding...

gatoWololo

The `Inner` struct for `TcpPeer` holds the established connections on the `established: HashMap` field. It seems the `remove` method is never called on this HashMap. So while the socket state...

gatoWololo

# Description We should support ECN in IPv4.

ppenna

# Description We should support DSCP in IPv4.

ppenna

# Description We should support Ipv4 options.

ppenna

# Description We should implement `close()` for passive sockets. Related Issues ---------------- - https://github.com/demikernel/catnip/issues/82

ppenna

Description ======== Currently, when we close a TCP connection using the `close()` system call, we do not `wait()` for the connection FIN packet to arrive, thereby leaving the connection in...

ppenna

Description ======== TCP checksum offloading seems to be malfunctioning. We should investigate and fix, if needed.

ppenna

Description ======== Currently, the default MSS size is hard-code (https://github.com/demikernel/catnip/blob/v0.7/src/protocols/tcp/constants.rs#L12). Instead, we should compute this from the MTU. Related Issues ----------------- demikernel/runtime#8

ppenna