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

[New Feature] Packet Sniffer #105

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

Conversation

LuhaoLiu
Copy link

@LuhaoLiu LuhaoLiu commented Jan 8, 2025

Description

Packet Sniffer for Coyote shell. This project was conducted as part of a semester project at ETH Zurich.
The artifact is in ./example_hw/apps/packet_sniffer/ and ./example_sw/apps/packet_sniffer/.
A simple documentation and some annotation about how to use the software (on a deployed Coyote shell) to capture network traffic are provided in ./exmaple_sw/apps/packet_sniffer/README.md.
More details about the system design and performance impact are discussed in another report. Please ask me for the link of this pdf report (still in draft).

Type of change

  • Bug fix
  • New feature
  • Documentation update
  • A new research paper code implementation
  • Other

Tests & Results

More details will be provided in the aforementioned pdf report.
One point to mention here: When this new feature is disabled by EN_SNIFFER 0 in CMake (which is the default), nothing will be essentially changed thanks to the macros. So merging this branch into the master branch WILL NOT affect anything unless you manually enable the new feature for relevant experiments or debugging.

Checklist

  • I have commented my code and made corresponding changes to the documentation.
  • Mostly. Please tell me if more is better.
  • I have added tests/results that prove my fix is effective or that my feature works.
  • This is not a FIX so not applicable. But some performance tests are feature works.
  • My changes generate no new warnings or errors & all tests successfully pass.
  • As mentioned, this merge will not affect anything by default.

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.

1 participant