From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from bh-25.webhostbox.net ([208.91.199.152]:52865 "EHLO bh-25.webhostbox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750777AbbEQQKb (ORCPT ); Sun, 17 May 2015 12:10:31 -0400 Message-ID: <5558BD72.9040403@roeck-us.net> Date: Sun, 17 May 2015 09:10:26 -0700 From: Guenter Roeck MIME-Version: 1.0 To: Sasha Levin CC: stable@vger.kernel.org, stable-commits@vger.kernel.org, Andy Lutomirski , Ingo Molnar , Quentin Casasnovas , Phil Turnbull , Chandrakala Chavva , David Daney Subject: Re: [PATCH 3.18 000/222] 3.18.14-review (build errors) References: <1431824764-20044-1-git-send-email-sasha.levin@oracle.com> <20150517042149.GA1778@roeck-us.net> <5558A8CF.1000107@oracle.com> In-Reply-To: <5558A8CF.1000107@oracle.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Sender: stable-owner@vger.kernel.org List-ID: On 05/17/2015 07:42 AM, Sasha Levin wrote: > Thanks for testing! > > On 05/17/2015 12:21 AM, Guenter Roeck wrote: >> x86_64 (qemu), i386: >> >> arch/x86/lib/insn.c: In function 'insn_init': >> arch/x86/lib/insn.c:59:6: error: 'buf_len' undeclared >> >> Caused by commit d597f507791d ("x86/asm/decoder: Fix and enforce max instruction >> size in the insn decoder"). Something must have gone wrong with the back-port, >> or some other commit is missing; the variable is not declared. > > I've removed d597f507791d, it doesn't fix a real bug. > >> --- >> xtensa, sparc64, s390, powerpc, mips, m68k, i386, arm64, alpha: >> >> drivers/usb/class/cdc-acm.c: In function ‘acm_probe’: >> drivers/usb/class/cdc-acm.c:1135:3: error: ‘elength’ undeclared >> >> Caused by commit bf2996daa6d2 ("cdc-acm: prevent infinite loop when parsing CDC >> headers"). Same thing - the variable is not declared. > > I've backported 7e860a6e7a ("cdc-acm: add sanity checks"). > > Re-pushed the branch. > This is a bit better. However, Building mips:cavium_octeon_defconfig ... failed -------------- Error log: arch/mips/cavium-octeon/setup.c: In function 'octeon_restart': arch/mips/cavium-octeon/setup.c:415:3: error: implicit declaration of function 'OCTEON_IS_OCTEON3' [-Werror=implicit-function-declaration] arch/mips/cavium-octeon/setup.c:416:19: error: 'CVMX_RST_SOFT_RST' undeclared (first use in this function) due to commit 03dc9c96d0e0 ("MIPS: OCTEON: Use correct CSR to soft reset"). This patch fixes Octeon III support, which I think is not in 3.18, so maybe you can just drop it. Also, I still see: arch/x86/lib/insn.c: In function 'insn_init': arch/x86/lib/insn.c:59:6: error: 'buf_len' undeclared for all x86_64 and i386 builds (d597f507791d is still in the image). Guenter