* [RESEND PATCH] arm64: bpf: add 'store immediate' instruction @ 2015-11-30 22:24 ` Yang Shi 0 siblings, 0 replies; 10+ messages in thread From: Yang Shi @ 2015-11-30 22:24 UTC (permalink / raw) To: linux-arm-kernel aarch64 doesn't have native store immediate instruction, such operation has to be implemented by the below instruction sequence: Load immediate to register Store register Signed-off-by: Yang Shi <yang.shi@linaro.org> CC: Zi Shen Lim <zlim.lnx@gmail.com> CC: Xi Wang <xi.wang@gmail.com> --- Thsi patch might be buried by the storm of xadd discussion, however, it is absolutely irrelevent to xadd, so resend the patch itself. arch/arm64/net/bpf_jit_comp.c | 20 +++++++++++++++++++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/arch/arm64/net/bpf_jit_comp.c b/arch/arm64/net/bpf_jit_comp.c index 6809647..49c1f1b 100644 --- a/arch/arm64/net/bpf_jit_comp.c +++ b/arch/arm64/net/bpf_jit_comp.c @@ -563,7 +563,25 @@ emit_cond_jmp: case BPF_ST | BPF_MEM | BPF_H: case BPF_ST | BPF_MEM | BPF_B: case BPF_ST | BPF_MEM | BPF_DW: - goto notyet; + /* Load imm to a register then store it */ + ctx->tmp_used = 1; + emit_a64_mov_i(1, tmp2, off, ctx); + emit_a64_mov_i(1, tmp, imm, ctx); + switch (BPF_SIZE(code)) { + case BPF_W: + emit(A64_STR32(tmp, dst, tmp2), ctx); + break; + case BPF_H: + emit(A64_STRH(tmp, dst, tmp2), ctx); + break; + case BPF_B: + emit(A64_STRB(tmp, dst, tmp2), ctx); + break; + case BPF_DW: + emit(A64_STR64(tmp, dst, tmp2), ctx); + break; + } + break; /* STX: *(size *)(dst + off) = src */ case BPF_STX | BPF_MEM | BPF_W: -- 2.0.2 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [RESEND PATCH] arm64: bpf: add 'store immediate' instruction @ 2015-11-30 22:24 ` Yang Shi 0 siblings, 0 replies; 10+ messages in thread From: Yang Shi @ 2015-11-30 22:24 UTC (permalink / raw) To: ast, catalin.marinas, will.deacon, davem Cc: zlim.lnx, xi.wang, linux-kernel, netdev, linux-arm-kernel, linaro-kernel, yang.shi aarch64 doesn't have native store immediate instruction, such operation has to be implemented by the below instruction sequence: Load immediate to register Store register Signed-off-by: Yang Shi <yang.shi@linaro.org> CC: Zi Shen Lim <zlim.lnx@gmail.com> CC: Xi Wang <xi.wang@gmail.com> --- Thsi patch might be buried by the storm of xadd discussion, however, it is absolutely irrelevent to xadd, so resend the patch itself. arch/arm64/net/bpf_jit_comp.c | 20 +++++++++++++++++++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/arch/arm64/net/bpf_jit_comp.c b/arch/arm64/net/bpf_jit_comp.c index 6809647..49c1f1b 100644 --- a/arch/arm64/net/bpf_jit_comp.c +++ b/arch/arm64/net/bpf_jit_comp.c @@ -563,7 +563,25 @@ emit_cond_jmp: case BPF_ST | BPF_MEM | BPF_H: case BPF_ST | BPF_MEM | BPF_B: case BPF_ST | BPF_MEM | BPF_DW: - goto notyet; + /* Load imm to a register then store it */ + ctx->tmp_used = 1; + emit_a64_mov_i(1, tmp2, off, ctx); + emit_a64_mov_i(1, tmp, imm, ctx); + switch (BPF_SIZE(code)) { + case BPF_W: + emit(A64_STR32(tmp, dst, tmp2), ctx); + break; + case BPF_H: + emit(A64_STRH(tmp, dst, tmp2), ctx); + break; + case BPF_B: + emit(A64_STRB(tmp, dst, tmp2), ctx); + break; + case BPF_DW: + emit(A64_STR64(tmp, dst, tmp2), ctx); + break; + } + break; /* STX: *(size *)(dst + off) = src */ case BPF_STX | BPF_MEM | BPF_W: -- 2.0.2 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [RESEND PATCH] arm64: bpf: add 'store immediate' instruction 2015-11-30 22:24 ` Yang Shi @ 2015-12-01 22:20 ` Shi, Yang -1 siblings, 0 replies; 10+ messages in thread From: Shi, Yang @ 2015-12-01 22:20 UTC (permalink / raw) To: linux-arm-kernel On 11/30/2015 2:24 PM, Yang Shi wrote: > aarch64 doesn't have native store immediate instruction, such operation > has to be implemented by the below instruction sequence: > > Load immediate to register > Store register > > Signed-off-by: Yang Shi <yang.shi@linaro.org> > CC: Zi Shen Lim <zlim.lnx@gmail.com> Had email exchange offline with Zi Shen Lim since he is traveling and cannot send text-only mail, quoted below for his reply: "I've given reviewed-by in response to original posting. Unless something has changed, feel free to add it." Since there is nothing changed, added his reviewed-by. Reviewed-by: Zi Shen Lim <zlim.lnx@gmail.com> Thanks, Yang > CC: Xi Wang <xi.wang@gmail.com> > --- > Thsi patch might be buried by the storm of xadd discussion, however, it is > absolutely irrelevent to xadd, so resend the patch itself. > > arch/arm64/net/bpf_jit_comp.c | 20 +++++++++++++++++++- > 1 file changed, 19 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/net/bpf_jit_comp.c b/arch/arm64/net/bpf_jit_comp.c > index 6809647..49c1f1b 100644 > --- a/arch/arm64/net/bpf_jit_comp.c > +++ b/arch/arm64/net/bpf_jit_comp.c > @@ -563,7 +563,25 @@ emit_cond_jmp: > case BPF_ST | BPF_MEM | BPF_H: > case BPF_ST | BPF_MEM | BPF_B: > case BPF_ST | BPF_MEM | BPF_DW: > - goto notyet; > + /* Load imm to a register then store it */ > + ctx->tmp_used = 1; > + emit_a64_mov_i(1, tmp2, off, ctx); > + emit_a64_mov_i(1, tmp, imm, ctx); > + switch (BPF_SIZE(code)) { > + case BPF_W: > + emit(A64_STR32(tmp, dst, tmp2), ctx); > + break; > + case BPF_H: > + emit(A64_STRH(tmp, dst, tmp2), ctx); > + break; > + case BPF_B: > + emit(A64_STRB(tmp, dst, tmp2), ctx); > + break; > + case BPF_DW: > + emit(A64_STR64(tmp, dst, tmp2), ctx); > + break; > + } > + break; > > /* STX: *(size *)(dst + off) = src */ > case BPF_STX | BPF_MEM | BPF_W: > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RESEND PATCH] arm64: bpf: add 'store immediate' instruction @ 2015-12-01 22:20 ` Shi, Yang 0 siblings, 0 replies; 10+ messages in thread From: Shi, Yang @ 2015-12-01 22:20 UTC (permalink / raw) To: ast, catalin.marinas, will.deacon, davem Cc: zlim.lnx, xi.wang, linux-kernel, netdev, linux-arm-kernel, linaro-kernel On 11/30/2015 2:24 PM, Yang Shi wrote: > aarch64 doesn't have native store immediate instruction, such operation > has to be implemented by the below instruction sequence: > > Load immediate to register > Store register > > Signed-off-by: Yang Shi <yang.shi@linaro.org> > CC: Zi Shen Lim <zlim.lnx@gmail.com> Had email exchange offline with Zi Shen Lim since he is traveling and cannot send text-only mail, quoted below for his reply: "I've given reviewed-by in response to original posting. Unless something has changed, feel free to add it." Since there is nothing changed, added his reviewed-by. Reviewed-by: Zi Shen Lim <zlim.lnx@gmail.com> Thanks, Yang > CC: Xi Wang <xi.wang@gmail.com> > --- > Thsi patch might be buried by the storm of xadd discussion, however, it is > absolutely irrelevent to xadd, so resend the patch itself. > > arch/arm64/net/bpf_jit_comp.c | 20 +++++++++++++++++++- > 1 file changed, 19 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/net/bpf_jit_comp.c b/arch/arm64/net/bpf_jit_comp.c > index 6809647..49c1f1b 100644 > --- a/arch/arm64/net/bpf_jit_comp.c > +++ b/arch/arm64/net/bpf_jit_comp.c > @@ -563,7 +563,25 @@ emit_cond_jmp: > case BPF_ST | BPF_MEM | BPF_H: > case BPF_ST | BPF_MEM | BPF_B: > case BPF_ST | BPF_MEM | BPF_DW: > - goto notyet; > + /* Load imm to a register then store it */ > + ctx->tmp_used = 1; > + emit_a64_mov_i(1, tmp2, off, ctx); > + emit_a64_mov_i(1, tmp, imm, ctx); > + switch (BPF_SIZE(code)) { > + case BPF_W: > + emit(A64_STR32(tmp, dst, tmp2), ctx); > + break; > + case BPF_H: > + emit(A64_STRH(tmp, dst, tmp2), ctx); > + break; > + case BPF_B: > + emit(A64_STRB(tmp, dst, tmp2), ctx); > + break; > + case BPF_DW: > + emit(A64_STR64(tmp, dst, tmp2), ctx); > + break; > + } > + break; > > /* STX: *(size *)(dst + off) = src */ > case BPF_STX | BPF_MEM | BPF_W: > ^ permalink raw reply [flat|nested] 10+ messages in thread
* [RESEND PATCH] arm64: bpf: add 'store immediate' instruction 2015-12-01 22:20 ` Shi, Yang @ 2015-12-02 9:15 ` Will Deacon -1 siblings, 0 replies; 10+ messages in thread From: Will Deacon @ 2015-12-02 9:15 UTC (permalink / raw) To: linux-arm-kernel On Tue, Dec 01, 2015 at 02:20:40PM -0800, Shi, Yang wrote: > On 11/30/2015 2:24 PM, Yang Shi wrote: > >aarch64 doesn't have native store immediate instruction, such operation > >has to be implemented by the below instruction sequence: > > > >Load immediate to register > >Store register > > > >Signed-off-by: Yang Shi <yang.shi@linaro.org> > >CC: Zi Shen Lim <zlim.lnx@gmail.com> > > Had email exchange offline with Zi Shen Lim since he is traveling and cannot > send text-only mail, quoted below for his reply: > > "I've given reviewed-by in response to original posting. Unless something > has changed, feel free to add it." > > Since there is nothing changed, added his reviewed-by. > > Reviewed-by: Zi Shen Lim <zlim.lnx@gmail.com> I assume David will take this via netdev. Will ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RESEND PATCH] arm64: bpf: add 'store immediate' instruction @ 2015-12-02 9:15 ` Will Deacon 0 siblings, 0 replies; 10+ messages in thread From: Will Deacon @ 2015-12-02 9:15 UTC (permalink / raw) To: Shi, Yang Cc: ast, catalin.marinas, davem, zlim.lnx, xi.wang, linux-kernel, netdev, linux-arm-kernel, linaro-kernel On Tue, Dec 01, 2015 at 02:20:40PM -0800, Shi, Yang wrote: > On 11/30/2015 2:24 PM, Yang Shi wrote: > >aarch64 doesn't have native store immediate instruction, such operation > >has to be implemented by the below instruction sequence: > > > >Load immediate to register > >Store register > > > >Signed-off-by: Yang Shi <yang.shi@linaro.org> > >CC: Zi Shen Lim <zlim.lnx@gmail.com> > > Had email exchange offline with Zi Shen Lim since he is traveling and cannot > send text-only mail, quoted below for his reply: > > "I've given reviewed-by in response to original posting. Unless something > has changed, feel free to add it." > > Since there is nothing changed, added his reviewed-by. > > Reviewed-by: Zi Shen Lim <zlim.lnx@gmail.com> I assume David will take this via netdev. Will ^ permalink raw reply [flat|nested] 10+ messages in thread
* [RESEND PATCH] arm64: bpf: add 'store immediate' instruction 2015-12-02 9:15 ` Will Deacon @ 2015-12-02 20:42 ` David Miller -1 siblings, 0 replies; 10+ messages in thread From: David Miller @ 2015-12-02 20:42 UTC (permalink / raw) To: linux-arm-kernel From: Will Deacon <will.deacon@arm.com> Date: Wed, 2 Dec 2015 09:15:18 +0000 > On Tue, Dec 01, 2015 at 02:20:40PM -0800, Shi, Yang wrote: >> On 11/30/2015 2:24 PM, Yang Shi wrote: >> >aarch64 doesn't have native store immediate instruction, such operation >> >has to be implemented by the below instruction sequence: >> > >> >Load immediate to register >> >Store register >> > >> >Signed-off-by: Yang Shi <yang.shi@linaro.org> >> >CC: Zi Shen Lim <zlim.lnx@gmail.com> >> >> Had email exchange offline with Zi Shen Lim since he is traveling and cannot >> send text-only mail, quoted below for his reply: >> >> "I've given reviewed-by in response to original posting. Unless something >> has changed, feel free to add it." >> >> Since there is nothing changed, added his reviewed-by. >> >> Reviewed-by: Zi Shen Lim <zlim.lnx@gmail.com> > > I assume David will take this via netdev. Yes, I will, thanks. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RESEND PATCH] arm64: bpf: add 'store immediate' instruction @ 2015-12-02 20:42 ` David Miller 0 siblings, 0 replies; 10+ messages in thread From: David Miller @ 2015-12-02 20:42 UTC (permalink / raw) To: will.deacon Cc: yang.shi, ast, catalin.marinas, zlim.lnx, xi.wang, linux-kernel, netdev, linux-arm-kernel, linaro-kernel From: Will Deacon <will.deacon@arm.com> Date: Wed, 2 Dec 2015 09:15:18 +0000 > On Tue, Dec 01, 2015 at 02:20:40PM -0800, Shi, Yang wrote: >> On 11/30/2015 2:24 PM, Yang Shi wrote: >> >aarch64 doesn't have native store immediate instruction, such operation >> >has to be implemented by the below instruction sequence: >> > >> >Load immediate to register >> >Store register >> > >> >Signed-off-by: Yang Shi <yang.shi@linaro.org> >> >CC: Zi Shen Lim <zlim.lnx@gmail.com> >> >> Had email exchange offline with Zi Shen Lim since he is traveling and cannot >> send text-only mail, quoted below for his reply: >> >> "I've given reviewed-by in response to original posting. Unless something >> has changed, feel free to add it." >> >> Since there is nothing changed, added his reviewed-by. >> >> Reviewed-by: Zi Shen Lim <zlim.lnx@gmail.com> > > I assume David will take this via netdev. Yes, I will, thanks. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [RESEND PATCH] arm64: bpf: add 'store immediate' instruction 2015-11-30 22:24 ` Yang Shi @ 2015-12-03 16:38 ` David Miller -1 siblings, 0 replies; 10+ messages in thread From: David Miller @ 2015-12-03 16:38 UTC (permalink / raw) To: linux-arm-kernel From: Yang Shi <yang.shi@linaro.org> Date: Mon, 30 Nov 2015 14:24:07 -0800 > aarch64 doesn't have native store immediate instruction, such operation > has to be implemented by the below instruction sequence: > > Load immediate to register > Store register > > Signed-off-by: Yang Shi <yang.shi@linaro.org> Applied, thanks. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RESEND PATCH] arm64: bpf: add 'store immediate' instruction @ 2015-12-03 16:38 ` David Miller 0 siblings, 0 replies; 10+ messages in thread From: David Miller @ 2015-12-03 16:38 UTC (permalink / raw) To: yang.shi Cc: ast, catalin.marinas, will.deacon, zlim.lnx, xi.wang, linux-kernel, netdev, linux-arm-kernel, linaro-kernel From: Yang Shi <yang.shi@linaro.org> Date: Mon, 30 Nov 2015 14:24:07 -0800 > aarch64 doesn't have native store immediate instruction, such operation > has to be implemented by the below instruction sequence: > > Load immediate to register > Store register > > Signed-off-by: Yang Shi <yang.shi@linaro.org> Applied, thanks. ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2015-12-03 16:38 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-11-30 22:24 [RESEND PATCH] arm64: bpf: add 'store immediate' instruction Yang Shi 2015-11-30 22:24 ` Yang Shi 2015-12-01 22:20 ` Shi, Yang 2015-12-01 22:20 ` Shi, Yang 2015-12-02 9:15 ` Will Deacon 2015-12-02 9:15 ` Will Deacon 2015-12-02 20:42 ` David Miller 2015-12-02 20:42 ` David Miller 2015-12-03 16:38 ` David Miller 2015-12-03 16:38 ` David Miller
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.