2008-02-13 21:31:00

by Adrian Bunk

Subject: [2.6 patch] x86: make mxcsr_feature_mask static again

mxcsr_feature_mask needlessly became global.

Signed-off-by: Adrian Bunk <[email protected]>

---

arch/x86/kernel/i387.c | 2 +-
include/asm-x86/i387.h | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)

76adc866d04ed20a5b2eb548920f93ad87268cbd
diff --git a/arch/x86/kernel/i387.c b/arch/x86/kernel/i387.c
index 26719bd..763dfc4 100644
--- a/arch/x86/kernel/i387.c
+++ b/arch/x86/kernel/i387.c
@@ -39,7 +39,7 @@
#define HAVE_HWFP 1
#endif

-unsigned int mxcsr_feature_mask __read_mostly = 0xffffffffu;
+static unsigned int mxcsr_feature_mask __read_mostly = 0xffffffffu;

void mxcsr_feature_mask_init(void)
{
diff --git a/include/asm-x86/i387.h b/include/asm-x86/i387.h
index ba8105c..4c7706a 100644
--- a/include/asm-x86/i387.h
+++ b/include/asm-x86/i387.h
@@ -19,7 +19,6 @@
#include <asm/uaccess.h>

extern void fpu_init(void);
-extern unsigned int mxcsr_feature_mask;
extern void mxcsr_feature_mask_init(void);
extern void init_fpu(struct task_struct *child);
extern asmlinkage void math_state_restore(void);

2008-02-13 21:59:51

by Roland McGrath

Subject: Re: [2.6 patch] x86: make mxcsr_feature_mask static again

Good change. Making it global was needed temporarily in between some of
the i387 unification patches, but is unnecessary after the final merge.


Thanks,
Roland

2008-02-16 18:22:50

by Thomas Gleixner

Subject: Re: [2.6 patch] x86: make mxcsr_feature_mask static again

On Wed, 13 Feb 2008, Adrian Bunk wrote:

> mxcsr_feature_mask needlessly became global.
>
> Signed-off-by: Adrian Bunk <[email protected]>

Applied. Thanks,

tglx

2008-02-17 13:56:21

by Thomas Gleixner

Subject: Re: [2.6 patch] x86: make mxcsr_feature_mask static again

On Wed, 13 Feb 2008, Adrian Bunk wrote:

> mxcsr_feature_mask needlessly became global.
>
> Signed-off-by: Adrian Bunk <[email protected]>

Applied along with the other x86 ones.

Thanks,
tglx

> ---
>
> arch/x86/kernel/i387.c | 2 +-
> include/asm-x86/i387.h | 1 -
> 2 files changed, 1 insertion(+), 2 deletions(-)
>
> 76adc866d04ed20a5b2eb548920f93ad87268cbd
> diff --git a/arch/x86/kernel/i387.c b/arch/x86/kernel/i387.c
> index 26719bd..763dfc4 100644
> --- a/arch/x86/kernel/i387.c
> +++ b/arch/x86/kernel/i387.c
> @@ -39,7 +39,7 @@
> #define HAVE_HWFP 1
> #endif
>
> -unsigned int mxcsr_feature_mask __read_mostly = 0xffffffffu;
> +static unsigned int mxcsr_feature_mask __read_mostly = 0xffffffffu;
>
> void mxcsr_feature_mask_init(void)
> {
> diff --git a/include/asm-x86/i387.h b/include/asm-x86/i387.h
> index ba8105c..4c7706a 100644
> --- a/include/asm-x86/i387.h
> +++ b/include/asm-x86/i387.h
> @@ -19,7 +19,6 @@
> #include <asm/uaccess.h>
>
> extern void fpu_init(void);
> -extern unsigned int mxcsr_feature_mask;
> extern void mxcsr_feature_mask_init(void);
> extern void init_fpu(struct task_struct *child);
> extern asmlinkage void math_state_restore(void);
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>