linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH - linux-next] ARM: ptrace: Fix audit caused compile error
@ 2012-02-21 12:04 Peter Ujfalusi
  2012-02-21 13:16 ` Will Deacon
  0 siblings, 1 reply; 5+ messages in thread
From: Peter Ujfalusi @ 2012-02-21 12:04 UTC (permalink / raw)
  To: linux-arm-kernel

While trying to compile the kernel for ARM (omap2plus_defconfig) the kernel
build fails with:

arch/arm/kernel/ptrace.c: In function ?syscall_trace?:
arch/arm/kernel/ptrace.c:920:3: error: implicit declaration of function ?audit_syscall_exit?
arch/arm/kernel/ptrace.c:922:3: error: implicit declaration of function ?audit_syscall_entry?
arch/arm/kernel/ptrace.c:922:23: error: ?AUDIT_ARCH_ARMEB? undeclared (first use in this function)
arch/arm/kernel/ptrace.c:922:23: note: each undeclared identifier is reported only once for each function it appears in
make[1]: *** [arch/arm/kernel/ptrace.o] Error 1

The issue created by commit:
29ef73b7 Kernel: Audit Support For The ARM Platform

We need to include the linux/audit.h header to the arch/arm/kernel/ptrace.c
file to be able to compile the kernel.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
---
 arch/arm/kernel/ptrace.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/arch/arm/kernel/ptrace.c b/arch/arm/kernel/ptrace.c
index e33870f..8e41ad2 100644
--- a/arch/arm/kernel/ptrace.c
+++ b/arch/arm/kernel/ptrace.c
@@ -15,6 +15,7 @@
 #include <linux/elf.h>
 #include <linux/smp.h>
 #include <linux/ptrace.h>
+#include <linux/audit.h>
 #include <linux/user.h>
 #include <linux/security.h>
 #include <linux/init.h>
-- 
1.7.8.4

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH - linux-next] ARM: ptrace: Fix audit caused compile error
  2012-02-21 12:04 [PATCH - linux-next] ARM: ptrace: Fix audit caused compile error Peter Ujfalusi
@ 2012-02-21 13:16 ` Will Deacon
  2012-02-21 13:30   ` Russell King - ARM Linux
  0 siblings, 1 reply; 5+ messages in thread
From: Will Deacon @ 2012-02-21 13:16 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Feb 21, 2012 at 12:04:15PM +0000, Peter Ujfalusi wrote:
> While trying to compile the kernel for ARM (omap2plus_defconfig) the kernel
> build fails with:
> 
> arch/arm/kernel/ptrace.c: In function ?syscall_trace?:
> arch/arm/kernel/ptrace.c:920:3: error: implicit declaration of function ?audit_syscall_exit?
> arch/arm/kernel/ptrace.c:922:3: error: implicit declaration of function ?audit_syscall_entry?
> arch/arm/kernel/ptrace.c:922:23: error: ?AUDIT_ARCH_ARMEB? undeclared (first use in this function)
> arch/arm/kernel/ptrace.c:922:23: note: each undeclared identifier is reported only once for each function it appears in
> make[1]: *** [arch/arm/kernel/ptrace.o] Error 1
> 
> The issue created by commit:
> 29ef73b7 Kernel: Audit Support For The ARM Platform
> 
> We need to include the linux/audit.h header to the arch/arm/kernel/ptrace.c
> file to be able to compile the kernel.

This was already reported and I think Russell was going to revert the
offending commit, since it needed some rework to handle little-endian
configurations.

I can't see the revert in any of the trees I'm tracking though...

Will

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH - linux-next] ARM: ptrace: Fix audit caused compile error
  2012-02-21 13:16 ` Will Deacon
@ 2012-02-21 13:30   ` Russell King - ARM Linux
  2012-02-21 13:35     ` Eric Paris
  0 siblings, 1 reply; 5+ messages in thread
From: Russell King - ARM Linux @ 2012-02-21 13:30 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Feb 21, 2012 at 01:16:17PM +0000, Will Deacon wrote:
> On Tue, Feb 21, 2012 at 12:04:15PM +0000, Peter Ujfalusi wrote:
> > While trying to compile the kernel for ARM (omap2plus_defconfig) the kernel
> > build fails with:
> > 
> > arch/arm/kernel/ptrace.c: In function ?syscall_trace?:
> > arch/arm/kernel/ptrace.c:920:3: error: implicit declaration of function ?audit_syscall_exit?
> > arch/arm/kernel/ptrace.c:922:3: error: implicit declaration of function ?audit_syscall_entry?
> > arch/arm/kernel/ptrace.c:922:23: error: ?AUDIT_ARCH_ARMEB? undeclared (first use in this function)
> > arch/arm/kernel/ptrace.c:922:23: note: each undeclared identifier is reported only once for each function it appears in
> > make[1]: *** [arch/arm/kernel/ptrace.o] Error 1
> > 
> > The issue created by commit:
> > 29ef73b7 Kernel: Audit Support For The ARM Platform
> > 
> > We need to include the linux/audit.h header to the arch/arm/kernel/ptrace.c
> > file to be able to compile the kernel.
> 
> This was already reported and I think Russell was going to revert the
> offending commit, since it needed some rework to handle little-endian
> configurations.
> 
> I can't see the revert in any of the trees I'm tracking though...

I never pushed that out because I thought someone was going to fix it.
Were any patches produced to fix it?  I can't see anything in the patch
system and I couldn't see anything on the list.

It seems the previous thread about it just died.  Oh well, I guess a
revert is what's required after all.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH - linux-next] ARM: ptrace: Fix audit caused compile error
  2012-02-21 13:30   ` Russell King - ARM Linux
@ 2012-02-21 13:35     ` Eric Paris
  2012-02-21 13:49       ` Eric Paris
  0 siblings, 1 reply; 5+ messages in thread
From: Eric Paris @ 2012-02-21 13:35 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, 2012-02-21 at 13:30 +0000, Russell King - ARM Linux wrote:
> On Tue, Feb 21, 2012 at 01:16:17PM +0000, Will Deacon wrote:
> > On Tue, Feb 21, 2012 at 12:04:15PM +0000, Peter Ujfalusi wrote:
> > > While trying to compile the kernel for ARM (omap2plus_defconfig) the kernel
> > > build fails with:
> > > 
> > > arch/arm/kernel/ptrace.c: In function ?syscall_trace?:
> > > arch/arm/kernel/ptrace.c:920:3: error: implicit declaration of function ?audit_syscall_exit?
> > > arch/arm/kernel/ptrace.c:922:3: error: implicit declaration of function ?audit_syscall_entry?
> > > arch/arm/kernel/ptrace.c:922:23: error: ?AUDIT_ARCH_ARMEB? undeclared (first use in this function)
> > > arch/arm/kernel/ptrace.c:922:23: note: each undeclared identifier is reported only once for each function it appears in
> > > make[1]: *** [arch/arm/kernel/ptrace.o] Error 1
> > > 
> > > The issue created by commit:
> > > 29ef73b7 Kernel: Audit Support For The ARM Platform
> > > 
> > > We need to include the linux/audit.h header to the arch/arm/kernel/ptrace.c
> > > file to be able to compile the kernel.
> > 
> > This was already reported and I think Russell was going to revert the
> > offending commit, since it needed some rework to handle little-endian
> > configurations.
> > 
> > I can't see the revert in any of the trees I'm tracking though...
> 
> I never pushed that out because I thought someone was going to fix it.
> Were any patches produced to fix it?  I can't see anything in the patch
> system and I couldn't see anything on the list.
> 
> It seems the previous thread about it just died.  Oh well, I guess a
> revert is what's required after all.

http://marc.info/?l=linux-arm-kernel&m=132928424120369&w=2

???

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH - linux-next] ARM: ptrace: Fix audit caused compile error
  2012-02-21 13:35     ` Eric Paris
@ 2012-02-21 13:49       ` Eric Paris
  0 siblings, 0 replies; 5+ messages in thread
From: Eric Paris @ 2012-02-21 13:49 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, 2012-02-21 at 08:35 -0500, Eric Paris wrote:
> On Tue, 2012-02-21 at 13:30 +0000, Russell King - ARM Linux wrote:
> > On Tue, Feb 21, 2012 at 01:16:17PM +0000, Will Deacon wrote:
> > > On Tue, Feb 21, 2012 at 12:04:15PM +0000, Peter Ujfalusi wrote:
> > > > While trying to compile the kernel for ARM (omap2plus_defconfig) the kernel
> > > > build fails with:
> > > > 
> > > > arch/arm/kernel/ptrace.c: In function ?syscall_trace?:
> > > > arch/arm/kernel/ptrace.c:920:3: error: implicit declaration of function ?audit_syscall_exit?
> > > > arch/arm/kernel/ptrace.c:922:3: error: implicit declaration of function ?audit_syscall_entry?
> > > > arch/arm/kernel/ptrace.c:922:23: error: ?AUDIT_ARCH_ARMEB? undeclared (first use in this function)
> > > > arch/arm/kernel/ptrace.c:922:23: note: each undeclared identifier is reported only once for each function it appears in
> > > > make[1]: *** [arch/arm/kernel/ptrace.o] Error 1
> > > > 
> > > > The issue created by commit:
> > > > 29ef73b7 Kernel: Audit Support For The ARM Platform
> > > > 
> > > > We need to include the linux/audit.h header to the arch/arm/kernel/ptrace.c
> > > > file to be able to compile the kernel.
> > > 
> > > This was already reported and I think Russell was going to revert the
> > > offending commit, since it needed some rework to handle little-endian
> > > configurations.
> > > 
> > > I can't see the revert in any of the trees I'm tracking though...
> > 
> > I never pushed that out because I thought someone was going to fix it.
> > Were any patches produced to fix it?  I can't see anything in the patch
> > system and I couldn't see anything on the list.
> > 
> > It seems the previous thread about it just died.  Oh well, I guess a
> > revert is what's required after all.
> 
> http://marc.info/?l=linux-arm-kernel&m=132928424120369&w=2
> 
> ???

You probably meant the arch endian bug.  I wrote a patch for it, but
guess i never sent it.  It was just sitting on my audit arm branch.  So
I just sent that one as well.

Thanks Russell!
-Eric

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2012-02-21 13:49 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-02-21 12:04 [PATCH - linux-next] ARM: ptrace: Fix audit caused compile error Peter Ujfalusi
2012-02-21 13:16 ` Will Deacon
2012-02-21 13:30   ` Russell King - ARM Linux
2012-02-21 13:35     ` Eric Paris
2012-02-21 13:49       ` Eric Paris

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).