[New Feature] Packet Sniffer

[LuhaoLiu]

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.

[bo3z]

@LuhaoLiu: As discussed, this PR looks mostly good to me. When you get the chance could you update it to work with the lates Coyote release and after that, we can hopefully merge it soon. Would like to include the traffic sniffer for the next Coyote release.

[LuhaoLiu]

@bo3z I've resolved all conflicts and confirmed that the packet sniffer is working as intended. I believe it is good to merge now. Please give a check when you get some time. Thanks!

[bo3z]

This mostly looks good to me. The requested changes are mostly cosmetic, extra comments, making sure the licence is there etc.

Two additional comments:

• Have you tested the RDMA example, but with the traffic sniffer code base? Just to confirm that nothing in the network_top and dynamic_top affects existing functionality. If not, could you please do it.

• Later today, I will be open-sourcing the Python binding for Coyote and we have implementations of all of the examples in Python. Would you mind adding one for the traffic sniffer (if you think it's possible)

[bo3z]

New changes look mostly ready to me. As soon as you can confirm that the RDMA example works as expected, and change the licence date, I am happy to merge this PR.

[LuhaoLiu]

@bo3z Thanks! Sorry for the delayed response, but now I've verified that the RDMA example works as expected. Also, the packet sniffer example now meets the timing constraints with the optimization option and the debug ILA removed.

[bo3z]

Excellent, merging.