Commit 9919c5c9 authored by Rafael Passos's avatar Rafael Passos Committed by Alexei Starovoitov
Browse files

bpf: remove unused parameter in bpf_jit_binary_pack_finalize 



Fixes a compiler warning. the bpf_jit_binary_pack_finalize function
was taking an extra bpf_prog parameter that went unused.
This removves it and updates the callers accordingly.

Signed-off-by: default avatarRafael Passos <rafael@rcpassos.me>
Link: https://lore.kernel.org/r/20240615022641.210320-2-rafael@rcpassos.me


Signed-off-by: default avatarAlexei Starovoitov <ast@kernel.org>
parent 3b063043

arch/arm64/net/bpf_jit_comp.c

+1 −2
Original line number Diff line number Diff line
@@ -1829,8 +1829,7 @@ struct bpf_prog *bpf_int_jit_compile(struct bpf_prog *prog) 
			prog->jited_len = 0;

			goto out_free_hdr;

		}

		if (WARN_ON(bpf_jit_binary_pack_finalize(prog, ro_header,

							 header))) {

		if (WARN_ON(bpf_jit_binary_pack_finalize(ro_header, header))) {

			/* ro_header has been freed */

			ro_header = NULL;

			prog = orig_prog;

arch/powerpc/net/bpf_jit_comp.c

+2 −2
Original line number Diff line number Diff line
@@ -225,7 +225,7 @@ struct bpf_prog *bpf_int_jit_compile(struct bpf_prog *fp) 
	fp->jited_len = proglen + FUNCTION_DESCR_SIZE;



	if (!fp->is_func || extra_pass) {

		if (bpf_jit_binary_pack_finalize(fp, fhdr, hdr)) {

		if (bpf_jit_binary_pack_finalize(fhdr, hdr)) {

			fp = org_fp;

			goto out_addrs;

		}
@@ -348,7 +348,7 @@ void bpf_jit_free(struct bpf_prog *fp) 
		 * before freeing it.

		 */

		if (jit_data) {

			bpf_jit_binary_pack_finalize(fp, jit_data->fhdr, jit_data->hdr);

			bpf_jit_binary_pack_finalize(jit_data->fhdr, jit_data->hdr);

			kvfree(jit_data->addrs);

			kfree(jit_data);

		}

arch/riscv/net/bpf_jit_core.c

+2 −3
Original line number Diff line number Diff line
@@ -178,8 +178,7 @@ struct bpf_prog *bpf_int_jit_compile(struct bpf_prog *prog) 
	prog->jited_len = prog_size - cfi_get_offset();



	if (!prog->is_func || extra_pass) {

		if (WARN_ON(bpf_jit_binary_pack_finalize(prog, jit_data->ro_header,

							 jit_data->header))) {

		if (WARN_ON(bpf_jit_binary_pack_finalize(jit_data->ro_header, jit_data->header))) {

			/* ro_header has been freed */

			jit_data->ro_header = NULL;

			prog = orig_prog;
@@ -258,7 +257,7 @@ void bpf_jit_free(struct bpf_prog *prog) 
		 * before freeing it.

		 */

		if (jit_data) {

			bpf_jit_binary_pack_finalize(prog, jit_data->ro_header, jit_data->header);

			bpf_jit_binary_pack_finalize(jit_data->ro_header, jit_data->header);

			kfree(jit_data);

		}

		hdr = bpf_jit_binary_pack_hdr(prog);

arch/x86/net/bpf_jit_comp.c

+2 −2
Original line number Diff line number Diff line
@@ -3356,7 +3356,7 @@ struct bpf_prog *bpf_int_jit_compile(struct bpf_prog *prog) 
			 *

			 * Both cases are serious bugs and justify WARN_ON.

			 */

			if (WARN_ON(bpf_jit_binary_pack_finalize(prog, header, rw_header))) {

			if (WARN_ON(bpf_jit_binary_pack_finalize(header, rw_header))) {

				/* header has been freed */

				header = NULL;

				goto out_image;
@@ -3435,7 +3435,7 @@ void bpf_jit_free(struct bpf_prog *prog) 
		 * before freeing it.

		 */

		if (jit_data) {

			bpf_jit_binary_pack_finalize(prog, jit_data->header,

			bpf_jit_binary_pack_finalize(jit_data->header,

						     jit_data->rw_header);

			kvfree(jit_data->addrs);

			kfree(jit_data);

include/linux/filter.h

+1 −2
Original line number Diff line number Diff line
@@ -1129,8 +1129,7 @@ bpf_jit_binary_pack_alloc(unsigned int proglen, u8 **ro_image, 
			  struct bpf_binary_header **rw_hdr,

			  u8 **rw_image,

			  bpf_jit_fill_hole_t bpf_fill_ill_insns);

int bpf_jit_binary_pack_finalize(struct bpf_prog *prog,

				 struct bpf_binary_header *ro_header,

int bpf_jit_binary_pack_finalize(struct bpf_binary_header *ro_header,

				 struct bpf_binary_header *rw_header);

void bpf_jit_binary_pack_free(struct bpf_binary_header *ro_header,

			      struct bpf_binary_header *rw_header);