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

WIP Broadcasting #22

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

WIP Broadcasting #22

wants to merge 6 commits into from

Conversation

povilasb
Copy link
Collaborator

Implements packet broadcasting.
Supports universal broadcast address 255.255.255.255 and subnet specific broadcast addresses, e.g., 192.168.1.255.

Latest mio version (0.6.16) depends on libc >= 0.2.42. I've tried
upgrading libc to 0.2.43 which is the latest at a given moment.
Unfortunately, some tests failed. But I noticed that tests pass
with 0.2.42 so didn't do proper research what caused tests failure
since this version satisfy my needs at this moment :)
Run those tests only on linux. They fail on other platforms because they
depend on Linux features like network namespaces and virtual network
devices.
and add more tests. Makes it easier to change the code.
Router connects multiple simulated machines and routes packets between them.
So it makes sense it is supposed to do broadcasting as well.
It supports 255.255.255.255 and subnet specific broadcast addresses,
e.g., 192.168.1.255 for 192.168.1.0/24 subnet.
@bkchr
Copy link

bkchr commented Jul 22, 2019

Is there any chance that this can be merged?

@povilasb
Copy link
Collaborator Author

Unfortunately it's not functioning properly :/

IP packet broadcasting does not work with TUN devices which are used to simulate networks when we use high level primitives like ipv4::{machine, router}.
I know this is possible and we probably have to use TAP devices which simulate the Ethernet. I simply haven't spent the time yet.

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