Skip to content

Commit b7b7013

Browse files
rnavmpe
authored andcommitted
powerpc/bpf: Add support for bpf constant blinding
In line with similar support for other architectures by Daniel Borkmann. 'MOD Default X' from test_bpf without constant blinding: 84 bytes emitted from JIT compiler (pass:3, flen:7) d0000000058a4688 + <x>: 0: nop 4: nop 8: std r27,-40(r1) c: std r28,-32(r1) 10: xor r8,r8,r8 14: xor r28,r28,r28 18: mr r27,r3 1c: li r8,66 20: cmpwi r28,0 24: bne 0x0000000000000030 28: li r8,0 2c: b 0x0000000000000044 30: divwu r9,r8,r28 34: mullw r9,r28,r9 38: subf r8,r9,r8 3c: rotlwi r8,r8,0 40: li r8,66 44: ld r27,-40(r1) 48: ld r28,-32(r1) 4c: mr r3,r8 50: blr ... and with constant blinding: 140 bytes emitted from JIT compiler (pass:3, flen:11) d00000000bd6ab24 + <x>: 0: nop 4: nop 8: std r27,-40(r1) c: std r28,-32(r1) 10: xor r8,r8,r8 14: xor r28,r28,r28 18: mr r27,r3 1c: lis r2,-22834 20: ori r2,r2,36083 24: rotlwi r2,r2,0 28: xori r2,r2,36017 2c: xoris r2,r2,42702 30: rotlwi r2,r2,0 34: mr r8,r2 38: rotlwi r8,r8,0 3c: cmpwi r28,0 40: bne 0x000000000000004c 44: li r8,0 48: b 0x000000000000007c 4c: divwu r9,r8,r28 50: mullw r9,r28,r9 54: subf r8,r9,r8 58: rotlwi r8,r8,0 5c: lis r2,-17137 60: ori r2,r2,39065 64: rotlwi r2,r2,0 68: xori r2,r2,39131 6c: xoris r2,r2,48399 70: rotlwi r2,r2,0 74: mr r8,r2 78: rotlwi r8,r8,0 7c: ld r27,-40(r1) 80: ld r28,-32(r1) 84: mr r3,r8 88: blr Signed-off-by: Naveen N. Rao <[email protected]> Acked-by: Daniel Borkmann <[email protected]> Signed-off-by: Michael Ellerman <[email protected]>
1 parent ce07614 commit b7b7013

File tree

2 files changed

+34
-11
lines changed

2 files changed

+34
-11
lines changed

Diff for: arch/powerpc/net/bpf_jit64.h

+5-4
Original file line numberDiff line numberDiff line change
@@ -39,10 +39,10 @@
3939
#ifndef __ASSEMBLY__
4040

4141
/* BPF register usage */
42-
#define SKB_HLEN_REG (MAX_BPF_REG + 0)
43-
#define SKB_DATA_REG (MAX_BPF_REG + 1)
44-
#define TMP_REG_1 (MAX_BPF_REG + 2)
45-
#define TMP_REG_2 (MAX_BPF_REG + 3)
42+
#define SKB_HLEN_REG (MAX_BPF_JIT_REG + 0)
43+
#define SKB_DATA_REG (MAX_BPF_JIT_REG + 1)
44+
#define TMP_REG_1 (MAX_BPF_JIT_REG + 2)
45+
#define TMP_REG_2 (MAX_BPF_JIT_REG + 3)
4646

4747
/* BPF to ppc register mappings */
4848
static const int b2p[] = {
@@ -62,6 +62,7 @@ static const int b2p[] = {
6262
/* frame pointer aka BPF_REG_10 */
6363
[BPF_REG_FP] = 31,
6464
/* eBPF jit internal registers */
65+
[BPF_REG_AX] = 2,
6566
[SKB_HLEN_REG] = 25,
6667
[SKB_DATA_REG] = 26,
6768
[TMP_REG_1] = 9,

Diff for: arch/powerpc/net/bpf_jit_comp64.c

+29-7
Original file line numberDiff line numberDiff line change
@@ -974,21 +974,37 @@ struct bpf_prog *bpf_int_jit_compile(struct bpf_prog *fp)
974974
int pass;
975975
int flen;
976976
struct bpf_binary_header *bpf_hdr;
977+
struct bpf_prog *org_fp = fp;
978+
struct bpf_prog *tmp_fp;
979+
bool bpf_blinded = false;
977980

978981
if (!bpf_jit_enable)
979-
return fp;
982+
return org_fp;
983+
984+
tmp_fp = bpf_jit_blind_constants(org_fp);
985+
if (IS_ERR(tmp_fp))
986+
return org_fp;
987+
988+
if (tmp_fp != org_fp) {
989+
bpf_blinded = true;
990+
fp = tmp_fp;
991+
}
980992

981993
flen = fp->len;
982994
addrs = kzalloc((flen+1) * sizeof(*addrs), GFP_KERNEL);
983-
if (addrs == NULL)
984-
return fp;
995+
if (addrs == NULL) {
996+
fp = org_fp;
997+
goto out;
998+
}
999+
1000+
memset(&cgctx, 0, sizeof(struct codegen_context));
9851001

986-
cgctx.idx = 0;
987-
cgctx.seen = 0;
9881002
/* Scouting faux-generate pass 0 */
989-
if (bpf_jit_build_body(fp, 0, &cgctx, addrs))
1003+
if (bpf_jit_build_body(fp, 0, &cgctx, addrs)) {
9901004
/* We hit something illegal or unsupported. */
1005+
fp = org_fp;
9911006
goto out;
1007+
}
9921008

9931009
/*
9941010
* Pretend to build prologue, given the features we've seen. This will
@@ -1003,8 +1019,10 @@ struct bpf_prog *bpf_int_jit_compile(struct bpf_prog *fp)
10031019

10041020
bpf_hdr = bpf_jit_binary_alloc(alloclen, &image, 4,
10051021
bpf_jit_fill_ill_insns);
1006-
if (!bpf_hdr)
1022+
if (!bpf_hdr) {
1023+
fp = org_fp;
10071024
goto out;
1025+
}
10081026

10091027
code_base = (u32 *)(image + FUNCTION_DESCR_SIZE);
10101028

@@ -1041,6 +1059,10 @@ struct bpf_prog *bpf_int_jit_compile(struct bpf_prog *fp)
10411059

10421060
out:
10431061
kfree(addrs);
1062+
1063+
if (bpf_blinded)
1064+
bpf_jit_prog_release_other(fp, fp == org_fp ? tmp_fp : org_fp);
1065+
10441066
return fp;
10451067
}
10461068

0 commit comments

Comments
 (0)