Expose neighbor module #1102
Closed
Expose neighbor module #1102
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* add dscp and ecn fields and parse them * update an instantiation * claude assisted global fill in * update unit test with non-zero values to parse * add remaining fixed fields * wip - add to all instances * fix doc test * fix dont frag flag in tests * update emit for all fields * fix icmp test * update bench test * fix whitespace
* Added neighbor cache interface * Added manual arp requests Co-authored-by: Jeffrey Moon <[email protected]>
* preserve header length field * add options field to ipv4 repr * update emit function * add raw socket test * add raw socket tx test * update bench test * if guard * make options optional * guard options copy if no options are present * Revert "guard options copy if no options are present" This reverts commit 183d062. * Revert "make options optional" This reverts commit 556c5c8. * address comments on constants * keep header len interface as usize * cargo fmt * clean up vector size in unit test and remove unneeded constant * verify recv queue bytes in test * add set options method to repr and clean up raw tx unit test * improve interface between ipv4 packet and repr * constant for header alignment --------- Co-authored-by: Jeffrey Moon <[email protected]>
Co-authored-by: Adam Michalik <[email protected]>
Contributor
Author
|
Crap - this was meant to go into our fork. Sorry about this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ooops, merged this back into
expose-arp-tableas well for that eventual PR