Re: [PATCH v2] bpf: s390: add JIT support for bpf line info

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> Am 29.08.2019 um 22:02 schrieb Yauheni Kaliuta <yauheni.kaliuta@xxxxxxxxxx>:
> 
> This adds support for generating bpf line info for JITed programs
> like commit 6f20c71d8505 ("bpf: powerpc64: add JIT support for bpf
> line info") does for powerpc, but it should pass the array starting
> from 1 like x86, see commit 7c2e988f400e ("bpf: fix x64 JIT code
> generation for jmp to 1st insn".
> 
> That fixes test_btf.
> 
> Signed-off-by: Yauheni Kaliuta <yauheni.kaliuta@xxxxxxxxxx>
> ---
> 
> The patch is on top of "bpf: s390: add JIT support for multi-function
> programs"
> 
> V1->V1:
> 
> - pass address array starting from element 1.
> 
> ---
> arch/s390/net/bpf_jit_comp.c | 1 +
> 1 file changed, 1 insertion(+)
> 
> diff --git a/arch/s390/net/bpf_jit_comp.c b/arch/s390/net/bpf_jit_comp.c
> index b6801d854c77..ce88211b9c6c 100644
> --- a/arch/s390/net/bpf_jit_comp.c
> +++ b/arch/s390/net/bpf_jit_comp.c
> @@ -1420,6 +1420,7 @@ struct bpf_prog *bpf_int_jit_compile(struct bpf_prog *fp)
> 	fp->jited_len = jit.size;
> 
> 	if (!fp->is_func || extra_pass) {
> +		bpf_prog_fill_jited_linfo(fp, jit.addrs + 1);
> free_addrs:
> 		kfree(jit.addrs);
> 		kfree(jit_data);
> -- 
> 2.22.0
> 

Checkpatch complains about the missing ")" at the end of 7c2e988f400e
commit description. With that fixed:

Acked-by: Ilya Leoshkevich <iii@xxxxxxxxxxxxx>
Tested-by: Ilya Leoshkevich <iii@xxxxxxxxxxxxx>

Thanks!



[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux