(2013/02/23 0:03), Matt Fleming wrote: > From: Matt Fleming <matt.flem...@intel.com> > > disable_runtime is only referenced from __init functions, so mark it > as __initdata. > > Reported-by: Yinghai Lu <ying...@kernel.org> > Cc: Satoru Takeuchi <takeuchi_sat...@jp.fujitsu.com> > Cc: H. Peter Anvin <h...@linux.intel.com> > Signed-off-by: Matt Fleming <matt.flem...@intel.com>
It looks good to me. Reviewed-by: Satoru Takeuchi <takeuchi_sat...@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; > -- 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/