On Sat, 24 Feb 2007 12:22:11 +0000 Ralf Baechle <[EMAIL PROTECTED]> wrote:
> sysdev.h uses THIS_MODULE so should include <linux/module.h>.
>
> Signed-off-by: Ralf Baechle <[EMAIL PROTECTED]>
>
> diff --git a/include/linux/sysdev.h b/include/linux/sysdev.h
> index 389ccf8..e699ab2 100644
> --- a/include/linux/sysdev.h
> +++ b/include/linux/sysdev.h
> @@ -22,6 +22,7 @@
> #define _SYSDEV_H_
>
> #include <linux/kobject.h>
> +#include <linux/module.h>
> #include <linux/pm.h>
>
You can't just make changes like this without a lot of compile testing, I'm
afraid.
This causes a recursive inclusion and sched.h blows up:
In file included from include/linux/utsname.h:35,
from include/asm/elf.h:12,
from include/linux/elf.h:7,
from include/linux/module.h:15,
from include/linux/sysdev.h:25,
from kernel/time/clocksource.c:28:
include/linux/sched.h:1648: warning: 'struct sysdev_class' declared inside
parameter list
include/linux/sched.h:1648: warning: its scope is only this definition or
declaration, which is probably not what you want
I think we can fix that by moving the declarations into cpu.h and getting
that unpleasant include out of sched.h.
Of course, this will probably make other things blow up and additional
sysdev.h includes will now be needed. We'll see..
diff -puN
include/linux/cpu.h~linux-sysdevh-needs-to-include-linux-moduleh-up-fix
include/linux/cpu.h
--- a/include/linux/cpu.h~linux-sysdevh-needs-to-include-linux-moduleh-up-fix
+++ a/include/linux/cpu.h
@@ -41,6 +41,9 @@ extern void cpu_remove_sysdev_attr(struc
extern int cpu_add_sysdev_attr_group(struct attribute_group *attrs);
extern void cpu_remove_sysdev_attr_group(struct attribute_group *attrs);
+extern struct sysdev_attribute attr_sched_mc_power_savings;
+extern struct sysdev_attribute attr_sched_smt_power_savings;
+extern int sched_create_sysfs_power_savings_entries(struct sysdev_class *cls);
#ifdef CONFIG_HOTPLUG_CPU
extern void unregister_cpu(struct cpu *cpu);
diff -puN
include/linux/sched.h~linux-sysdevh-needs-to-include-linux-moduleh-up-fix
include/linux/sched.h
--- a/include/linux/sched.h~linux-sysdevh-needs-to-include-linux-moduleh-up-fix
+++ a/include/linux/sched.h
@@ -1642,10 +1642,7 @@ static inline void arch_pick_mmap_layout
extern long sched_setaffinity(pid_t pid, cpumask_t new_mask);
extern long sched_getaffinity(pid_t pid, cpumask_t *mask);
-#include <linux/sysdev.h>
extern int sched_mc_power_savings, sched_smt_power_savings;
-extern struct sysdev_attribute attr_sched_mc_power_savings,
attr_sched_smt_power_savings;
-extern int sched_create_sysfs_power_savings_entries(struct sysdev_class *cls);
extern void normalize_rt_tasks(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/
