At Fri, 29 May 2015 19:18:47 +1000, Stephen Rothwell wrote: > > Nothing in asm/io.h uses anything from vmalloc.h, so remove the include > and fix up the build problems in an allmodconfig (64 bit and 32 bit) > build. > > This may be the place where x86 builds get vmalloc.h implicitly included > and that tends to hide places where vmalloc() et al are added to files > but the include of vmalloc.h is forgotten. > > Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx> > Cc: Ingo Molnar <mingo@xxxxxxxxxx> > Cc: "H. Peter Anvin" <hpa@xxxxxxxxx> > Cc: <x86@xxxxxxxxxx> > Cc: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> > Cc: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx> > Cc: David Vrabel <david.vrabel@xxxxxxxxxx> > Cc: Anton Vorontsov <anton@xxxxxxxxxx> > Cc: Colin Cross <ccross@xxxxxxxxxxx> > Cc: Kees Cook <keescook@xxxxxxxxxxxx> > Cc: Tony Luck <tony.luck@xxxxxxxxx> > Cc: "Rafael J. Wysocki" <rjw@xxxxxxxxxxxxx> > Cc: Len Brown <lenb@xxxxxxxxxx> > Cc: Kristen Carlson Accardi <kristen@xxxxxxxxxxxxxxx> > Cc: Viresh Kumar <viresh.kumar@xxxxxxxxxx> > Cc: Vinod Koul <vinod.koul@xxxxxxxxx> > Cc: "K. Y. Srinivasan" <kys@xxxxxxxxxxxxx> > Cc: Haiyang Zhang <haiyangz@xxxxxxxxxxxxx> > Cc: Hiral Patel <hiralpat@xxxxxxxxx> > Cc: Suma Ramars <sramars@xxxxxxxxx> > Cc: Brian Uchino <buchino@xxxxxxxxx> > Cc: "James E.J. Bottomley" <JBottomley@xxxxxxxx> > Cc: Jaroslav Kysela <perex@xxxxxxxx> > Cc: Takashi Iwai <tiwai@xxxxxxx> For the sound bits, Acked-by: Takashi Iwai <tiwai@xxxxxxx> thanks, Takashi > Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> > Suggested-by: David Miller <davem@xxxxxxxxxxxxx> > Signed-off-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> > > --- > > Based in Linus' tree of today. > > There are probably more places that need vmalloc.h included, but this > passes 64 bit and 32 bit allmodconfig builds, so is a place to start. > > Dave Miller suggested that I start this journey. > > arch/x86/include/asm/io.h | 2 -- > arch/x86/kernel/crash.c | 1 + > arch/x86/kernel/machine_kexec_64.c | 1 + > arch/x86/mm/pageattr-test.c | 1 + > arch/x86/mm/pageattr.c | 1 + > arch/x86/xen/p2m.c | 1 + > drivers/acpi/apei/erst.c | 1 + > drivers/cpufreq/intel_pstate.c | 1 + > drivers/dma/mic_x100_dma.c | 1 + > drivers/net/hyperv/netvsc.c | 1 + > drivers/net/hyperv/rndis_filter.c | 1 + > drivers/scsi/fnic/fnic_debugfs.c | 1 + > drivers/scsi/fnic/fnic_trace.c | 1 + > sound/pci/asihpi/hpioctl.c | 1 + > 14 files changed, 13 insertions(+), 2 deletions(-) > > diff --git a/arch/x86/include/asm/io.h b/arch/x86/include/asm/io.h > index 34a5b93704d3..5791e7ace9db 100644 > --- a/arch/x86/include/asm/io.h > +++ b/arch/x86/include/asm/io.h > @@ -197,8 +197,6 @@ extern void set_iounmap_nonlazy(void); > > #include <asm-generic/iomap.h> > > -#include <linux/vmalloc.h> > - > /* > * Convert a virtual cached pointer to an uncached pointer > */ > diff --git a/arch/x86/kernel/crash.c b/arch/x86/kernel/crash.c > index c76d3e37c6e1..e068d6683dba 100644 > --- a/arch/x86/kernel/crash.c > +++ b/arch/x86/kernel/crash.c > @@ -22,6 +22,7 @@ > #include <linux/elfcore.h> > #include <linux/module.h> > #include <linux/slab.h> > +#include <linux/vmalloc.h> > > #include <asm/processor.h> > #include <asm/hardirq.h> > diff --git a/arch/x86/kernel/machine_kexec_64.c b/arch/x86/kernel/machine_kexec_64.c > index 415480d3ea84..11546b462fa6 100644 > --- a/arch/x86/kernel/machine_kexec_64.c > +++ b/arch/x86/kernel/machine_kexec_64.c > @@ -17,6 +17,7 @@ > #include <linux/ftrace.h> > #include <linux/io.h> > #include <linux/suspend.h> > +#include <linux/vmalloc.h> > > #include <asm/init.h> > #include <asm/pgtable.h> > diff --git a/arch/x86/mm/pageattr-test.c b/arch/x86/mm/pageattr-test.c > index 6629f397b467..8ff686aa7e8c 100644 > --- a/arch/x86/mm/pageattr-test.c > +++ b/arch/x86/mm/pageattr-test.c > @@ -9,6 +9,7 @@ > #include <linux/random.h> > #include <linux/kernel.h> > #include <linux/mm.h> > +#include <linux/vmalloc.h> > > #include <asm/cacheflush.h> > #include <asm/pgtable.h> > diff --git a/arch/x86/mm/pageattr.c b/arch/x86/mm/pageattr.c > index 89af288ec674..bedfc794b4ba 100644 > --- a/arch/x86/mm/pageattr.c > +++ b/arch/x86/mm/pageattr.c > @@ -14,6 +14,7 @@ > #include <linux/percpu.h> > #include <linux/gfp.h> > #include <linux/pci.h> > +#include <linux/vmalloc.h> > > #include <asm/e820.h> > #include <asm/processor.h> > diff --git a/arch/x86/xen/p2m.c b/arch/x86/xen/p2m.c > index b47124d4cd67..8b7f18e200aa 100644 > --- a/arch/x86/xen/p2m.c > +++ b/arch/x86/xen/p2m.c > @@ -67,6 +67,7 @@ > #include <linux/seq_file.h> > #include <linux/bootmem.h> > #include <linux/slab.h> > +#include <linux/vmalloc.h> > > #include <asm/cache.h> > #include <asm/setup.h> > diff --git a/drivers/acpi/apei/erst.c b/drivers/acpi/apei/erst.c > index ed65e9c4b5b0..3670bbab57a3 100644 > --- a/drivers/acpi/apei/erst.c > +++ b/drivers/acpi/apei/erst.c > @@ -35,6 +35,7 @@ > #include <linux/nmi.h> > #include <linux/hardirq.h> > #include <linux/pstore.h> > +#include <linux/vmalloc.h> > #include <acpi/apei.h> > > #include "apei-internal.h" > diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c > index 6414661ac1c4..2ba53f4f6af2 100644 > --- a/drivers/cpufreq/intel_pstate.c > +++ b/drivers/cpufreq/intel_pstate.c > @@ -26,6 +26,7 @@ > #include <linux/fs.h> > #include <linux/debugfs.h> > #include <linux/acpi.h> > +#include <linux/vmalloc.h> > #include <trace/events/power.h> > > #include <asm/div64.h> > diff --git a/drivers/dma/mic_x100_dma.c b/drivers/dma/mic_x100_dma.c > index 6de2e677be04..74d9db05a5ad 100644 > --- a/drivers/dma/mic_x100_dma.c > +++ b/drivers/dma/mic_x100_dma.c > @@ -22,6 +22,7 @@ > #include <linux/module.h> > #include <linux/io.h> > #include <linux/seq_file.h> > +#include <linux/vmalloc.h> > > #include "mic_x100_dma.h" > > diff --git a/drivers/net/hyperv/netvsc.c b/drivers/net/hyperv/netvsc.c > index ea091bc5ff09..1e09243d5449 100644 > --- a/drivers/net/hyperv/netvsc.c > +++ b/drivers/net/hyperv/netvsc.c > @@ -28,6 +28,7 @@ > #include <linux/slab.h> > #include <linux/netdevice.h> > #include <linux/if_ether.h> > +#include <linux/vmalloc.h> > #include <asm/sync_bitops.h> > > #include "hyperv_net.h" > diff --git a/drivers/net/hyperv/rndis_filter.c b/drivers/net/hyperv/rndis_filter.c > index 9118cea91882..35a482d526d9 100644 > --- a/drivers/net/hyperv/rndis_filter.c > +++ b/drivers/net/hyperv/rndis_filter.c > @@ -27,6 +27,7 @@ > #include <linux/netdevice.h> > #include <linux/if_vlan.h> > #include <linux/nls.h> > +#include <linux/vmalloc.h> > > #include "hyperv_net.h" > > diff --git a/drivers/scsi/fnic/fnic_debugfs.c b/drivers/scsi/fnic/fnic_debugfs.c > index 5980c10c734d..d6498fabe628 100644 > --- a/drivers/scsi/fnic/fnic_debugfs.c > +++ b/drivers/scsi/fnic/fnic_debugfs.c > @@ -18,6 +18,7 @@ > #include <linux/module.h> > #include <linux/errno.h> > #include <linux/debugfs.h> > +#include <linux/vmalloc.h> > #include "fnic.h" > > static struct dentry *fnic_trace_debugfs_root; > diff --git a/drivers/scsi/fnic/fnic_trace.c b/drivers/scsi/fnic/fnic_trace.c > index 65a9bde26974..4e15c4bf0795 100644 > --- a/drivers/scsi/fnic/fnic_trace.c > +++ b/drivers/scsi/fnic/fnic_trace.c > @@ -21,6 +21,7 @@ > #include <linux/spinlock.h> > #include <linux/kallsyms.h> > #include <linux/time.h> > +#include <linux/vmalloc.h> > #include "fnic_io.h" > #include "fnic.h" > > diff --git a/sound/pci/asihpi/hpioctl.c b/sound/pci/asihpi/hpioctl.c > index 6610bd096fc9..d17937b92331 100644 > --- a/sound/pci/asihpi/hpioctl.c > +++ b/sound/pci/asihpi/hpioctl.c > @@ -32,6 +32,7 @@ > #include <linux/pci.h> > #include <linux/stringify.h> > #include <linux/module.h> > +#include <linux/vmalloc.h> > > #ifdef MODULE_FIRMWARE > MODULE_FIRMWARE("asihpi/dsp5000.bin"); > -- > 2.1.4 > > -- > Cheers, > Stephen Rothwell sfr@xxxxxxxxxxxxxxxx > [2 OpenPGP digital signature <application/pgp-signature (7bit)>] > -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html