From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932673AbbKMLqr (ORCPT ); Fri, 13 Nov 2015 06:46:47 -0500 Received: from mout.kundenserver.de ([217.72.192.73]:57685 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932553AbbKMLqo (ORCPT ); Fri, 13 Nov 2015 06:46:44 -0500 From: Arnd Bergmann To: linux-arm-kernel@lists.infradead.org Cc: Christoffer Dall , Marc Zyngier , Catalin Marinas , Will Deacon , linux-kernel@vger.kernel.org, kvmarm@lists.cs.columbia.edu Subject: Re: [PATCH] ARM64: use linux/types.h in kvm.h Date: Fri, 13 Nov 2015 12:46:09 +0100 Message-ID: <8866273.XDAptVUNIk@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: <20151113113843.GG12968@cbox> References: <9679324.UdV9GV0mFA@wuerfel> <20151113113843.GG12968@cbox> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:zV4RI6K597lJHxf61NgGAg7mtDXAOhogSXyaZw/U8jPGf9iBkqo rlh5l2wug4hs+O/iPDdCXT1zIHIjyYR2DSI9ej8PrHBHAPdh3jd2W9Mvg1qQiz86aiD+iUb mY3263PEMWdgmlfnew3KJ9b9fgAk3ERv3hobG3ppjlUrJYTjozmBmVPAiZr7h7DLFcr+uit 8bzfcTkB/4fxDXjFmCqRA== X-UI-Out-Filterresults: notjunk:1;V01:K0:wYA0imtmCS8=:yHMtJwZ/hGYSlqXLHrFnPG FGTnuJRo9mNT2iOVd5Y37RaRG0eww5eKtROmHcm6dUHrLGXki9OItuqXnUmpeAxnRrNvDfojZ 8Kr61qca9O57WriN7Pvd/UNfA43yKBXkXtKpTltKQS6RJQgIBdApXFu63ogmuZPPvc8ZALO0M AN8H64UHCVqGriaKHDQKoFSsP6U3TWxiSsL/ZE+4ADCxDoRypmstL9rb30pjZSqPflC/HQGxq Yr/kN4ThQ7gqcRKra+naN7UPit/TzlEdKQYifgkFoMWMcjROq+d/MJwQmiuhgmlhYL5uSV7O0 dc/Y5ztmLjOFJx2+PJr0CR4eDeSVxashbzhbzrhsM+3yN7VLckiWqdbeguLBAFibdQw3t9ABS hkxDoBhksu8RhFxILzgtRE8CmC22plsVkynYMs6OZ3LXtOAOKZfy1XxejIVBNzxEhiiAoC49I UjCzQgO4+gBthM0vz5sgYilKTVkg22Gmr4cNtYTEJwY8yQnnI0epYU1VWIhO3mP7BId2btMIe 846Tnsa5POu/djSnFg47q+oCZDb606Dy+gWypKQo20h2TdY0+u3y+4KTkEgUvutSfzrlXyvOP N+S4kkpqZuaiywLT2WBknGbl2m/FMLD5xU3bwTa1d+bFsX+57DFd6f5xxmn3Pfy0qtTJcqopU gVgwITZbiM4NOJutQOEy0lMEsdmxXFfcdHbWOC6Nw1UfnW3NMkwfLqJG0e0A+H3umvYr/eqeC yTyfO0drwe+7HqWv Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday 13 November 2015 12:38:43 Christoffer Dall wrote: > On Thu, Nov 12, 2015 at 03:41:08PM +0100, Arnd Bergmann wrote: > > We should always use linux/types.h instead of asm/types.h for > > consistency, and Kbuild actually warns about it: > > > > ./usr/include/asm/kvm.h:35: include of is preferred over > > > > This patch does as Kbuild asks us. > > > > Signed-off-by: Arnd Bergmann > > I don't see this error when building the kernel, does it require a > special option to check for these things? CONFIG_HEADERS_CHECK should be included in 'allmodconfig', which is the configuration I used. I think at the moment, allmodconfig builds fail, so you don't get to this point unless you also apply my other fixes. > In any case, thanks for the patch, I have applied it. Thanks! Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH] ARM64: use linux/types.h in kvm.h Date: Fri, 13 Nov 2015 12:46:09 +0100 Message-ID: <8866273.XDAptVUNIk@wuerfel> References: <9679324.UdV9GV0mFA@wuerfel> <20151113113843.GG12968@cbox> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from localhost (localhost [127.0.0.1]) by mm01.cs.columbia.edu (Postfix) with ESMTP id 7DA7F48529 for ; Fri, 13 Nov 2015 06:46:01 -0500 (EST) Received: from mm01.cs.columbia.edu ([127.0.0.1]) by localhost (mm01.cs.columbia.edu [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id FazJLANa+NDt for ; Fri, 13 Nov 2015 06:46:00 -0500 (EST) Received: from mout.kundenserver.de (mout.kundenserver.de [217.72.192.73]) by mm01.cs.columbia.edu (Postfix) with ESMTPS id ECC5941280 for ; Fri, 13 Nov 2015 06:45:59 -0500 (EST) In-Reply-To: <20151113113843.GG12968@cbox> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: kvmarm-bounces@lists.cs.columbia.edu Sender: kvmarm-bounces@lists.cs.columbia.edu To: linux-arm-kernel@lists.infradead.org Cc: Marc Zyngier , Catalin Marinas , Will Deacon , linux-kernel@vger.kernel.org, kvmarm@lists.cs.columbia.edu List-Id: kvmarm@lists.cs.columbia.edu On Friday 13 November 2015 12:38:43 Christoffer Dall wrote: > On Thu, Nov 12, 2015 at 03:41:08PM +0100, Arnd Bergmann wrote: > > We should always use linux/types.h instead of asm/types.h for > > consistency, and Kbuild actually warns about it: > > > > ./usr/include/asm/kvm.h:35: include of is preferred over > > > > This patch does as Kbuild asks us. > > > > Signed-off-by: Arnd Bergmann > > I don't see this error when building the kernel, does it require a > special option to check for these things? CONFIG_HEADERS_CHECK should be included in 'allmodconfig', which is the configuration I used. I think at the moment, allmodconfig builds fail, so you don't get to this point unless you also apply my other fixes. > In any case, thanks for the patch, I have applied it. Thanks! Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Fri, 13 Nov 2015 12:46:09 +0100 Subject: [PATCH] ARM64: use linux/types.h in kvm.h In-Reply-To: <20151113113843.GG12968@cbox> References: <9679324.UdV9GV0mFA@wuerfel> <20151113113843.GG12968@cbox> Message-ID: <8866273.XDAptVUNIk@wuerfel> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Friday 13 November 2015 12:38:43 Christoffer Dall wrote: > On Thu, Nov 12, 2015 at 03:41:08PM +0100, Arnd Bergmann wrote: > > We should always use linux/types.h instead of asm/types.h for > > consistency, and Kbuild actually warns about it: > > > > ./usr/include/asm/kvm.h:35: include of is preferred over > > > > This patch does as Kbuild asks us. > > > > Signed-off-by: Arnd Bergmann > > I don't see this error when building the kernel, does it require a > special option to check for these things? CONFIG_HEADERS_CHECK should be included in 'allmodconfig', which is the configuration I used. I think at the moment, allmodconfig builds fail, so you don't get to this point unless you also apply my other fixes. > In any case, thanks for the patch, I have applied it. Thanks! Arnd