Commit ad74557a authored by Ashok Raj's avatar Ashok Raj Committed by Linus Torvalds

[PATCH] introduce get_cpu_sysdev() to retrieve a sysfs entry for a cpu.

Some modules creating sysfs entries under /sys/devices/system/cpu/cpuX/
need to know the parent sysfs entry to make devices under them.  This will
just return the sysfs entry for a given cpu.

sysfs entries showing under each cpu sysfs can be easily created if such
entries can be created by registering a sysfs driver for cpuclass.  The
issue is when the entry is created the CPU may not be online, hence we
would need to defer the creation until the online notification comes.

Current users: cache entries for Intel CPU's and cpufreq subsystem.
Signed-off-by: default avatarAshok Raj <ashok.raj@intel.com>
Signed-off-by: default avatarVenkatesh Pallipadi <venkatesh.pallipadi@intel.com>
Cc: Dave Jones <davej@codemonkey.org.uk>
Cc: Zwane Mwaikambo <zwane@holomorphy.com>
Cc: Greg KH <greg@kroah.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 5d357040
...@@ -16,6 +16,8 @@ struct sysdev_class cpu_sysdev_class = { ...@@ -16,6 +16,8 @@ struct sysdev_class cpu_sysdev_class = {
}; };
EXPORT_SYMBOL(cpu_sysdev_class); EXPORT_SYMBOL(cpu_sysdev_class);
static struct sys_device *cpu_sys_devices[NR_CPUS];
#ifdef CONFIG_HOTPLUG_CPU #ifdef CONFIG_HOTPLUG_CPU
int __attribute__((weak)) smp_prepare_cpu (int cpu) int __attribute__((weak)) smp_prepare_cpu (int cpu)
{ {
...@@ -64,6 +66,7 @@ static void __devinit register_cpu_control(struct cpu *cpu) ...@@ -64,6 +66,7 @@ static void __devinit register_cpu_control(struct cpu *cpu)
} }
void unregister_cpu(struct cpu *cpu, struct node *root) void unregister_cpu(struct cpu *cpu, struct node *root)
{ {
int logical_cpu = cpu->sysdev.id;
if (root) if (root)
sysfs_remove_link(&root->sysdev.kobj, sysfs_remove_link(&root->sysdev.kobj,
...@@ -71,7 +74,7 @@ void unregister_cpu(struct cpu *cpu, struct node *root) ...@@ -71,7 +74,7 @@ void unregister_cpu(struct cpu *cpu, struct node *root)
sysdev_remove_file(&cpu->sysdev, &attr_online); sysdev_remove_file(&cpu->sysdev, &attr_online);
sysdev_unregister(&cpu->sysdev); sysdev_unregister(&cpu->sysdev);
cpu_sys_devices[logical_cpu] = NULL;
return; return;
} }
#else /* ... !CONFIG_HOTPLUG_CPU */ #else /* ... !CONFIG_HOTPLUG_CPU */
...@@ -103,10 +106,19 @@ int __devinit register_cpu(struct cpu *cpu, int num, struct node *root) ...@@ -103,10 +106,19 @@ int __devinit register_cpu(struct cpu *cpu, int num, struct node *root)
kobject_name(&cpu->sysdev.kobj)); kobject_name(&cpu->sysdev.kobj));
if (!error && !cpu->no_control) if (!error && !cpu->no_control)
register_cpu_control(cpu); register_cpu_control(cpu);
if (!error)
cpu_sys_devices[num] = &cpu->sysdev;
return error; return error;
} }
struct sys_device *get_cpu_sysdev(int cpu)
{
if (cpu < NR_CPUS)
return cpu_sys_devices[cpu];
else
return NULL;
}
EXPORT_SYMBOL_GPL(get_cpu_sysdev);
int __init cpu_dev_init(void) int __init cpu_dev_init(void)
{ {
......
...@@ -32,6 +32,7 @@ struct cpu { ...@@ -32,6 +32,7 @@ struct cpu {
}; };
extern int register_cpu(struct cpu *, int, struct node *); extern int register_cpu(struct cpu *, int, struct node *);
extern struct sys_device *get_cpu_sysdev(int cpu);
#ifdef CONFIG_HOTPLUG_CPU #ifdef CONFIG_HOTPLUG_CPU
extern void unregister_cpu(struct cpu *, struct node *); extern void unregister_cpu(struct cpu *, struct node *);
#endif #endif
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment