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

Feat bpfd client v5 #1

Draft
wants to merge 17 commits into
base: main
Choose a base branch
from
Draft

Feat bpfd client v5 #1

wants to merge 17 commits into from

Conversation

maryamtahhan
Copy link
Collaborator

  • Rebase based on the latest branch
  • Retest.

* Update the documentation to reflect configuration
changes needed.
* Update the program to pin the xsk map to just use
xdp-loader. The previous c implementation had issues
and the version of libxdp in the container images is
too old for libxdp pinning support. Reverting to bpf
(for pinning support) would break the libxdp based
program unloading.
* Don't return an error from the bpf clean up function
when there's no program on the interface.

Signed-off-by: Maryam Tahhan <[email protected]>
1st stab at integration with bpfd. Currently, the implementation
is using a simple xdp_pass program to start with. This will need
to change to an af_xdp program from xdp-tools "xsk_def_prog".
TODO:
- Disable local BPF prog managment if this is enabled.
- Use bpfd for veth peer in kind network when it's created.
- Update the xdp program.

Signed-off-by: Maryam Tahhan <[email protected]>
Fails at the moment due to lack of support from Aya.

```
[2023-08-14T12:42:53Z WARN  aya::bpf] The map xsks_map is of type BPF_MAP_TYPE_XSKMAP which is currently unsupported in Aya, use `allow_unsupported_maps()` to load it anyways
[2023-08-14T12:42:53Z WARN  bpfd::rpc] BPFD load error: BpfLoadError(
        UnsupportedMap,
    )
```

Signed-off-by: Maryam Tahhan <[email protected]>
Signed-off-by: Maryam Tahhan <[email protected]>
Signed-off-by: Maryam Tahhan <[email protected]>
Signed-off-by: Maryam Tahhan <[email protected]>
Signed-off-by: Maryam Tahhan <[email protected]>
Still not fully working with bpfd as it seems to be loading
the program but not pinning the map.

Signed-off-by: Maryam Tahhan <[email protected]>
Temporary fix to for bpfd v0.3.0 API.

Signed-off-by: astoycos <[email protected]>
Signed-off-by: Maryam Tahhan <[email protected]>
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