* [uml-devel] [PATCH] UML - Add asm/fixmap.h include
@ 2007-12-14 16:16 ` Jeff Dike
0 siblings, 0 replies; 6+ messages in thread
From: Jeff Dike @ 2007-12-14 16:16 UTC (permalink / raw)
To: Andrew Morton; +Cc: Peter Zijlstra, LKML, uml-devel
[ This needs to go into 2.6.24, as it fixes a build breakage seen on x86_64 ]
um_uaccess.h refers to FIXADDR_USER_*, so it should include fixmap.h.
I also changed the non-UML includes to use <> instead of "".
Signed-off-by: Jeff Dike <jdike@linux.intel.com>
---
arch/um/include/um_uaccess.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
Index: linux-2.6.22/arch/um/include/um_uaccess.h
===================================================================
--- linux-2.6.22.orig/arch/um/include/um_uaccess.h 2007-12-13 10:09:22.000000000 -0500
+++ linux-2.6.22/arch/um/include/um_uaccess.h 2007-12-13 15:11:42.000000000 -0500
@@ -6,7 +6,8 @@
#ifndef __ARCH_UM_UACCESS_H
#define __ARCH_UM_UACCESS_H
-#include "asm/elf.h"
+#include <asm/elf.h>
+#include <asm/fixmap.h>
#include "sysdep/archsetjmp.h"
#define __under_task_size(addr, size) \
-------------------------------------------------------------------------
SF.Net email is sponsored by:
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services
for just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] UML - Add asm/fixmap.h include
@ 2007-12-14 16:16 ` Jeff Dike
0 siblings, 0 replies; 6+ messages in thread
From: Jeff Dike @ 2007-12-14 16:16 UTC (permalink / raw)
To: Andrew Morton; +Cc: LKML, uml-devel, Peter Zijlstra
[ This needs to go into 2.6.24, as it fixes a build breakage seen on x86_64 ]
um_uaccess.h refers to FIXADDR_USER_*, so it should include fixmap.h.
I also changed the non-UML includes to use <> instead of "".
Signed-off-by: Jeff Dike <jdike@linux.intel.com>
---
arch/um/include/um_uaccess.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
Index: linux-2.6.22/arch/um/include/um_uaccess.h
===================================================================
--- linux-2.6.22.orig/arch/um/include/um_uaccess.h 2007-12-13 10:09:22.000000000 -0500
+++ linux-2.6.22/arch/um/include/um_uaccess.h 2007-12-13 15:11:42.000000000 -0500
@@ -6,7 +6,8 @@
#ifndef __ARCH_UM_UACCESS_H
#define __ARCH_UM_UACCESS_H
-#include "asm/elf.h"
+#include <asm/elf.h>
+#include <asm/fixmap.h>
#include "sysdep/archsetjmp.h"
#define __under_task_size(addr, size) \
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [uml-devel] [PATCH] UML - Add asm/fixmap.h include
2007-12-14 16:16 ` Jeff Dike
@ 2007-12-14 17:58 ` Andrew Morton
-1 siblings, 0 replies; 6+ messages in thread
From: Andrew Morton @ 2007-12-14 17:58 UTC (permalink / raw)
To: Jeff Dike; +Cc: Peter Zijlstra, LKML, uml-devel
On Fri, 14 Dec 2007 11:16:47 -0500 Jeff Dike <jdike@addtoit.com> wrote:
> [ This needs to go into 2.6.24, as it fixes a build breakage seen on x86_64 ]
>
> um_uaccess.h refers to FIXADDR_USER_*, so it should include fixmap.h.
>
> I also changed the non-UML includes to use <> instead of "".
>
> Signed-off-by: Jeff Dike <jdike@linux.intel.com>
> ---
> arch/um/include/um_uaccess.h | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> Index: linux-2.6.22/arch/um/include/um_uaccess.h
> ===================================================================
> --- linux-2.6.22.orig/arch/um/include/um_uaccess.h 2007-12-13 10:09:22.000000000 -0500
> +++ linux-2.6.22/arch/um/include/um_uaccess.h 2007-12-13 15:11:42.000000000 -0500
> @@ -6,7 +6,8 @@
> #ifndef __ARCH_UM_UACCESS_H
> #define __ARCH_UM_UACCESS_H
>
> -#include "asm/elf.h"
> +#include <asm/elf.h>
> +#include <asm/fixmap.h>
> #include "sysdep/archsetjmp.h"
>
> #define __under_task_size(addr, size) \
arch/um/include/um_uaccess.h already includes fixmap.h. Is this a -mm-only
fix?
-------------------------------------------------------------------------
SF.Net email is sponsored by:
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services
for just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] UML - Add asm/fixmap.h include
@ 2007-12-14 17:58 ` Andrew Morton
0 siblings, 0 replies; 6+ messages in thread
From: Andrew Morton @ 2007-12-14 17:58 UTC (permalink / raw)
To: Jeff Dike; +Cc: LKML, uml-devel, Peter Zijlstra
On Fri, 14 Dec 2007 11:16:47 -0500 Jeff Dike <jdike@addtoit.com> wrote:
> [ This needs to go into 2.6.24, as it fixes a build breakage seen on x86_64 ]
>
> um_uaccess.h refers to FIXADDR_USER_*, so it should include fixmap.h.
>
> I also changed the non-UML includes to use <> instead of "".
>
> Signed-off-by: Jeff Dike <jdike@linux.intel.com>
> ---
> arch/um/include/um_uaccess.h | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> Index: linux-2.6.22/arch/um/include/um_uaccess.h
> ===================================================================
> --- linux-2.6.22.orig/arch/um/include/um_uaccess.h 2007-12-13 10:09:22.000000000 -0500
> +++ linux-2.6.22/arch/um/include/um_uaccess.h 2007-12-13 15:11:42.000000000 -0500
> @@ -6,7 +6,8 @@
> #ifndef __ARCH_UM_UACCESS_H
> #define __ARCH_UM_UACCESS_H
>
> -#include "asm/elf.h"
> +#include <asm/elf.h>
> +#include <asm/fixmap.h>
> #include "sysdep/archsetjmp.h"
>
> #define __under_task_size(addr, size) \
arch/um/include/um_uaccess.h already includes fixmap.h. Is this a -mm-only
fix?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [uml-devel] [PATCH] UML - Add asm/fixmap.h include
2007-12-14 17:58 ` Andrew Morton
@ 2007-12-14 18:55 ` Jeff Dike
-1 siblings, 0 replies; 6+ messages in thread
From: Jeff Dike @ 2007-12-14 18:55 UTC (permalink / raw)
To: Andrew Morton; +Cc: Peter Zijlstra, LKML, uml-devel
On Fri, Dec 14, 2007 at 09:58:40AM -0800, Andrew Morton wrote:
> arch/um/include/um_uaccess.h already includes fixmap.h. Is this a -mm-only
> fix?
Whoops, it is. The include was removed by uml-header-untangling.patch,
so folding this into that would work.
Jeff
--
Work email - jdike at linux dot intel dot com
-------------------------------------------------------------------------
SF.Net email is sponsored by:
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services
for just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] UML - Add asm/fixmap.h include
@ 2007-12-14 18:55 ` Jeff Dike
0 siblings, 0 replies; 6+ messages in thread
From: Jeff Dike @ 2007-12-14 18:55 UTC (permalink / raw)
To: Andrew Morton; +Cc: LKML, uml-devel, Peter Zijlstra
On Fri, Dec 14, 2007 at 09:58:40AM -0800, Andrew Morton wrote:
> arch/um/include/um_uaccess.h already includes fixmap.h. Is this a -mm-only
> fix?
Whoops, it is. The include was removed by uml-header-untangling.patch,
so folding this into that would work.
Jeff
--
Work email - jdike at linux dot intel dot com
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2007-12-14 18:56 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-12-14 16:16 [uml-devel] [PATCH] UML - Add asm/fixmap.h include Jeff Dike
2007-12-14 16:16 ` Jeff Dike
2007-12-14 17:58 ` [uml-devel] " Andrew Morton
2007-12-14 17:58 ` Andrew Morton
2007-12-14 18:55 ` [uml-devel] " Jeff Dike
2007-12-14 18:55 ` Jeff Dike
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.