diff mbox series

x86/boot: properly "ignore" early evaluated "no-real-mode"

Message ID 6b7e3281-4893-a5fb-cd1e-5b1918a1ee05@suse.com (mailing list archive)
State New, archived
Headers show
Series x86/boot: properly "ignore" early evaluated "no-real-mode" | expand

Commit Message

Jan Beulich Sept. 13, 2021, 4:02 p.m. UTC 
The option parser takes off "no-" prefixes before matching, so they also
shouldn't be specified to match against.

Fixes: e44d98608476 ("x86/setup: Ignore early boot parameters like no-real-mode")
Signed-off-by: Jan Beulich <jbeulich@suse.com>

Comments

Andrew Cooper Sept. 13, 2021, 4:38 p.m. UTC | #1 
On 13/09/2021 17:02, Jan Beulich wrote:
> The option parser takes off "no-" prefixes before matching, so they also
> shouldn't be specified to match against.
>
> Fixes: e44d98608476 ("x86/setup: Ignore early boot parameters like no-real-mode")
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Oops.

Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>

>
> --- a/xen/arch/x86/setup.c
> +++ b/xen/arch/x86/setup.c
> @@ -692,7 +692,7 @@ static void __init noreturn reinit_bsp_s
>   * has options that are only used during the very initial boot process,
>   * so they can be ignored now.
>   */
> -ignore_param("no-real-mode");
> +ignore_param("real-mode");
>  ignore_param("edd");
>  ignore_param("edid");
>  
>
diff mbox series

Patch

--- a/xen/arch/x86/setup.c
+++ b/xen/arch/x86/setup.c
@@ -692,7 +692,7 @@  static void __init noreturn reinit_bsp_s
  * has options that are only used during the very initial boot process,
  * so they can be ignored now.
  */
-ignore_param("no-real-mode");
+ignore_param("real-mode");
 ignore_param("edd");
 ignore_param("edid");