Re: PATCH powerpc Merge asm-ppc*/seccomp.h

So, like, the other day Christoph Hellwig mumbled:

quoted

+#include <linux/thread_info.h>
+
+#if defined(__powerpc64__) && !defined(TIF_32BIT)
+#error "unexpected TIF_32BIT on ppc64"
+#endif

just kill this check, it's rather pointless

OK.

But keep the #include?  It's actually the part
that is defining TIF_32BIT for ppc64 (indirectly
through linux/thread_info.h and asm/thread_info.h).
Won't those bits be needed still?

jdl

On Thu, Sep 22, 2005 at 01:36:52PM -0500, Jon Loeliger wrote:

So, like, the other day Christoph Hellwig mumbled:

quoted

quoted

+#include <linux/thread_info.h>
+
+#if defined(__powerpc64__) && !defined(TIF_32BIT)
+#error "unexpected TIF_32BIT on ppc64"
+#endif

just kill this check, it's rather pointless

OK.

But keep the #include?  It's actually the part
that is defining TIF_32BIT for ppc64 (indirectly
through linux/thread_info.h and asm/thread_info.h).
Won't those bits be needed still?

I'm not sure.  This header definitly doesn't need it, but I suspect
seccompt.c expects to get it via this header.