OSDN Git Service

bpf, arm32: save 4 bytes of unneeded stack space
authorDaniel Borkmann <daniel@iogearbox.net>
Mon, 14 May 2018 21:22:30 +0000 (23:22 +0200)
committerAlexei Starovoitov <ast@kernel.org>
Tue, 15 May 2018 02:11:45 +0000 (19:11 -0700)
The extra skb_copy_bits() buffer is not used anymore, therefore
remove the extra 4 byte stack space requirement.

Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
arch/arm/net/bpf_jit_32.c

index 82689b9..d3ea645 100644 (file)
@@ -234,18 +234,11 @@ static void jit_fill_hole(void *area, unsigned int size)
 #define SCRATCH_SIZE 80
 
 /* total stack size used in JITed code */
-#define _STACK_SIZE \
-       (ctx->prog->aux->stack_depth + \
-        + SCRATCH_SIZE + \
-        + 4 /* extra for skb_copy_bits buffer */)
-
-#define STACK_SIZE ALIGN(_STACK_SIZE, STACK_ALIGNMENT)
+#define _STACK_SIZE    (ctx->prog->aux->stack_depth + SCRATCH_SIZE)
+#define STACK_SIZE     ALIGN(_STACK_SIZE, STACK_ALIGNMENT)
 
 /* Get the offset of eBPF REGISTERs stored on scratch space. */
-#define STACK_VAR(off) (STACK_SIZE-off-4)
-
-/* Offset of skb_copy_bits buffer */
-#define SKB_BUFFER STACK_VAR(SCRATCH_SIZE)
+#define STACK_VAR(off) (STACK_SIZE - off)
 
 #if __LINUX_ARM_ARCH__ < 7