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

Prefer SOCK_DGRAM #317

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Prefer SOCK_DGRAM #317

wants to merge 1 commit into from

Conversation

Yenya
Copy link

@Yenya Yenya commented May 3, 2024

ping(8) from iputils-20240117 prefers SOCK_DGRAM instead of SOCK_RAW, which has a bonus of using kernel-assigned Identification. Apart from unprivileged operations, this also avoids collisions caused by user-assigned Identification on busy monitoring hosts (see also: birthday paradox).

So let's try SOCK_DGRAM first, and only if that fails, fall back to SOCK_RAW.

ping(8) from iputils-20240117 prefers SOCK_DGRAM instead of SOCK_RAW,
which has a bonus of using kernel-assigned Identification. Apart from
unprivileged operations, this also avoids collisions caused by
user-assigned Identification on busy monitoring hosts
(see also: birthday paradox).

So let's try SOCK_DGRAM first, and only if that fails, fall back to
SOCK_RAW.
@auerswal
Copy link
Collaborator

auerswal commented May 4, 2024

It seems as if SOCK_DGRAM does not work as well as SOCK_RAW for fping when using the kernel Linux, according to the test failures.

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