Blaisorblade wrote:
On Friday 10 December 2004 17:45, [EMAIL PROTECTED] wrote:

From: Bodo Stroesser <[EMAIL PROTECTED]>

make arch/um/kernel/skas/include/skas.h include asm/types.h
This is necessary to have a typedef for __u64


*Probably* won't work, as headers in arch/um/include.... are also includable by userspace files, which can't include <asm/types.h>;
My patch changes arch/um/kernel/skas/include/skas.h, which is included by
arch/um/kernel/process.c, arch/um/sys-XXXX/signal.c and some
arch/um/kernel/skas/XXXX.c. IMHO, this should be safe, or do I miss somesthing?

well, it could happen
to include the *host* asm/types.h, which will be bad on a x86_64 host, since you could get a "unsigned long" (which is safe) even when compiling a 32-bit UML!

Otherwise, there would not be includes in arch/um! They'd be in include/asm-um!

And yes, this sucks, but is reality.

Workaround it - unsigned long long is always a correct definition.
This would be another possibility, but then the definition of "void map()" in
arch/um/kernel/skas/mem_user.c also should be changed. I choosed the option,
which changes less. Feel free to choose the other one.

Bodo

Signed-off-by: Bodo Stroesser <[EMAIL PROTECTED]>
---


diff -puN arch/um/kernel/skas/include/skas.h~skas.h-include-types.h arch/um/kernel/skas/include/skas.h --- linux-2.6.10-rc2-mm4/arch/um/kernel/skas/include/skas.h~skas.h-include-type s.h 2004-12-10 15:53:03.726530054 +0100 +++ linux-2.6.10-rc2-mm4-root/arch/um/kernel/skas/include/skas.h 2004-12-10 15:53:03.730528661 +0100 @@ -7,6 +7,7 @@ #define __SKAS_H

#include "sysdep/ptrace.h"
+#include "asm/types.h"

extern int userspace_pid[];

_




-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now. http://productguide.itmanagersjournal.com/
_______________________________________________
User-mode-linux-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

Reply via email to