public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Use LOAD_PHYSICAL_ADDR in vmlinux.lds.S
@ 2014-01-26 20:13 Zhihui Zhang
  2014-01-26 21:12 ` David Rientjes
  0 siblings, 1 reply; 2+ messages in thread
From: Zhihui Zhang @ 2014-01-26 20:13 UTC (permalink / raw)
  To: tglx, mingo, hpa; +Cc: linux-kernel

This unifies the way to specify start VMA on both 32 and 64-bit platforms.
I would like to remove __PHYSICAL_START as well, but that appears to be harder.

Signed-off-by: Zhihui Zhang <zzhsuny@gmail.com>
---
 arch/x86/kernel/vmlinux.lds.S | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/x86/kernel/vmlinux.lds.S b/arch/x86/kernel/vmlinux.lds.S
index da6b35a..e81bf49 100644
--- a/arch/x86/kernel/vmlinux.lds.S
+++ b/arch/x86/kernel/vmlinux.lds.S
@@ -81,11 +81,10 @@ PHDRS {
 
 SECTIONS
 {
-#ifdef CONFIG_X86_32
         . = LOAD_OFFSET + LOAD_PHYSICAL_ADDR;
+#ifdef CONFIG_X86_32
         phys_startup_32 = startup_32 - LOAD_OFFSET;
 #else
-        . = __START_KERNEL;
         phys_startup_64 = startup_64 - LOAD_OFFSET;
 #endif
 
-- 
1.8.1.2


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

* Re: [PATCH] Use LOAD_PHYSICAL_ADDR in vmlinux.lds.S
  2014-01-26 20:13 [PATCH] Use LOAD_PHYSICAL_ADDR in vmlinux.lds.S Zhihui Zhang
@ 2014-01-26 21:12 ` David Rientjes
  0 siblings, 0 replies; 2+ messages in thread
From: David Rientjes @ 2014-01-26 21:12 UTC (permalink / raw)
  To: Zhihui Zhang; +Cc: tglx, mingo, hpa, linux-kernel

On Sun, 26 Jan 2014, Zhihui Zhang wrote:

> diff --git a/arch/x86/kernel/vmlinux.lds.S b/arch/x86/kernel/vmlinux.lds.S
> index da6b35a..e81bf49 100644
> --- a/arch/x86/kernel/vmlinux.lds.S
> +++ b/arch/x86/kernel/vmlinux.lds.S
> @@ -81,11 +81,10 @@ PHDRS {
>  
>  SECTIONS
>  {
> -#ifdef CONFIG_X86_32
>          . = LOAD_OFFSET + LOAD_PHYSICAL_ADDR;
> +#ifdef CONFIG_X86_32
>          phys_startup_32 = startup_32 - LOAD_OFFSET;
>  #else
> -        . = __START_KERNEL;
>          phys_startup_64 = startup_64 - LOAD_OFFSET;
>  #endif
>  

Makes sense, but any reason to not make LOAD_PHYSICAL_ADDR to use ALIGN()?

(You'll also want to prefix the patch title in the subject line with 
"x86, boot:" and cc x86@kernel.org on patches like this.)

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

end of thread, other threads:[~2014-01-26 21:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-26 20:13 [PATCH] Use LOAD_PHYSICAL_ADDR in vmlinux.lds.S Zhihui Zhang
2014-01-26 21:12 ` David Rientjes

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox