* [PATCH] x86, efi: Mark disable_runtime as __initdata
@ 2013-02-22 15:03 ` Matt Fleming
0 siblings, 0 replies; 5+ messages in thread
From: Matt Fleming @ 2013-02-22 15:03 UTC (permalink / raw)
To: H. Peter Anvin
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-efi-u79uwXL29TY76Z2rM5mHXA, Yinghai Lu, Matt Fleming,
Satoru Takeuchi
From: Matt Fleming <matt.fleming-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
disable_runtime is only referenced from __init functions, so mark it
as __initdata.
Reported-by: Yinghai Lu <yinghai-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: Satoru Takeuchi <takeuchi_satoru-+CUm20s59erQFUHtdCDX3A@public.gmane.org>
Cc: H. Peter Anvin <hpa-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
Signed-off-by: Matt Fleming <matt.fleming-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
---
arch/x86/platform/efi/efi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/platform/efi/efi.c b/arch/x86/platform/efi/efi.c
index 2f81db4..5ae99f9 100644
--- a/arch/x86/platform/efi/efi.c
+++ b/arch/x86/platform/efi/efi.c
@@ -85,7 +85,7 @@ int efi_enabled(int facility)
}
EXPORT_SYMBOL(efi_enabled);
-static bool disable_runtime = false;
+static bool __initdata disable_runtime = false;
static int __init setup_noefi(char *arg)
{
disable_runtime = true;
--
1.7.11.7
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] x86, efi: Mark disable_runtime as __initdata
@ 2013-02-22 15:03 ` Matt Fleming
0 siblings, 0 replies; 5+ messages in thread
From: Matt Fleming @ 2013-02-22 15:03 UTC (permalink / raw)
To: H. Peter Anvin
Cc: linux-kernel, linux-efi, Yinghai Lu, Matt Fleming,
Satoru Takeuchi
From: Matt Fleming <matt.fleming@intel.com>
disable_runtime is only referenced from __init functions, so mark it
as __initdata.
Reported-by: Yinghai Lu <yinghai@kernel.org>
Cc: Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com>
Cc: H. Peter Anvin <hpa@linux.intel.com>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>
---
arch/x86/platform/efi/efi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/platform/efi/efi.c b/arch/x86/platform/efi/efi.c
index 2f81db4..5ae99f9 100644
--- a/arch/x86/platform/efi/efi.c
+++ b/arch/x86/platform/efi/efi.c
@@ -85,7 +85,7 @@ int efi_enabled(int facility)
}
EXPORT_SYMBOL(efi_enabled);
-static bool disable_runtime = false;
+static bool __initdata disable_runtime = false;
static int __init setup_noefi(char *arg)
{
disable_runtime = true;
--
1.7.11.7
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] x86, efi: Mark disable_runtime as __initdata
2013-02-22 15:03 ` Matt Fleming
@ 2013-02-25 23:55 ` Satoru Takeuchi
-1 siblings, 0 replies; 5+ messages in thread
From: Satoru Takeuchi @ 2013-02-25 23:55 UTC (permalink / raw)
To: Matt Fleming
Cc: H. Peter Anvin, linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-efi-u79uwXL29TY76Z2rM5mHXA, Yinghai Lu, Matt Fleming
(2013/02/23 0:03), Matt Fleming wrote:
> From: Matt Fleming <matt.fleming-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
>
> disable_runtime is only referenced from __init functions, so mark it
> as __initdata.
>
> Reported-by: Yinghai Lu <yinghai-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> Cc: Satoru Takeuchi <takeuchi_satoru-+CUm20s59erQFUHtdCDX3A@public.gmane.org>
> Cc: H. Peter Anvin <hpa-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
> Signed-off-by: Matt Fleming <matt.fleming-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
It looks good to me.
Reviewed-by: Satoru Takeuchi <takeuchi_satoru-+CUm20s59erQFUHtdCDX3A@public.gmane.org>
> ---
> arch/x86/platform/efi/efi.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/x86/platform/efi/efi.c b/arch/x86/platform/efi/efi.c
> index 2f81db4..5ae99f9 100644
> --- a/arch/x86/platform/efi/efi.c
> +++ b/arch/x86/platform/efi/efi.c
> @@ -85,7 +85,7 @@ int efi_enabled(int facility)
> }
> EXPORT_SYMBOL(efi_enabled);
>
> -static bool disable_runtime = false;
> +static bool __initdata disable_runtime = false;
> static int __init setup_noefi(char *arg)
> {
> disable_runtime = true;
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] x86, efi: Mark disable_runtime as __initdata
@ 2013-02-25 23:55 ` Satoru Takeuchi
0 siblings, 0 replies; 5+ messages in thread
From: Satoru Takeuchi @ 2013-02-25 23:55 UTC (permalink / raw)
To: Matt Fleming
Cc: H. Peter Anvin, linux-kernel, linux-efi, Yinghai Lu, Matt Fleming
(2013/02/23 0:03), Matt Fleming wrote:
> From: Matt Fleming <matt.fleming@intel.com>
>
> disable_runtime is only referenced from __init functions, so mark it
> as __initdata.
>
> Reported-by: Yinghai Lu <yinghai@kernel.org>
> Cc: Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com>
> Cc: H. Peter Anvin <hpa@linux.intel.com>
> Signed-off-by: Matt Fleming <matt.fleming@intel.com>
It looks good to me.
Reviewed-by: Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com>
> ---
> arch/x86/platform/efi/efi.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/x86/platform/efi/efi.c b/arch/x86/platform/efi/efi.c
> index 2f81db4..5ae99f9 100644
> --- a/arch/x86/platform/efi/efi.c
> +++ b/arch/x86/platform/efi/efi.c
> @@ -85,7 +85,7 @@ int efi_enabled(int facility)
> }
> EXPORT_SYMBOL(efi_enabled);
>
> -static bool disable_runtime = false;
> +static bool __initdata disable_runtime = false;
> static int __init setup_noefi(char *arg)
> {
> disable_runtime = true;
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [tip:x86/urgent] x86, efi: Mark disable_runtime as __initdata
2013-02-22 15:03 ` Matt Fleming
(?)
(?)
@ 2013-02-26 0:22 ` tip-bot for Matt Fleming
-1 siblings, 0 replies; 5+ messages in thread
From: tip-bot for Matt Fleming @ 2013-02-26 0:22 UTC (permalink / raw)
To: linux-tip-commits
Cc: linux-kernel, hpa, mingo, yinghai, matt.fleming, takeuchi_satoru,
tglx, hpa
Commit-ID: 058e7b5814534461b0e2468fce5a8f8d2f43c38f
Gitweb: http://git.kernel.org/tip/058e7b5814534461b0e2468fce5a8f8d2f43c38f
Author: Matt Fleming <matt.fleming@intel.com>
AuthorDate: Fri, 22 Feb 2013 15:03:47 +0000
Committer: H. Peter Anvin <hpa@linux.intel.com>
CommitDate: Mon, 25 Feb 2013 16:01:35 -0800
x86, efi: Mark disable_runtime as __initdata
disable_runtime is only referenced from __init functions, so mark it
as __initdata.
Reported-by: Yinghai Lu <yinghai@kernel.org>
Reviewed-by: Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>
Link: http://lkml.kernel.org/r/1361545427-26393-1-git-send-email-matt@console-pimps.org
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
---
arch/x86/platform/efi/efi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/platform/efi/efi.c b/arch/x86/platform/efi/efi.c
index e2cd38f..ec9f325 100644
--- a/arch/x86/platform/efi/efi.c
+++ b/arch/x86/platform/efi/efi.c
@@ -85,7 +85,7 @@ int efi_enabled(int facility)
}
EXPORT_SYMBOL(efi_enabled);
-static bool disable_runtime = false;
+static bool __initdata disable_runtime = false;
static int __init setup_noefi(char *arg)
{
disable_runtime = true;
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-02-26 0:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-02-22 15:03 [PATCH] x86, efi: Mark disable_runtime as __initdata Matt Fleming
2013-02-22 15:03 ` Matt Fleming
[not found] ` <1361545427-26393-1-git-send-email-matt-HNK1S37rvNbeXh+fF434Mdi2O/JbrIOy@public.gmane.org>
2013-02-25 23:55 ` Satoru Takeuchi
2013-02-25 23:55 ` Satoru Takeuchi
2013-02-26 0:22 ` [tip:x86/urgent] " tip-bot for Matt Fleming
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.