Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932434AbaFQLJd (ORCPT ); Tue, 17 Jun 2014 07:09:33 -0400 Received: from mailapp01.imgtec.com ([195.59.15.196]:34511 "EHLO mailapp01.imgtec.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932298AbaFQLJc (ORCPT ); Tue, 17 Jun 2014 07:09:32 -0400 Message-ID: <53A021E8.8020209@imgtec.com> Date: Tue, 17 Jun 2014 12:09:28 +0100 From: Markos Chandras User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 MIME-Version: 1.0 To: Daniel Borkmann , Guenter Roeck CC: "linux-kernel@vger.kernel.org" , "Linux MIPS Mailing List" , Ralf Baechle , Alexei Starovoitov Subject: Re: mips:allmodconfig build failure in 3.16-rc1 due to bpf_jit code References: <539FA6CB.5050703@roeck-us.net> <539FFA6B.9030004@redhat.com> <53A01572.5090603@redhat.com> <53A01AED.4010008@roeck-us.net> <53A01ECA.6090504@redhat.com> In-Reply-To: <53A01ECA.6090504@redhat.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit X-Originating-IP: [192.168.154.28] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/17/2014 11:56 AM, Daniel Borkmann wrote: > On 06/17/2014 12:39 PM, Guenter Roeck wrote: >> On 06/17/2014 03:16 AM, Daniel Borkmann wrote: >>> On 06/17/2014 10:20 AM, Daniel Borkmann wrote: >>>> On 06/17/2014 04:24 AM, Guenter Roeck wrote: >>>>> mips:allmodconfig fails in 3.16-rc1 with lots of undefined symbols. >>>>> >>>>> arch/mips/net/bpf_jit.c: In function 'is_load_to_a': >>>>> arch/mips/net/bpf_jit.c:559:7: error: 'BPF_S_LD_W_LEN' undeclared >>>>> (first use in this function) >>>>> arch/mips/net/bpf_jit.c:559:7: note: each undeclared identifier is >>>>> reported only once for each function it appears in >>>>> arch/mips/net/bpf_jit.c:560:7: error: 'BPF_S_LD_W_ABS' undeclared >>>>> (first use in this function) >>>>> arch/mips/net/bpf_jit.c:561:7: error: 'BPF_S_LD_H_ABS' undeclared >>>>> (first use in this function) >>>>> arch/mips/net/bpf_jit.c:562:7: error: 'BPF_S_LD_B_ABS' undeclared >>>>> (first use in this function) >>>>> arch/mips/net/bpf_jit.c:563:7: error: 'BPF_S_ANC_CPU' undeclared >>>>> (first use in this function) >>>>> arch/mips/net/bpf_jit.c:564:7: error: 'BPF_S_ANC_IFINDEX' >>>>> undeclared (first use in this function) >>>>> arch/mips/net/bpf_jit.c:565:7: error: 'BPF_S_ANC_MARK' undeclared >>>>> (first use in this function) >>>>> >>>>> and so on. >>>>> >>>>> Those symbols are not defined anywhere. >>>>> >>>>> The problem is due to a conflict with commit 348059313 (net: >>>>> filter: get rid of BPF_S_* >>>>> enum), which removed those definitions. >>>> >>>> Yep, it seems both got in this merge window from different trees. >>>> Don't have mips, but >>>> I'll have a look, and send you a patch. >>> >>> Could you give the attached patch a try? >> >> Yes, this fixes the build problem. Obviously I have no idea if it >> works ;-) > > Thanks Guenter! > > We have a test suite for BPF under lib/test_bpf.c that was designed for the > interpreter and for BPF JIT developers in mind and pretty much covers most > cases. E.g. one possibility to run it would be: > > 0) compile module test_bpf (Kernel Hacking -> Test BPF filter > functionality) > 1) echo 1 > /proc/sys/net/core/bpf_jit_enable [<- enables > BPF JIT] > 2) modprobe test_bpf [<- runs test > suite] > > The results will be visible in the klog via dmesg (if at least one test > fails, > the module init handler will return an error). > > Best, > > Daniel Hi Daniel, Thanks for these instructions. I will try them myself once I find some time since I don't think bpf_jit for MIPS has ever been tested with all the opcodes. -- markos -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/