Thread (5 messages) 5 messages, 4 authors, 2020-08-31

Re: [PATCH bpf] bpf: refer to struct xdp_md in user space comments

From: Jakub Kicinski <kuba@kernel.org>
Date: 2020-08-31 22:43:22
Also in: bpf

On Thu, 20 Aug 2020 16:16:47 +0200 Daniel Borkmann wrote:
On 8/19/20 9:27 PM, Jakub Kicinski wrote:
quoted
uAPI uses xdp_md, not xdp_buff. Fix comments.

Signed-off-by: Jakub Kicinski <kuba@kernel.org>
---
  include/uapi/linux/bpf.h | 8 ++++----
  1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/include/uapi/linux/bpf.h b/include/uapi/linux/bpf.h
index 0480f893facd..cc3553a102d0 100644
--- a/include/uapi/linux/bpf.h
+++ b/include/uapi/linux/bpf.h
@@ -1554,7 +1554,7 @@ union bpf_attr {  
Needs also tooling header copy, but once that is done, it needs fixup for libbpf:

[root@pc-9 bpf]# make
   GEN      bpf_helper_defs.h
Unrecognized type 'struct xdp_md', please add it to known types!
make[1]: *** [Makefile:186: bpf_helper_defs.h] Error 1
make[1]: *** Deleting file 'bpf_helper_defs.h'
make: *** [Makefile:160: all] Error 2
[root@pc-9 bpf]#

Pls fix up and send v2, thanks.
FWIW upon closer inspection it appears that this is intentional
(even if confusing) and bpf_helpers_doc.py swaps the types to 
__sk_buff and xdp_md when generating man pages and the header.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help