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

Integration of MoonSniff #227

Closed
wants to merge 124 commits into from
Closed

Integration of MoonSniff #227

wants to merge 124 commits into from

Conversation

AP-Frank
Copy link
Contributor

Adds the following:

  • MoonSniff scripts for sniffer, postprocessor, traffic generator, and DUT
  • Hashmap support
  • Writer/Reader for MSCAP files

Tests with the old implementation have shown that the hit rate does not meet the expectations. The reason for this is that the time to insert a timestamp where fast, whereas time to match an existing entry was linear an took up to 100 times as long.

The new implementation aims to have equal insert and find properties.
todo: remove all unneccessary syncing points
new implementation should make errors because of
race conditions very improbable
potentially causes misses/memory errors
this is a compromise between, possible measurement time and
memory requirements
examples/moonsniff-demo.lua Outdated Show resolved Hide resolved
examples/moonsniff-demo.lua Outdated Show resolved Hide resolved
examples/moonsniff-demo.lua Outdated Show resolved Hide resolved
examples/moonsniff-demo.lua Outdated Show resolved Hide resolved
examples/moonsniff-demo.lua Outdated Show resolved Hide resolved
examples/moonsniff/traffic-gen.lua Outdated Show resolved Hide resolved
lua/moonsniff-io.lua Show resolved Hide resolved
src/hashmap.cpp Outdated Show resolved Hide resolved
src/moonsniff.cpp Outdated Show resolved Hide resolved
src/moonsniff.cpp Outdated Show resolved Hide resolved
@emmericp
Copy link
Owner

merged and squashed manually

@emmericp emmericp closed this Oct 18, 2018
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