From mboxrd@z Thu Jan  1 00:00:00 1970
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Subject: Re: [PATCH] EFI: suppress bogus loader warning
Date: Fri, 19 Dec 2014 14:08:23 -0500
Message-ID: <20141219190823.GA9213@laptop.dumpdata.com>
References: <5494173E0200007800050F59@mail.emea.novell.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
Received: from mail6.bemta3.messagelabs.com ([195.245.230.39])
	by lists.xen.org with esmtp (Exim 4.72)
	(envelope-from <konrad.wilk@oracle.com>) id 1Y22uc-0008Eg-2c
	for xen-devel@lists.xenproject.org; Fri, 19 Dec 2014 19:08:30 +0000
Content-Disposition: inline
In-Reply-To: <5494173E0200007800050F59@mail.emea.novell.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Jan Beulich <JBeulich@suse.com>
Cc: roy.franz@linaro.org, xen-devel <xen-devel@lists.xenproject.org>
List-Id: xen-devel@lists.xenproject.org

On Fri, Dec 19, 2014 at 11:17:02AM +0000, Jan Beulich wrote:
> This was accidentally lost in commit fbc3d9a220 ("EFI: add
> efi_arch_handle_cmdline() for processing commandline"), leading to the
> "Unknown command line option" warning being printed whenever options
> get passed to the core hypervisor or the Dom0 kernel.
> 
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Release-Acked-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> 
> --- a/xen/common/efi/boot.c
> +++ b/xen/common/efi/boot.c
> @@ -311,6 +311,7 @@ static unsigned int __init get_argv(unsi
>                  ++argc;
>              else if ( prev && wstrcmp(prev, L"--") == 0 )
>              {
> +                --argv;
>                  if ( options )
>                      *options = cmdline;
>                  break;
> 
> 
> 

> EFI: suppress bogus loader warning
> 
> This was accidentally lost in commit fbc3d9a220 ("EFI: add
> efi_arch_handle_cmdline() for processing commandline"), leading to the
> "Unknown command line option" warning being printed whenever options
> get passed to the core hypervisor or the Dom0 kernel.
> 
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> 
> --- a/xen/common/efi/boot.c
> +++ b/xen/common/efi/boot.c
> @@ -311,6 +311,7 @@ static unsigned int __init get_argv(unsi
>                  ++argc;
>              else if ( prev && wstrcmp(prev, L"--") == 0 )
>              {
> +                --argv;
>                  if ( options )
>                      *options = cmdline;
>                  break;