Léo Gillot-Lamure

Results 25 comments of Léo Gillot-Lamure

Digging a bit more… I don’t really speak Go but I’ll try… Clearly this function [RemoveConnection](https://github.com/traefik/traefik/blob/6a06560318c4d940a211e7007f5e0b715480c360/pkg/server/server_entrypoint_tcp.go#L507) is being called too much, or [AddConnection](https://github.com/traefik/traefik/blob/6a06560318c4d940a211e7007f5e0b715480c360/pkg/server/server_entrypoint_tcp.go#L496) not enough.

Errr, so something looks off to me, but probably is not. Reproducing the code of RemoveConnection here: ```go func (c *connectionTracker) RemoveConnection(conn net.Conn) { c.connsMu.Lock() delete(c.conns, conn) c.connsMu.Unlock() if c.openConnectionsGauge...

@rtribotte I see that you authored this highly relevant commit https://github.com/traefik/traefik/commit/7c2af10bbd7791241cea2dce52abf5899e75f3c4 from PR #9656, might you be able to chip in ?

does *mut T::offset() gives you the address plus the offset, or just the offset (the difference itself, regardless of the base pointer). You might want to check that. Also, as...

No, not within rst-mmap, within the standard lib, I'm talking about the one you use here : `data.offset(0x30)`.

It’s in addition, which means that your change in #7 will try to attach twice, which is probably harmless but still not great.

Oh ok that’s much curious, thanks for the testing. Let me see my current file… Ahah ! Interesting, I have an additionnal "sleep 1" at the end (in between line...

Most extensive documentation seems to be this LWN article (bless jcorbet): https://lwn.net/Articles/837033/.

Ok so reading that, definitely "bind" is the later event and the one we want to use. I’ll rework my branch later today to use it (and ignore other events,...

Hi. Yes, sorry, I needed to quickly create this issue so that I didn’t forget about it. Now I had time to extract an example: ```rust use std::{thread::sleep, time::Duration}; use...