pping/TODO.md

# TODO

## For initial merge
- [x] Clean up commits and add signed-off-by tags
- [x] Add SPDX-license-identifier tags
- [x] Format C-code in kernel style
- [] Use existing funcionality to reuse maps by using BTF-defined maps

## Future
- [] Use libxdp to load XDP program
- [] Cleanup: Unload TC-BPF at program shutdown, and unpin and delete map - In userspace part
- [] Add IPv6 support - In TC-BPF, XDP and userspace part
- [] Check for existance of reverse flow before adding to hash-map (to avoid adding timestamps for flows that we can't see the reverse traffic for) - In TC-BPF part
  -  This could miss the first few packets, would not be ideal for short flows
- [] Keep track of minimum RTT for each flow (done by Pollere's pping, and helps identify buffer bloat) - In XDP part
- [] Add configurable rate-limit for how often each flow can add entries to the map (prevent high-rate flows from quickly filling up the map) - In TCP-BPF part
- [] Improve map cleaning: Use a dynamic time to live for hash map entries based on flow's RTT, instead of static 10s limit - In TC-BPF, XDP and userspace
pping: Format code and add SPDX lincense tags Format the code using the .clang-format from the kernel source tree, with a few manual tweaks here and there. Also, remove the TODO list from comment of pping.c and instead put it in TODO.md. Signed-off-by: Simon Sundberg <simon.sundberg@kau.se> 2021-01-18 13:13:51 +01:00			`# TODO`

			`## For initial merge`
			`- [x] Clean up commits and add signed-off-by tags`
			`- [x] Add SPDX-license-identifier tags`
			`- [x] Format C-code in kernel style`
			`- [] Use existing funcionality to reuse maps by using BTF-defined maps`

			`## Future`
			`- [] Use libxdp to load XDP program`
			`- [] Cleanup: Unload TC-BPF at program shutdown, and unpin and delete map - In userspace part`
			`- [] Add IPv6 support - In TC-BPF, XDP and userspace part`
			`- [] Check for existance of reverse flow before adding to hash-map (to avoid adding timestamps for flows that we can't see the reverse traffic for) - In TC-BPF part`
			`- This could miss the first few packets, would not be ideal for short flows`
			`- [] Keep track of minimum RTT for each flow (done by Pollere's pping, and helps identify buffer bloat) - In XDP part`
			`- [] Add configurable rate-limit for how often each flow can add entries to the map (prevent high-rate flows from quickly filling up the map) - In TCP-BPF part`
			`- [] Improve map cleaning: Use a dynamic time to live for hash map entries based on flow's RTT, instead of static 10s limit - In TC-BPF, XDP and userspace`