From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161314AbaDPNcr (ORCPT ); Wed, 16 Apr 2014 09:32:47 -0400 Received: from cam-admin0.cambridge.arm.com ([217.140.96.50]:36184 "EHLO cam-admin0.cambridge.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1161040AbaDPNcp (ORCPT ); Wed, 16 Apr 2014 09:32:45 -0400 Date: Wed, 16 Apr 2014 14:29:31 +0100 From: Will Deacon To: AKASHI Takahiro Cc: "viro@zeniv.linux.org.uk" , "eparis@redhat.com" , "rgb@redhat.com" , Catalin Marinas , "dsaxena@linaro.org" , "arndb@arndb.de" , "linux-arm-kernel@lists.infradead.org" , "linaro-kernel@lists.linaro.org" , "linux-kernel@vger.kernel.org" , "linux-audit@redhat.com" Subject: Re: [PATCH v5 4/4] arm64: is_compat_task is defined both in asm/compat.h and linux/compat.h Message-ID: <20140416132931.GB1001@arm.com> References: <1394705491-12343-1-git-send-email-takahiro.akashi@linaro.org> <1394861948-28712-1-git-send-email-takahiro.akashi@linaro.org> <1394861948-28712-5-git-send-email-takahiro.akashi@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1394861948-28712-5-git-send-email-takahiro.akashi@linaro.org> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Mar 15, 2014 at 05:39:08AM +0000, AKASHI Takahiro wrote: > Some kernel files may include both linux/compat.h and asm/compat.h directly > or indirectly. Since both header files contain is_compat_task() under > !CONFIG_COMPAT, compiling them with !CONFIG_COMPAT will eventually fail. > Such files include kernel/auditsc.c, kernel/seccomp.c and init/do_mountfs.c > (do_mountfs.c may read asm/compat.h via asm/ftrace.h once ftrace is > implemented). > > So this patch proactively > 1) removes is_compat_task() under !CONFIG_COMPAT from asm/compat.h > 2) replaces asm/compat.h to linux/compat.h in kernel/*.c, > but asm/compat.h is still necessary in ptrace.c and process.c because > they use is_compat_thread(). > > Signed-off-by: AKASHI Takahiro Acked-by: Will Deacon Will From mboxrd@z Thu Jan 1 00:00:00 1970 From: Will Deacon Subject: Re: [PATCH v5 4/4] arm64: is_compat_task is defined both in asm/compat.h and linux/compat.h Date: Wed, 16 Apr 2014 14:29:31 +0100 Message-ID: <20140416132931.GB1001@arm.com> References: <1394705491-12343-1-git-send-email-takahiro.akashi@linaro.org> <1394861948-28712-1-git-send-email-takahiro.akashi@linaro.org> <1394861948-28712-5-git-send-email-takahiro.akashi@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1394861948-28712-5-git-send-email-takahiro.akashi@linaro.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: AKASHI Takahiro Cc: "linaro-kernel@lists.linaro.org" , "rgb@redhat.com" , Catalin Marinas , "arndb@arndb.de" , "eparis@redhat.com" , "linux-kernel@vger.kernel.org" , "dsaxena@linaro.org" , "viro@zeniv.linux.org.uk" , "linux-audit@redhat.com" , "linux-arm-kernel@lists.infradead.org" List-Id: linux-audit@redhat.com On Sat, Mar 15, 2014 at 05:39:08AM +0000, AKASHI Takahiro wrote: > Some kernel files may include both linux/compat.h and asm/compat.h directly > or indirectly. Since both header files contain is_compat_task() under > !CONFIG_COMPAT, compiling them with !CONFIG_COMPAT will eventually fail. > Such files include kernel/auditsc.c, kernel/seccomp.c and init/do_mountfs.c > (do_mountfs.c may read asm/compat.h via asm/ftrace.h once ftrace is > implemented). > > So this patch proactively > 1) removes is_compat_task() under !CONFIG_COMPAT from asm/compat.h > 2) replaces asm/compat.h to linux/compat.h in kernel/*.c, > but asm/compat.h is still necessary in ptrace.c and process.c because > they use is_compat_thread(). > > Signed-off-by: AKASHI Takahiro Acked-by: Will Deacon Will From mboxrd@z Thu Jan 1 00:00:00 1970 From: will.deacon@arm.com (Will Deacon) Date: Wed, 16 Apr 2014 14:29:31 +0100 Subject: [PATCH v5 4/4] arm64: is_compat_task is defined both in asm/compat.h and linux/compat.h In-Reply-To: <1394861948-28712-5-git-send-email-takahiro.akashi@linaro.org> References: <1394705491-12343-1-git-send-email-takahiro.akashi@linaro.org> <1394861948-28712-1-git-send-email-takahiro.akashi@linaro.org> <1394861948-28712-5-git-send-email-takahiro.akashi@linaro.org> Message-ID: <20140416132931.GB1001@arm.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Sat, Mar 15, 2014 at 05:39:08AM +0000, AKASHI Takahiro wrote: > Some kernel files may include both linux/compat.h and asm/compat.h directly > or indirectly. Since both header files contain is_compat_task() under > !CONFIG_COMPAT, compiling them with !CONFIG_COMPAT will eventually fail. > Such files include kernel/auditsc.c, kernel/seccomp.c and init/do_mountfs.c > (do_mountfs.c may read asm/compat.h via asm/ftrace.h once ftrace is > implemented). > > So this patch proactively > 1) removes is_compat_task() under !CONFIG_COMPAT from asm/compat.h > 2) replaces asm/compat.h to linux/compat.h in kernel/*.c, > but asm/compat.h is still necessary in ptrace.c and process.c because > they use is_compat_thread(). > > Signed-off-by: AKASHI Takahiro Acked-by: Will Deacon Will