2018-10-17 16:24 UTC+0200 ~ Nicolas Dichtel <[email protected]>
len_diff is signed.

Fixes: fa15601ab31e ("bpf: add documentation for eBPF helpers (33-41)")
CC: Quentin Monnet <[email protected]>
Signed-off-by: Nicolas Dichtel <[email protected]>
---
  include/uapi/linux/bpf.h       | 2 +-
  tools/include/uapi/linux/bpf.h | 2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/uapi/linux/bpf.h b/include/uapi/linux/bpf.h
index 66917a4eba27..c4ffe91d5598 100644
--- a/include/uapi/linux/bpf.h
+++ b/include/uapi/linux/bpf.h
@@ -1430,7 +1430,7 @@ union bpf_attr {
   *    Return
   *            0 on success, or a negative error in case of failure.
   *
- * int bpf_skb_adjust_room(struct sk_buff *skb, u32 len_diff, u32 mode, u64 
flags)
+ * int bpf_skb_adjust_room(struct sk_buff *skb, s32 len_diff, u32 mode, u64 
flags)
   *    Description
   *            Grow or shrink the room for data in the packet associated to
   *            *skb* by *len_diff*, and according to the selected *mode*.
diff --git a/tools/include/uapi/linux/bpf.h b/tools/include/uapi/linux/bpf.h
index 66917a4eba27..c4ffe91d5598 100644
--- a/tools/include/uapi/linux/bpf.h
+++ b/tools/include/uapi/linux/bpf.h
@@ -1430,7 +1430,7 @@ union bpf_attr {
   *    Return
   *            0 on success, or a negative error in case of failure.
   *
- * int bpf_skb_adjust_room(struct sk_buff *skb, u32 len_diff, u32 mode, u64 
flags)
+ * int bpf_skb_adjust_room(struct sk_buff *skb, s32 len_diff, u32 mode, u64 
flags)
   *    Description
   *            Grow or shrink the room for data in the packet associated to
   *            *skb* by *len_diff*, and according to the selected *mode*.


Correct, thank you Nicolas! :)

Reviewed-by: Quentin Monnet <[email protected]>

Reply via email to