lkml.org 
[lkml]   [2019]   [Jan]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] x86/boot: comments cleanup & indentation fix
From
Date
Hi,
Is the patch not right?

--
Sincerely,
Cao jin

On 1/4/19 8:04 PM, Cao jin wrote:
> No real code modification, just cleanup:
> 1. remove redundant comments which have already appeared above
> 2. comments improvement:
> "aligned to a 2M boundary"
> -->
> "aligned up to CONFIG_PHYSICAL_ALIGN boundary"
> 3. typo fix: uncompression --> decompression
> 4. indentation fix in linker script: spaces -> tab
>
> Signed-off-by: Cao jin <caoj.fnst@cn.fujitsu.com>
> ---
> arch/x86/boot/compressed/head_64.S | 13 +++++--------
> arch/x86/boot/compressed/vmlinux.lds.S | 4 ++--
> 2 files changed, 7 insertions(+), 10 deletions(-)
>
> diff --git a/arch/x86/boot/compressed/head_64.S b/arch/x86/boot/compressed/head_64.S
> index 64037895b085..58f6a467f1fa 100644
> --- a/arch/x86/boot/compressed/head_64.S
> +++ b/arch/x86/boot/compressed/head_64.S
> @@ -90,9 +90,6 @@ ENTRY(startup_32)
> jnz no_longmode
>
> /*
> - * Compute the delta between where we were compiled to run at
> - * and where the code will actually run at.
> - *
> * %ebp contains the address we are loaded at by the boot loader and %ebx
> * contains the address where we should move the kernel image temporarily
> * for safe in-place decompression.
> @@ -272,12 +269,12 @@ ENTRY(startup_64)
>
> /*
> * Compute the decompressed kernel start address. It is where
> - * we were loaded at aligned to a 2M boundary. %rbp contains the
> - * decompressed kernel start address.
> + * we were loaded at aligned up to CONFIG_PHYSICAL_ALIGN boundary.
> + * %rbp contains the decompressed kernel start address.
> *
> * If it is a relocatable kernel then decompress and run the kernel
> - * from load address aligned to 2MB addr, otherwise decompress and
> - * run the kernel from LOAD_PHYSICAL_ADDR
> + * from load address aligned up to CONFIG_PHYSICAL_ALIGN, otherwise
> + * decompress and run the kernel from LOAD_PHYSICAL_ADDR
> *
> * We cannot rely on the calculation done in 32-bit mode, since we
> * may have been invoked via the 64-bit entry point.
> @@ -525,7 +522,7 @@ relocated:
> */
> pushq %rsi /* Save the real mode argument */
> movq %rsi, %rdi /* real mode address */
> - leaq boot_heap(%rip), %rsi /* malloc area for uncompression */
> + leaq boot_heap(%rip), %rsi /* malloc area for decompression */
> leaq input_data(%rip), %rdx /* input_data */
> movl $z_input_len, %ecx /* input_len */
> movq %rbp, %r8 /* output target address */
> diff --git a/arch/x86/boot/compressed/vmlinux.lds.S b/arch/x86/boot/compressed/vmlinux.lds.S
> index f491bbde8493..c07c8aba0755 100644
> --- a/arch/x86/boot/compressed/vmlinux.lds.S
> +++ b/arch/x86/boot/compressed/vmlinux.lds.S
> @@ -64,8 +64,8 @@ SECTIONS
> _ebss = .;
> }
> #ifdef CONFIG_X86_64
> - . = ALIGN(PAGE_SIZE);
> - .pgtable : {
> + . = ALIGN(PAGE_SIZE);
> + .pgtable : {
> _pgtable = . ;
> *(.pgtable)
> _epgtable = . ;
>





\
 
 \ /
  Last update: 2019-01-30 02:23    [W:0.054 / U:4.336 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site