From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933167AbeALEVz (ORCPT + 1 other); Thu, 11 Jan 2018 23:21:55 -0500 Received: from mail-pf0-f195.google.com ([209.85.192.195]:37470 "EHLO mail-pf0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932927AbeALEVw (ORCPT ); Thu, 11 Jan 2018 23:21:52 -0500 X-Google-Smtp-Source: ACJfBot/yDCJamAbUuz2loy5IoZONM4gwKfiWCh+MfWygovXFBFPr1J94dKk3zA5uyfyuja2xPV2TA== Date: Thu, 11 Jan 2018 20:21:49 -0800 From: Alexei Starovoitov To: David Miller Cc: sfr@canb.auug.org.au, netdev@vger.kernel.org, daniel@iogearbox.net, ast@kernel.org, linux-next@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: linux-next: build failure after merge of the net-next tree Message-ID: <20180112042148.jsxb26fodw3fgneb@ast-mbp> References: <20180111115355.29c2f905@canb.auug.org.au> <20180111015853.2pmxtqlrsyxelqf6@ast-mbp> <20180111.221145.643386077098131140.davem@davemloft.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180111.221145.643386077098131140.davem@davemloft.net> User-Agent: NeoMutt/20170421 (1.8.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On Thu, Jan 11, 2018 at 10:11:45PM -0500, David Miller wrote: > From: Alexei Starovoitov > Date: Wed, 10 Jan 2018 17:58:54 -0800 > > > On Thu, Jan 11, 2018 at 11:53:55AM +1100, Stephen Rothwell wrote: > >> Hi all, > >> > >> After merging the net-next tree, today's linux-next build (x86_64 > >> allmodconfig) failed like this: > >> > >> kernel/bpf/verifier.o: In function `bpf_check': > >> verifier.c:(.text+0xd86e): undefined reference to `bpf_patch_call_args' > >> > >> Caused by commit > >> > >> 1ea47e01ad6e ("bpf: add support for bpf_call to interpreter") > >> > >> interacting with commit > >> > >> 290af86629b2 ("bpf: introduce BPF_JIT_ALWAYS_ON config") > >> > >> from the bpf and net trees. > >> > >> I have just reverted commit 290af86629b2 for today. A better solution > >> would be nice (lie fixing this in a merge between the net-next and net > >> trees). > > > > that's due to 'endif' from 290af86629b2 needs to be moved above > > bpf_patch_call_args() definition. > > That doesn't fix it, because then you'd need to expose > interpreters_args as well and obviously that can't be right. > > Instead, we should never call bpf_patch_call_args() when JIT always on > is enabled. So if we fail to JIT the subprogs we should fail > immediately. right, as I was trying to say one extra hunk would be needed for net-next. I was reading this patch: diff --git a/kernel/bpf/verifier.c b/kernel/bpf/verifier.c index a2b211262c25..ca80559c4ec3 100644 --- a/kernel/bpf/verifier.c +++ b/kernel/bpf/verifier.c @@ -5267,7 +5267,11 @@ static int fixup_call_args(struct bpf_verifier_env *env) depth = get_callee_stack_depth(env, insn, i); if (depth < 0) return depth; +#ifdef CONFIG_BPF_JIT_ALWAYS_ON + return -ENOTSUPP; +#else bpf_patch_call_args(insn, depth); +#endif } return 0; but below should be fine too. Will test it asap. > This is the net --> net-next merge resolution I am about to use to fix > this: > > ... > +static int fixup_call_args(struct bpf_verifier_env *env) > +{ > + struct bpf_prog *prog = env->prog; > + struct bpf_insn *insn = prog->insnsi; > - int i, depth; > ++ int i, depth, err; > + > - if (env->prog->jit_requested) > - if (jit_subprogs(env) == 0) > ++ err = 0; > ++ if (env->prog->jit_requested) { > ++ err = jit_subprogs(env); > ++ if (err == 0) > + return 0; > - > ++ } > ++#ifndef CONFIG_BPF_JIT_ALWAYS_ON > + for (i = 0; i < prog->len; i++, insn++) { > + if (insn->code != (BPF_JMP | BPF_CALL) || > + insn->src_reg != BPF_PSEUDO_CALL) > + continue; > + depth = get_callee_stack_depth(env, insn, i); > + if (depth < 0) > + return depth; > + bpf_patch_call_args(insn, depth); > + } > - return 0; > ++ err = 0; > ++#endif > ++ return err; > +} > + > /* fixup insn->imm field of bpf_call instructions > * and inline eligible helpers as explicit sequence of BPF instructions > *