Commit df9705ea authored by Kui-Feng Lee's avatar Kui-Feng Lee Committed by Martin KaFai Lau
Browse files

bpf: Remove an unnecessary check. 



The "i" here is always equal to "btf_type_vlen(t)" since
the "for_each_member()" loop never breaks.

Signed-off-by: default avatarKui-Feng Lee <thinker.li@gmail.com>
Acked-by: default avatarYonghong Song <yonghong.song@linux.dev>
Link: https://lore.kernel.org/r/20240203055119.2235598-1-thinker.li@gmail.com


Signed-off-by: default avatarMartin KaFai Lau <martin.lau@kernel.org>
parent 2a79690e

kernel/bpf/bpf_struct_ops.c

+9 −12
Original line number Diff line number Diff line
@@ -189,19 +189,16 @@ int bpf_struct_ops_desc_init(struct bpf_struct_ops_desc *st_ops_desc, 
		}

	}



	if (i == btf_type_vlen(t)) {

	if (st_ops->init(btf)) {

		pr_warn("Error in init bpf_struct_ops %s\n",

			st_ops->name);

		return -EINVAL;

		} else {

	}



	st_ops_desc->type_id = type_id;

	st_ops_desc->type = t;

	st_ops_desc->value_id = value_id;

			st_ops_desc->value_type = btf_type_by_id(btf,

								 value_id);

		}

	}

	st_ops_desc->value_type = btf_type_by_id(btf, value_id);



	return 0;

}