diff mbox series

[bpf-next] bpf: add comments to BPF ld/ldx sizes

Message ID 151618713588.21869.16178302277967693140.stgit@firesoul
State Accepted, archived
Delegated to: BPF Maintainers
Headers show
Series [bpf-next] bpf: add comments to BPF ld/ldx sizes | expand

Commit Message

Jesper Dangaard Brouer Jan. 17, 2018, 11:05 a.m. UTC
Doc BPF ld/ldx size defines as comments in code, as it makes in
faster to lookup in a programming/review setting, than looking up
the sizes in Documentation/networking/filter.txt.

Signed-off-by: Jesper Dangaard Brouer <brouer@redhat.com>
---
 include/uapi/linux/bpf.h        |    2 +-
 include/uapi/linux/bpf_common.h |    7 ++++---
 2 files changed, 5 insertions(+), 4 deletions(-)

Comments

Daniel Borkmann Jan. 18, 2018, 9:14 p.m. UTC | #1
On 01/17/2018 12:05 PM, Jesper Dangaard Brouer wrote:
> Doc BPF ld/ldx size defines as comments in code, as it makes in
> faster to lookup in a programming/review setting, than looking up
> the sizes in Documentation/networking/filter.txt.
> 
> Signed-off-by: Jesper Dangaard Brouer <brouer@redhat.com>

Applied to bpf-next, thanks Jesper!
diff mbox series

Patch

diff --git a/include/uapi/linux/bpf.h b/include/uapi/linux/bpf.h
index 395d261948de..4729d9a002d4 100644
--- a/include/uapi/linux/bpf.h
+++ b/include/uapi/linux/bpf.h
@@ -17,7 +17,7 @@ 
 #define BPF_ALU64	0x07	/* alu mode in double word width */
 
 /* ld/ldx fields */
-#define BPF_DW		0x18	/* double word */
+#define BPF_DW		0x18	/* double word (64-bit) */
 #define BPF_XADD	0xc0	/* exclusive add */
 
 /* alu/jmp fields */
diff --git a/include/uapi/linux/bpf_common.h b/include/uapi/linux/bpf_common.h
index 18be90725ab0..ee97668bdadb 100644
--- a/include/uapi/linux/bpf_common.h
+++ b/include/uapi/linux/bpf_common.h
@@ -15,9 +15,10 @@ 
 
 /* ld/ldx fields */
 #define BPF_SIZE(code)  ((code) & 0x18)
-#define		BPF_W		0x00
-#define		BPF_H		0x08
-#define		BPF_B		0x10
+#define		BPF_W		0x00 /* 32-bit */
+#define		BPF_H		0x08 /* 16-bit */
+#define		BPF_B		0x10 /*  8-bit */
+/* eBPF		BPF_DW		0x18    64-bit */
 #define BPF_MODE(code)  ((code) & 0xe0)
 #define		BPF_IMM		0x00
 #define		BPF_ABS		0x20