2012-10-27 22:34:25

by Cesar Eduardo Barros

[permalink] [raw]
Subject: [PATCH] x86: fix comment for set_real_mode_permissions

The function which gets called very early is setup_real_mode(). The
set_real_mode_permissions() function is the arch_initcall.

Cc: Jarkko Sakkinen <[email protected]>
Cc: Thomas Gleixner <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: "H. Peter Anvin" <[email protected]>
Signed-off-by: Cesar Eduardo Barros <[email protected]>
---
arch/x86/realmode/init.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/x86/realmode/init.c b/arch/x86/realmode/init.c
index cbca565..b4c92c5 100644
--- a/arch/x86/realmode/init.c
+++ b/arch/x86/realmode/init.c
@@ -84,7 +84,7 @@ void __init setup_real_mode(void)
}

/*
- * set_real_mode_permissions() gets called very early, to guarantee the
+ * setup_real_mode() gets called very early, to guarantee the
* availability of low memory. This is before the proper kernel page
* tables are set up, so we cannot set page permissions in that
* function. Thus, we use an arch_initcall instead.
--
1.7.11.7


2012-10-29 07:42:00

by Sakkinen, Jarkko

[permalink] [raw]
Subject: Re: [PATCH] x86: fix comment for set_real_mode_permissions

On Sat, Oct 27, 2012 at 08:34:14PM -0200, Cesar Eduardo Barros wrote:
> The function which gets called very early is setup_real_mode(). The
> set_real_mode_permissions() function is the arch_initcall.
>
> Cc: Jarkko Sakkinen <[email protected]>
> Cc: Thomas Gleixner <[email protected]>
> Cc: Ingo Molnar <[email protected]>
> Cc: "H. Peter Anvin" <[email protected]>
> Signed-off-by: Cesar Eduardo Barros <[email protected]>

Acked-by: Jarkko Sakkinen <[email protected]>

> ---
> arch/x86/realmode/init.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/x86/realmode/init.c b/arch/x86/realmode/init.c
> index cbca565..b4c92c5 100644
> --- a/arch/x86/realmode/init.c
> +++ b/arch/x86/realmode/init.c
> @@ -84,7 +84,7 @@ void __init setup_real_mode(void)
> }
>
> /*
> - * set_real_mode_permissions() gets called very early, to guarantee the
> + * setup_real_mode() gets called very early, to guarantee the
> * availability of low memory. This is before the proper kernel page
> * tables are set up, so we cannot set page permissions in that
> * function. Thus, we use an arch_initcall instead.
> --
> 1.7.11.7
>

/Jarkko