On Thu, Jan 03, 2013 at 04:48:33PM -0800, Yinghai Lu wrote:
> later will check ext_cmd_line_ptr at the same time.
> 
> Signed-off-by: Yinghai Lu <ying...@kernel.org>
> Cc: Gokul Caushik <caush...@gmail.com>
> Cc: Josh Triplett <j...@joshtriplett.org>
> Cc: Joe Millenbach <jmillenb...@gmail.com>
> Cc: Alexander Duyck <alexander.h.du...@intel.com>
> ---
>  arch/x86/boot/compressed/cmdline.c |   10 ++++++++--
>  arch/x86/kernel/head64.c           |   13 +++++++++++--
>  2 files changed, 19 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/x86/boot/compressed/cmdline.c 
> b/arch/x86/boot/compressed/cmdline.c
> index 10f6b11..b4c913c 100644
> --- a/arch/x86/boot/compressed/cmdline.c
> +++ b/arch/x86/boot/compressed/cmdline.c
> @@ -13,13 +13,19 @@ static inline char rdfs8(addr_t addr)
>       return *((char *)(fs + addr));
>  }
>  #include "../cmdline.c"
> +static unsigned long get_cmd_line_ptr(void)
> +{
> +     unsigned long cmd_line_ptr = real_mode->hdr.cmd_line_ptr;
> +
> +     return cmd_line_ptr;

        return (unsigned long)real_mode->hdr.cmd_line_ptr;

should suffice.

> +}
>  int cmdline_find_option(const char *option, char *buffer, int bufsize)
>  {
> -     return __cmdline_find_option(real_mode->hdr.cmd_line_ptr, option, 
> buffer, bufsize);
> +     return __cmdline_find_option(get_cmd_line_ptr(), option, buffer, 
> bufsize);
>  }
>  int cmdline_find_option_bool(const char *option)
>  {
> -     return __cmdline_find_option_bool(real_mode->hdr.cmd_line_ptr, option);
> +     return __cmdline_find_option_bool(get_cmd_line_ptr(), option);
>  }
>  
>  #endif
> diff --git a/arch/x86/kernel/head64.c b/arch/x86/kernel/head64.c
> index c463725..316e7b2 100644
> --- a/arch/x86/kernel/head64.c
> +++ b/arch/x86/kernel/head64.c
> @@ -111,13 +111,22 @@ static void __init clear_bss(void)
>              (unsigned long) __bss_stop - (unsigned long) __bss_start);
>  }
>  
> +static unsigned long get_cmd_line_ptr(void)
> +{
> +     unsigned long cmd_line_ptr = boot_params.hdr.cmd_line_ptr;
> +
> +     return cmd_line_ptr;

ditto.

> +}
> +
>  static void __init copy_bootdata(char *real_mode_data)
>  {
>       char * command_line;
> +     unsigned long cmd_line_ptr;
>  
>       memcpy(&boot_params, real_mode_data, sizeof boot_params);
> -     if (boot_params.hdr.cmd_line_ptr) {
> -             command_line = __va(boot_params.hdr.cmd_line_ptr);
> +     cmd_line_ptr = get_cmd_line_ptr();
> +     if (cmd_line_ptr) {
> +             command_line = __va(cmd_line_ptr);
>               memcpy(boot_command_line, command_line, COMMAND_LINE_SIZE);
>       }
>  }
> -- 
> 1.7.10.4
> 
> 

-- 
Regards/Gruss,
Boris.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to