diff options
| author | Paul Gortmaker <paul.gortmaker@windriver.com> | 2011-07-18 13:03:04 -0400 |
|---|---|---|
| committer | Paul Gortmaker <paul.gortmaker@windriver.com> | 2011-10-31 09:20:12 -0400 |
| commit | 967d1f90625ed9c1ab205d3f738fedf9d852e1fd (patch) | |
| tree | b21c7da43e181eee150db89e3eebec083b3b1c2c /kernel/irq_work.c | |
| parent | 74da1ff71350f3638c51613085f89c0865d7fe08 (diff) | |
| download | lwn-967d1f90625ed9c1ab205d3f738fedf9d852e1fd.tar.gz lwn-967d1f90625ed9c1ab205d3f738fedf9d852e1fd.zip | |
kernel: fix two implicit header assumptions in irq_work.c
Up until now, this file was getting percpu.h because nearly every
file was implicitly getting module.h (and all its sub-includes).
But we want to clean that up, so call out percpu.h explicitly.
Otherwise we'll get things like this on an ARM build:
kernel/irq_work.c:48: error: expected declaration specifiers or '...' before 'irq_work_list'
kernel/irq_work.c:48: warning: type defaults to 'int' in declaration of 'DEFINE_PER_CPU'
The same thing was happening for builds on ARM for asm/processor.h
kernel/irq_work.c: In function 'irq_work_sync':
kernel/irq_work.c:166: error: implicit declaration of function 'cpu_relax'
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Diffstat (limited to 'kernel/irq_work.c')
| -rw-r--r-- | kernel/irq_work.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/kernel/irq_work.c b/kernel/irq_work.c index 3e460ea44955..c3c46c72046e 100644 --- a/kernel/irq_work.c +++ b/kernel/irq_work.c @@ -8,7 +8,9 @@ #include <linux/kernel.h> #include <linux/export.h> #include <linux/irq_work.h> +#include <linux/percpu.h> #include <linux/hardirq.h> +#include <asm/processor.h> /* * An entry can be in one of four states: |