Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reintroduce: Remove NAT logic (v2) #127

Merged
merged 2 commits into from
Mar 7, 2020
Merged

Conversation

kaechele
Copy link
Contributor

@kaechele kaechele commented Mar 6, 2020

Re-submission of #115; see here for some relevant discussion.

Added note and link to issue description regarding the SO_REUSEPORT bug in some Kernel versions.
Otherwise just reverted the revert, meaning that the NAT removal code from the initial pull request has been re-introduced unmodified. That should make a re-review redundant.

Add a link to useful information on this particular error which is most
likely triggered by a bug in the Kernel.
This should give users a way to identify if they are affected and give
them hints on how to rectify the situation.

Signed-off-by: Felix Kaechele <felix@kaechele.ca>
This reverts commit a461a11.
@RalfJung
Copy link
Member

RalfJung commented Mar 7, 2020

I adjusted the issue some more: from what I could see, the faulty commit landed in 4.5, but the 4.4 kernel should be unaffected.

Copy link
Member

@RalfJung RalfJung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All right, let's try again. :)

@RalfJung RalfJung merged commit e9866b5 into wlanslovenija:master Mar 7, 2020
@RalfJung
Copy link
Member

RalfJung commented Mar 7, 2020

I'm afraid this still doesn't work here... see #129.

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.

None yet

2 participants