eeaaa96a3a
It was reported that compiling for 32-bit caused a bunch of section mismatch warnings: VDSOSYM arch/x86/vdso/vdso32-syms.lds LD arch/x86/vdso/built-in.o LD arch/x86/built-in.o WARNING: arch/x86/built-in.o(.data+0x5af0): Section mismatch in reference from the variable test_nmi_ipi_callback_na.10451 to the function .init.text:test_nmi_ipi_callback() [...] WARNING: arch/x86/built-in.o(.data+0x5b04): Section mismatch in reference from the variable nmi_unk_cb_na.10399 to the function .init.text:nmi_unk_cb() The variable nmi_unk_cb_na.10399 references the function __init nmi_unk_cb() [...] Both of these are attributed to the internal representation of the nmiaction struct created during register_nmi_handler. The reason for this is that those structs are not defined in the init section whereas the rest of the code in nmi_selftest.c is. To resolve this, I created a new #define, register_nmi_handler_initonly, that tags the struct as __initdata to resolve the mismatch. This #define should only be used in rare situations where the register/unregister is called during init of the kernel. Big thanks to Jan Beulich for decoding this for me as I didn't have a clue what was going on. Reported-by: Witold Baryluk <baryluk@smp.if.uj.edu.pl> Tested-by: Witold Baryluk <baryluk@smp.if.uj.edu.pl> Cc: Jan Beulich <JBeulich@suse.com> Signed-off-by: Don Zickus <dzickus@redhat.com> Link: http://lkml.kernel.org/r/1338991542-23000-1-git-send-email-dzickus@redhat.com Signed-off-by: Ingo Molnar <mingo@kernel.org>
79 lines
1.7 KiB
C
79 lines
1.7 KiB
C
#ifndef _ASM_X86_NMI_H
|
|
#define _ASM_X86_NMI_H
|
|
|
|
#include <linux/pm.h>
|
|
#include <asm/irq.h>
|
|
#include <asm/io.h>
|
|
|
|
#ifdef CONFIG_X86_LOCAL_APIC
|
|
|
|
extern int avail_to_resrv_perfctr_nmi_bit(unsigned int);
|
|
extern int reserve_perfctr_nmi(unsigned int);
|
|
extern void release_perfctr_nmi(unsigned int);
|
|
extern int reserve_evntsel_nmi(unsigned int);
|
|
extern void release_evntsel_nmi(unsigned int);
|
|
|
|
struct ctl_table;
|
|
extern int proc_nmi_enabled(struct ctl_table *, int ,
|
|
void __user *, size_t *, loff_t *);
|
|
extern int unknown_nmi_panic;
|
|
|
|
void arch_trigger_all_cpu_backtrace(void);
|
|
#define arch_trigger_all_cpu_backtrace arch_trigger_all_cpu_backtrace
|
|
#endif
|
|
|
|
#define NMI_FLAG_FIRST 1
|
|
|
|
enum {
|
|
NMI_LOCAL=0,
|
|
NMI_UNKNOWN,
|
|
NMI_SERR,
|
|
NMI_IO_CHECK,
|
|
NMI_MAX
|
|
};
|
|
|
|
#define NMI_DONE 0
|
|
#define NMI_HANDLED 1
|
|
|
|
typedef int (*nmi_handler_t)(unsigned int, struct pt_regs *);
|
|
|
|
struct nmiaction {
|
|
struct list_head list;
|
|
nmi_handler_t handler;
|
|
unsigned long flags;
|
|
const char *name;
|
|
};
|
|
|
|
#define register_nmi_handler(t, fn, fg, n) \
|
|
({ \
|
|
static struct nmiaction fn##_na = { \
|
|
.handler = (fn), \
|
|
.name = (n), \
|
|
.flags = (fg), \
|
|
}; \
|
|
__register_nmi_handler((t), &fn##_na); \
|
|
})
|
|
|
|
/*
|
|
* For special handlers that register/unregister in the
|
|
* init section only. This should be considered rare.
|
|
*/
|
|
#define register_nmi_handler_initonly(t, fn, fg, n) \
|
|
({ \
|
|
static struct nmiaction fn##_na __initdata = { \
|
|
.handler = (fn), \
|
|
.name = (n), \
|
|
.flags = (fg), \
|
|
}; \
|
|
__register_nmi_handler((t), &fn##_na); \
|
|
})
|
|
|
|
int __register_nmi_handler(unsigned int, struct nmiaction *);
|
|
|
|
void unregister_nmi_handler(unsigned int, const char *);
|
|
|
|
void stop_nmi(void);
|
|
void restart_nmi(void);
|
|
void local_touch_nmi(void);
|
|
|
|
#endif /* _ASM_X86_NMI_H */
|