Skip to content

Commit 64fdb53

Browse files
Xu KuohaiNobody
Xu Kuohai
authored and
Nobody
committed
arm64: insn: add ldr/str with immediate offset
This patch introduces ldr/str with immediate offset support to simplify the JIT implementation of BPF LDX/STX instructions on arm64. Although arm64 ldr/str immediate is available in pre-index, post-index and unsigned offset forms, the unsigned offset form is sufficient for BPF, so this patch only adds this type. Signed-off-by: Xu Kuohai <[email protected]>
1 parent 5f84953 commit 64fdb53

File tree

2 files changed

+62
-14
lines changed

2 files changed

+62
-14
lines changed

arch/arm64/include/asm/insn.h

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -200,6 +200,8 @@ enum aarch64_insn_size_type {
200200
enum aarch64_insn_ldst_type {
201201
AARCH64_INSN_LDST_LOAD_REG_OFFSET,
202202
AARCH64_INSN_LDST_STORE_REG_OFFSET,
203+
AARCH64_INSN_LDST_LOAD_IMM_OFFSET,
204+
AARCH64_INSN_LDST_STORE_IMM_OFFSET,
203205
AARCH64_INSN_LDST_LOAD_PAIR_PRE_INDEX,
204206
AARCH64_INSN_LDST_STORE_PAIR_PRE_INDEX,
205207
AARCH64_INSN_LDST_LOAD_PAIR_POST_INDEX,
@@ -334,13 +336,15 @@ __AARCH64_INSN_FUNCS(load_pre, 0x3FE00C00, 0x38400C00)
334336
__AARCH64_INSN_FUNCS(store_post, 0x3FE00C00, 0x38000400)
335337
__AARCH64_INSN_FUNCS(load_post, 0x3FE00C00, 0x38400400)
336338
__AARCH64_INSN_FUNCS(str_reg, 0x3FE0EC00, 0x38206800)
339+
__AARCH64_INSN_FUNCS(str_imm, 0x3FC00000, 0x39000000)
337340
__AARCH64_INSN_FUNCS(ldadd, 0x3F20FC00, 0x38200000)
338341
__AARCH64_INSN_FUNCS(ldclr, 0x3F20FC00, 0x38201000)
339342
__AARCH64_INSN_FUNCS(ldeor, 0x3F20FC00, 0x38202000)
340343
__AARCH64_INSN_FUNCS(ldset, 0x3F20FC00, 0x38203000)
341344
__AARCH64_INSN_FUNCS(swp, 0x3F20FC00, 0x38208000)
342345
__AARCH64_INSN_FUNCS(cas, 0x3FA07C00, 0x08A07C00)
343346
__AARCH64_INSN_FUNCS(ldr_reg, 0x3FE0EC00, 0x38606800)
347+
__AARCH64_INSN_FUNCS(ldr_imm, 0x3FC00000, 0x39400000)
344348
__AARCH64_INSN_FUNCS(ldr_lit, 0xBF000000, 0x18000000)
345349
__AARCH64_INSN_FUNCS(ldrsw_lit, 0xFF000000, 0x98000000)
346350
__AARCH64_INSN_FUNCS(exclusive, 0x3F800000, 0x08000000)
@@ -500,6 +504,11 @@ u32 aarch64_insn_gen_load_store_reg(enum aarch64_insn_register reg,
500504
enum aarch64_insn_register offset,
501505
enum aarch64_insn_size_type size,
502506
enum aarch64_insn_ldst_type type);
507+
u32 aarch64_insn_gen_load_store_imm(enum aarch64_insn_register reg,
508+
enum aarch64_insn_register base,
509+
unsigned int imm,
510+
enum aarch64_insn_size_type size,
511+
enum aarch64_insn_ldst_type type);
503512
u32 aarch64_insn_gen_load_store_pair(enum aarch64_insn_register reg1,
504513
enum aarch64_insn_register reg2,
505514
enum aarch64_insn_register base,

arch/arm64/lib/insn.c

Lines changed: 53 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -299,29 +299,24 @@ static u32 aarch64_insn_encode_register(enum aarch64_insn_register_type type,
299299
return insn;
300300
}
301301

302+
static const u32 aarch64_insn_ldst_size[] = {
303+
[AARCH64_INSN_SIZE_8] = 0,
304+
[AARCH64_INSN_SIZE_16] = 1,
305+
[AARCH64_INSN_SIZE_32] = 2,
306+
[AARCH64_INSN_SIZE_64] = 3,
307+
};
308+
302309
static u32 aarch64_insn_encode_ldst_size(enum aarch64_insn_size_type type,
303310
u32 insn)
304311
{
305312
u32 size;
306313

307-
switch (type) {
308-
case AARCH64_INSN_SIZE_8:
309-
size = 0;
310-
break;
311-
case AARCH64_INSN_SIZE_16:
312-
size = 1;
313-
break;
314-
case AARCH64_INSN_SIZE_32:
315-
size = 2;
316-
break;
317-
case AARCH64_INSN_SIZE_64:
318-
size = 3;
319-
break;
320-
default:
314+
if (type < AARCH64_INSN_SIZE_8 || type > AARCH64_INSN_SIZE_64) {
321315
pr_err("%s: unknown size encoding %d\n", __func__, type);
322316
return AARCH64_BREAK_FAULT;
323317
}
324318

319+
size = aarch64_insn_ldst_size[type];
325320
insn &= ~GENMASK(31, 30);
326321
insn |= size << 30;
327322

@@ -504,6 +499,50 @@ u32 aarch64_insn_gen_load_store_reg(enum aarch64_insn_register reg,
504499
offset);
505500
}
506501

502+
u32 aarch64_insn_gen_load_store_imm(enum aarch64_insn_register reg,
503+
enum aarch64_insn_register base,
504+
unsigned int imm,
505+
enum aarch64_insn_size_type size,
506+
enum aarch64_insn_ldst_type type)
507+
{
508+
u32 insn;
509+
u32 shift;
510+
511+
if (size < AARCH64_INSN_SIZE_8 || size > AARCH64_INSN_SIZE_64) {
512+
pr_err("%s: unknown size encoding %d\n", __func__, type);
513+
return AARCH64_BREAK_FAULT;
514+
}
515+
516+
shift = aarch64_insn_ldst_size[size];
517+
if (imm & ~(BIT(12 + shift) - BIT(shift))) {
518+
pr_err("%s: invalid imm: %d\n", __func__, imm);
519+
return AARCH64_BREAK_FAULT;
520+
}
521+
522+
imm >>= shift;
523+
524+
switch (type) {
525+
case AARCH64_INSN_LDST_LOAD_IMM_OFFSET:
526+
insn = aarch64_insn_get_ldr_imm_value();
527+
break;
528+
case AARCH64_INSN_LDST_STORE_IMM_OFFSET:
529+
insn = aarch64_insn_get_str_imm_value();
530+
break;
531+
default:
532+
pr_err("%s: unknown load/store encoding %d\n", __func__, type);
533+
return AARCH64_BREAK_FAULT;
534+
}
535+
536+
insn = aarch64_insn_encode_ldst_size(size, insn);
537+
538+
insn = aarch64_insn_encode_register(AARCH64_INSN_REGTYPE_RT, insn, reg);
539+
540+
insn = aarch64_insn_encode_register(AARCH64_INSN_REGTYPE_RN, insn,
541+
base);
542+
543+
return aarch64_insn_encode_immediate(AARCH64_INSN_IMM_12, insn, imm);
544+
}
545+
507546
u32 aarch64_insn_gen_load_store_pair(enum aarch64_insn_register reg1,
508547
enum aarch64_insn_register reg2,
509548
enum aarch64_insn_register base,

0 commit comments

Comments
 (0)