On 03/15, Martin KaFai Lau wrote: > From: Martin KaFai Lau <martin.lau@xxxxxxxxxx> > > There is a "if (err)" check earlier, so the "if (err < 0)" > check that this patch removing is unnecessary. It was my overlook > when making adjustments to the bpf_struct_ops_prepare_trampoline() > such that the caller does not have to worry about the new page when > the function returns error. > > Cc: Kui-Feng Lee <thinker.li@xxxxxxxxx> > Fixes: 187e2af05abe ("bpf: struct_ops supports more than one page for trampolines.") > Signed-off-by: Martin KaFai Lau <martin.lau@xxxxxxxxxx> Acked-by: Stanislav Fomichev <sdf@xxxxxxxxxx>