Skip to content

Ignore a dup SYN for older connections as well#546

Merged
djs55 merged 1 commit intomoby:masterfrom
yamt:drop-dup-syn
Sep 21, 2021
Merged

Ignore a dup SYN for older connections as well#546
djs55 merged 1 commit intomoby:masterfrom
yamt:drop-dup-syn

Conversation

@yamt
Copy link
Copy Markdown
Contributor

@yamt yamt commented Jul 12, 2021

I occasionally observed stalls because of this
when using NuttX sim as a client.

I occasionally observed stalls because of this
when using NuttX sim as a client.

Signed-off-by: YAMAMOTO Takashi <yamamoto@midokura.com>
@djs55
Copy link
Copy Markdown
Collaborator

djs55 commented Jul 12, 2021

@yamt thanks for the patch! I think I might have seen a similar problem.

I'll test it and let you know what I find. Unfortunately the CI isn't fully enabled on this repo at the moment.

@yamt
Copy link
Copy Markdown
Contributor Author

yamt commented Sep 21, 2021

i applied this to our private ci two months ago.
i haven't noticed any issues since then.
before applying this patch, it was occasionally failing. (once a few days or so)

@djs55
Copy link
Copy Markdown
Collaborator

djs55 commented Sep 21, 2021

@yamt thanks for the update. I'll merge the patch now.

@djs55 djs55 merged commit 16ed722 into moby:master Sep 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants