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

Update libbpf submodule to v1.0 #10

Merged
merged 1 commit into from
Sep 20, 2022
Merged

Update libbpf submodule to v1.0 #10

merged 1 commit into from
Sep 20, 2022

Conversation

tohojo
Copy link
Member

@tohojo tohojo commented Sep 20, 2022

Update the libbpf submodule, and update the code to remove all usage of
deprecated libbpf APIs.

Compile tested only...

Update the libbpf submodule, and update the code to remove all usage of
deprecated libbpf APIs.

Signed-off-by: Toke Høiland-Jørgensen <[email protected]>
@tohojo
Copy link
Member Author

tohojo commented Sep 20, 2022

Cc @interduo

@interduo
Copy link

This was that. No warnings with new lib. Could You merge that with master?

@tohojo
Copy link
Member Author

tohojo commented Sep 20, 2022

Yup, will do. Also Cc @thebracket FYI

@tohojo tohojo merged commit 8547b7a into master Sep 20, 2022
@tohojo tohojo deleted the libbpf-1.0 branch September 20, 2022 11:36
@interduo
Copy link

interduo commented Sep 22, 2022

@tohojo tried to compile xdp-cpumap-tc with libbpf from master branch.

In file included from /usr/include/linux/bpf.h:11:
/usr/include/linux/types.h:5:10: fatal error: 'asm/types.h' file not found
#include <asm/types.h>
         ^~~~~~~~~~~~~
1 error generated.
make: *** [Makefile:83: xdp_pass_kern.o] Error 1

Do I need any special package in ubuntu for that?

@interduo
Copy link

Ok - there is a need to install gcc-multilib package for ubuntu

@interduo
Copy link

interduo commented Sep 22, 2022

@tohojo only this error ./xdp-cpumap-tc/bin/xps_setup.sh: line 103: echo: write error: No such file or directory I made an issue here: #12

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.

2 participants