On Tue, 19 May 2026 19:03:52 -0700 Jakub Kicinski <[email protected]> wrote:
> On Mon, 18 May 2026 20:34:22 +0200 Stefano Brivio wrote: > > Stefano Brivio (2): > > tcp: Don't accept data when socket is in repair mode > > Not sure Eric is on board with this patch in the first place. > Sound like it's not the intended use case for REPAIR so IMO > it's up to TCP maintainers whether we want to support this. > And it's definitely not a Fix. Jakub, thanks for looking into this. As I was pointing out on the v1 thread, I think it's a race condition regardless of the usage, because after you switch a socket to repair mode you can do separate operations on the socket and the outcome will be inconsistent between them. It depends on external conditions and looks quite fragile. For example, you could dump a given length of the receive queue, just to read it a moment later, but now the length is wrong. Note that now, on the v1 thread, also Andrei, one of the authors of TCP_REPAIR and the matching feature in CRIU, agreed that my "fix" makes things safer: https://lore.kernel.org/all/caewa0a4d-ppwpvexygp5slruzj8hs1w1_ww6qa4bbrkzss4...@mail.gmail.com/ But I won't certainly insist on handling it as a fix and I'm now taking care of the new feedback coming from Eric of course. > > selftests: Add data path tests for TCP_REPAIR mode > > Please don't add a new target, fold it under net. > Targets are a PITA to deal with in kselftests. Sorry, I had no idea, I'll change that. -- Stefano
