All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] x86: parocessor.h - use PAGE_SIZE instead of numeric value v2
@ 2008-03-13 16:44 Cyrill Gorcunov
  2008-03-21 11:37 ` Ingo Molnar
  0 siblings, 1 reply; 2+ messages in thread
From: Cyrill Gorcunov @ 2008-03-13 16:44 UTC (permalink / raw)
  To: Ingo Molnar; +Cc: H. Peter Anvin, LKML

This patch replaces numeric constant with an appropriate macro

Also 0x800000000000UL is changed to bit shifting which is complement
to the code comment (thanks hpa for notice)

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>
---

Index: linux-2.6.git/include/asm-x86/processor.h
===================================================================
--- linux-2.6.git.orig/include/asm-x86/processor.h	2008-03-12 23:17:25.000000000 +0300
+++ linux-2.6.git/include/asm-x86/processor.h	2008-03-13 18:35:13.000000000 +0300
@@ -786,7 +786,7 @@ extern unsigned long thread_saved_pc(str
 /*
  * User space process size. 47bits minus one guard page.
  */
-#define TASK_SIZE64	(0x800000000000UL - 4096)
+#define TASK_SIZE64	((1UL << 47) - PAGE_SIZE)
 
 /* This decides where the kernel will search for a free chunk of vm
  * space during mmap's.

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

* Re: [PATCH] x86: parocessor.h - use PAGE_SIZE instead of numeric value v2
  2008-03-13 16:44 [PATCH] x86: parocessor.h - use PAGE_SIZE instead of numeric value v2 Cyrill Gorcunov
@ 2008-03-21 11:37 ` Ingo Molnar
  0 siblings, 0 replies; 2+ messages in thread
From: Ingo Molnar @ 2008-03-21 11:37 UTC (permalink / raw)
  To: Cyrill Gorcunov; +Cc: H. Peter Anvin, LKML


* Cyrill Gorcunov <gorcunov@gmail.com> wrote:

> This patch replaces numeric constant with an appropriate macro
> 
> Also 0x800000000000UL is changed to bit shifting which is complement 
> to the code comment (thanks hpa for notice)

> -#define TASK_SIZE64	(0x800000000000UL - 4096)
> +#define TASK_SIZE64	((1UL << 47) - PAGE_SIZE)

thanks Cyrill, applied.

	Ingo

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

end of thread, other threads:[~2008-03-21 11:37 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-03-13 16:44 [PATCH] x86: parocessor.h - use PAGE_SIZE instead of numeric value v2 Cyrill Gorcunov
2008-03-21 11:37 ` Ingo Molnar

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.