From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754897AbcASOJi (ORCPT ); Tue, 19 Jan 2016 09:09:38 -0500 Received: from mout.kundenserver.de ([212.227.126.187]:57176 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754117AbcASOJa convert rfc822-to-8bit (ORCPT ); Tue, 19 Jan 2016 09:09:30 -0500 From: Arnd Bergmann To: James Hogan Cc: Michal Marek , linux-kernel@vger.kernel.org, Heinrich Schuchardt , Ralf Baechle , Paul Burton , Florian Fainelli , linux-kbuild@vger.kernel.org, linux-arch@vger.kernel.org, linux-mips@linux-mips.org Subject: Re: [PATCH v2 2/2] kbuild: Remove stale asm-generic wrappers Date: Tue, 19 Jan 2016 15:09:14 +0100 Message-ID: <1667268.iM977TQnEK@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: <1453210670-12596-3-git-send-email-james.hogan@imgtec.com> References: <1453210670-12596-1-git-send-email-james.hogan@imgtec.com> <1453210670-12596-3-git-send-email-james.hogan@imgtec.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="utf-8" X-Provags-ID: V03:K0:jE20OMq+t6LKtHoWcpAxdX+hbJPZWDumu93RclJZi2IcBmu1BB5 ApWBlF+fqIWzp8PJTomirPu17XKl67+CVhsikWdHR9bHNzmqGvBIsY7NJt/AVPJzeQKxWsU 2WMElwcpVHV1fNhy3cImPS7yam3WYaTzGjQHhSDzSFqWB6Qj3nhjjao+mZNGJ8iOi7HoyBm mCyws3ajUtyJ94eV2rLrg== X-UI-Out-Filterresults: notjunk:1;V01:K0:megdsB3B028=:tmlHz4C1XL/U93M9ICXy7Z pS6KjRPBd18LUi4LWaTTZ7i7EDkFzPvAxBYsSISxQ28KCRU82tzSwOUPbRnxtkSXp1WuHDzYT gj4SBdUdrqi89Ncp1+OvkuGayouOMihEI1dV4PPNcxzw8fxOuktkvLaFbZPObHvc22d1tLGGP jY6cvnK0IKvpDo8vZlBIPPkLNLmvM+XbGH5R4Fsse+imla/Ep/keHwnTo+DW7UopMXJz42Adl gsQhN3/s5GEps3VBDuGRBXOaraNQNRHV9EEIOh5jeth8ZKbq6H0jUWhvt8WwlF8TnsdGvBXnb k4m5iBLr/cRrxjz4JmYcDenSCZDGQIDU9z731kW6jT4me0NfQg11aFX4oYUeGaGRZEJdVIEZ1 tnPz106P2dKFZ9gXPiYCPx+X/ubetHaganCB2/J2YWoPBhXpX+AJkuS2Z0OBUedO540Li7WDa ymdRegr68DIx9fDcLA87I6WOyyrRIxBw2MFDzQM/iu2QEmn6/Fbj4iBoHkW96xCLiaTuevzd8 RHG55EB8Jgk8wLawH+SU8VXiuuvA6RzFHnquiR9A9QTwZz1qi8aOqmKk5YQTcukO8uFdXQXwA W3jPZ1MDY8w7UCTxaIu+gQbssKGC43jnYYbU9cB5WDCJy7UvKRzF6idghLFWL8mKPnOpjpzyS qc00A43Njcnv1Y6kIcPnUiiUS+9XDxjs1cZ2SV15pykeTDdbhXDsXPT8D48x6AfaD89QS9EDe IKhzR7/1njT7QRdk Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday 19 January 2016 13:37:50 James Hogan wrote: > When a header file is removed from generic-y (often accompanied by the > addition of an arch specific header), the generated wrapper file will > persist, and in some cases may still take precedence over the new arch > header. > > For example commit f1fe2d21f4e1 ("MIPS: Add definitions for extended > context") removed ucontext.h from generic-y in arch/mips/include/asm/, > and added an arch/mips/include/uapi/asm/ucontext.h. The continued use of > the wrapper when reusing a dirty build tree resulted in build failures > in arch/mips/kernel/signal.c: > > arch/mips/kernel/signal.c: In function ‘sc_to_extcontext’: > arch/mips/kernel/signal.c:142:12: error: ‘struct ucontext’ has no member named ‘uc_extcontext’ > return &uc->uc_extcontext; > ^ > > Fix by detecting and removing wrapper headers in generated header > directories that do not correspond to a filename in generic-y, genhdr-y, > or the newly introduced generated-y. Good idea. Acked-by: Arnd Bergmann Can you merge this through the mips tree, or do you need me to pick it up through asm-generic?