[patch] Re: 2.6.14-rc4-mm1 ntfs/namei.c missing compat.h?

From: Tim Schmielau
Date: Mon Oct 17 2005 - 11:34:28 EST


On Mon, 17 Oct 2005, nyk wrote:

> Looks like fs/ntfs/namei.c is missing linux/compat.h
>
> CHK include/linux/version.h
> CHK include/linux/compile.h
> CHK usr/initramfs_list
> CC [M] fs/ntfs/namei.o
> In file included from include/linux/dcache.h:6,
> from fs/ntfs/namei.c:23:
> include/asm/atomic.h:383: error: syntax error before '*' token
> include/asm/atomic.h:384: warning: function declaration isn't a prototype
> include/asm/atomic.h: In function 'atomic_scrub':
> include/asm/atomic.h:385: error: 'u32' undeclared (first use in this function)
> include/asm/atomic.h:385: error: (Each undeclared identifier is reported only once
[...]
>
> Compiles fine with #include <linux/compat.h> added to the top of
> fs/ntfs/namei.c
>
> If that's the right place for it, of course.

[Looks like you are on x86_64, as i386 compiles fine]

Actually, <asm-x86_64/atomic.h> seems to need <asm/types.h> for the
declaration of u32.
The patch below makes NTFS compile on x86_64 for me. Andi?

Tim


--- linux-2.6.14-rc4-mm1/include/asm-x86_64/atomic.h 2005-10-17 17:48:12.000000000 +0200
+++ linux-2.6.14-rc4-mm1-build/include/asm-x86_64/atomic.h 2005-10-17 18:20:19.000000000 +0200
@@ -2,6 +2,7 @@
#define __ARCH_X86_64_ATOMIC__

#include <linux/config.h>
+#include <asm/types.h>

/* atomic_t should be 32 bit signed type */

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/