Date: Tue, 9 Jan 2018 10:13:01 +0100
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Guenter Roeck <linux@roeck-us.net>
Cc: linux-kernel@vger.kernel.org, torvalds@linux-foundation.org,
        akpm@linux-foundation.org, shuahkh@osg.samsung.com,
        patches@kernelci.org, ben.hutchings@codethink.co.uk,
        lkft-triage@lists.linaro.org, stable@vger.kernel.org
Subject: Re: [PATCH 4.4 00/22] 4.4.111-stable review
Message-ID: <20180109091301.GB18146@kroah.com>
References: <20180108125925.601688333@linuxfoundation.org>
 <20180108232907.GA22001@roeck-us.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20180108232907.GA22001@roeck-us.net>
User-Agent: Mutt/1.9.2 (2017-12-15)
Sender: linux-kernel-owner@vger.kernel.org

On Mon, Jan 08, 2018 at 03:29:07PM -0800, Guenter Roeck wrote:
> On Mon, Jan 08, 2018 at 01:59:27PM +0100, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 4.4.111 release.
> > There are 22 patches in this series, all will be posted as a response
> > to this one.  If anyone has any issues with these being applied, please
> > let me know.
> > 
> > Responses should be made by Wed Jan 10 12:59:14 UTC 2018.
> > Anything received after that time might be too late.
> > 
> 
> x86:64:allnoconfig still fails to build.
> 
> To reproduce:
>   make ARCH=x86_64 allnoconfig
>   make ARCH=x86_64 arch/x86/entry/vdso/vma.o
> 
> arch/x86/entry/vdso/vma.c: In function ‘map_vdso’:
> arch/x86/entry/vdso/vma.c:175:9: error:
> 	implicit declaration of function ‘pvclock_pvti_cpu0_va’
> 
> The function is declared in arch/x86/include/asm/pvclock.h, which is only 
> indirectly included. For 32-bit allnoconfig builds, the include path is:
>   arch/x86/entry/vdso/vma.c
>     include/linux/mm.h
>       ./arch/x86/include/asm/pgtable.h
>         ./arch/x86/include/asm/pgtable_32.h
> 
> but pgtable_64.h doesn't include pgtable_32.h, and the declaration is
> missing as result.

Yeah, I haven't had the chance to work on that yet, sorry.

> The include of asm/pvclock.h was added upstream with commit cc1e24fdb064d
> ("x86/vdso: Remove pvclock fixmap machinery") which applies cleanly to
> v4.4.y. Please consider adding it, or at least add the missing include.

Is it really that easy?  Hey, nice, let me queue that up and test...

thanks,

greg k-h