Re: [PATCH v2 2/3] sysfs cpu probe/release files

2009-11-25 Thread Greg KH
On Wed, Nov 25, 2009 at 01:12:21AM -0600, Nathan Fontenot wrote:
 In order to support kernel DLPAR of CPU resources we need to provide an
 interface to add (probe) and remove (release) the resource from the system.
 This patch Creates new generic probe and release sysfs files to facilitate
 cpu probe/release.  The probe/release interface provides for allowing each
 arch to supply their own routines for implementing the backend of adding
 and removing cpus to/from the system.

 This also creates the powerpc specific stubs to handle the arch callouts 
 from writes to the sysfs files.

Can you document these new sysfs files with some information in the
Documentation/ABI/ directory please?

thanks,

greg k-h
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH v2 2/3] sysfs cpu probe/release files

2009-11-25 Thread Paul Mackerras
Nathan Fontenot writes:

 In order to support kernel DLPAR of CPU resources we need to provide an
 interface to add (probe) and remove (release) the resource from the system.
 This patch Creates new generic probe and release sysfs files to facilitate
 cpu probe/release.  The probe/release interface provides for allowing each
 arch to supply their own routines for implementing the backend of adding
 and removing cpus to/from the system.
 
 This also creates the powerpc specific stubs to handle the arch callouts 
 from writes to the sysfs files.
 
 The creation and use of these files is regulated by the 
 CONFIG_ARCH_CPU_PROBE_RELEASE option so that only architectures that need the
 capability will have the files created.
 
 Signed-off-by: Nathan Fontenot nf...@austin.ibm.com

Acked-by: Paul Mackerras pau...@samba.org
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


[PATCH v2 2/3] sysfs cpu probe/release files

2009-11-24 Thread Nathan Fontenot

In order to support kernel DLPAR of CPU resources we need to provide an
interface to add (probe) and remove (release) the resource from the system.
This patch Creates new generic probe and release sysfs files to facilitate
cpu probe/release.  The probe/release interface provides for allowing each
arch to supply their own routines for implementing the backend of adding
and removing cpus to/from the system.

This also creates the powerpc specific stubs to handle the arch callouts 
from writes to the sysfs files.


The creation and use of these files is regulated by the 
CONFIG_ARCH_CPU_PROBE_RELEASE option so that only architectures that need the

capability will have the files created.

Signed-off-by: Nathan Fontenot nf...@austin.ibm.com
---

arch/powerpc/Kconfig   |4 
arch/powerpc/include/asm/machdep.h |5 +
arch/powerpc/kernel/sysfs.c|   19 +++
drivers/base/cpu.c |   32 
include/linux/cpu.h|2 ++
5 files changed, 62 insertions(+)

Index: powerpc/drivers/base/cpu.c
===
--- powerpc.orig/drivers/base/cpu.c 2009-11-25 04:06:10.0 -0600
+++ powerpc/drivers/base/cpu.c  2009-11-25 04:11:01.0 -0600
@@ -72,6 +72,38 @@
per_cpu(cpu_sys_devices, logical_cpu) = NULL;
return;
}
+
+#ifdef CONFIG_ARCH_CPU_PROBE_RELEASE
+static ssize_t cpu_probe_store(struct class *class, const char *buf,
+  size_t count)
+{
+   return arch_cpu_probe(buf, count);
+}
+
+static ssize_t cpu_release_store(struct class *class, const char *buf,
+size_t count)
+{
+   return arch_cpu_release(buf, count);
+}
+
+static CLASS_ATTR(probe, S_IWUSR, NULL, cpu_probe_store);
+static CLASS_ATTR(release, S_IWUSR, NULL, cpu_release_store);
+
+int __init cpu_probe_release_init(void)
+{
+   int rc;
+
+   rc = sysfs_create_file(cpu_sysdev_class.kset.kobj,
+  class_attr_probe.attr);
+   if (!rc)
+   rc = sysfs_create_file(cpu_sysdev_class.kset.kobj,
+  class_attr_release.attr);
+
+   return rc;
+}
+device_initcall(cpu_probe_release_init);
+#endif /* CONFIG_ARCH_CPU_PROBE_RELEASE */
+
#else /* ... !CONFIG_HOTPLUG_CPU */
static inline void register_cpu_control(struct cpu *cpu)
{
Index: powerpc/arch/powerpc/include/asm/machdep.h
===
--- powerpc.orig/arch/powerpc/include/asm/machdep.h 2009-11-25 
04:06:10.0 -0600
+++ powerpc/arch/powerpc/include/asm/machdep.h  2009-11-25 04:11:01.0 
-0600
@@ -266,6 +266,11 @@
void (*suspend_disable_irqs)(void);
void (*suspend_enable_irqs)(void);
#endif
+
+#ifdef CONFIG_ARCH_CPU_PROBE_RELEASE
+   ssize_t (*cpu_probe)(const char *, size_t);
+   ssize_t (*cpu_release)(const char *, size_t);
+#endif
};

extern void e500_idle(void);
Index: powerpc/arch/powerpc/kernel/sysfs.c
===
--- powerpc.orig/arch/powerpc/kernel/sysfs.c2009-11-25 04:06:10.0 
-0600
+++ powerpc/arch/powerpc/kernel/sysfs.c 2009-11-25 04:11:01.0 -0600
@@ -461,6 +461,25 @@

cacheinfo_cpu_offline(cpu);
}
+
+#ifdef CONFIG_ARCH_CPU_PROBE_RELEASE
+ssize_t arch_cpu_probe(const char *buf, size_t count)
+{
+   if (ppc_md.cpu_probe)
+   return ppc_md.cpu_probe(buf, count);
+
+   return -EINVAL;
+}
+
+ssize_t arch_cpu_release(const char *buf, size_t count)
+{
+   if (ppc_md.cpu_release)
+   return ppc_md.cpu_release(buf, count);
+
+   return -EINVAL;
+}
+#endif /* CONFIG_ARCH_CPU_PROBE_RELEASE */
+
#endif /* CONFIG_HOTPLUG_CPU */

static int __cpuinit sysfs_cpu_notify(struct notifier_block *self,
Index: powerpc/arch/powerpc/Kconfig
===
--- powerpc.orig/arch/powerpc/Kconfig   2009-11-25 04:06:10.0 -0600
+++ powerpc/arch/powerpc/Kconfig2009-11-25 04:11:01.0 -0600
@@ -320,6 +320,10 @@

  Say N if you are unsure.

+config ARCH_CPU_PROBE_RELEASE
+   def_bool y
+   depends on HOTPLUG_CPU
+
config ARCH_ENABLE_MEMORY_HOTPLUG
def_bool y

Index: powerpc/include/linux/cpu.h
===
--- powerpc.orig/include/linux/cpu.h2009-11-25 04:06:10.0 -0600
+++ powerpc/include/linux/cpu.h 2009-11-25 04:11:56.0 -0600
@@ -43,6 +43,8 @@

#ifdef CONFIG_HOTPLUG_CPU
extern void unregister_cpu(struct cpu *cpu);
+extern ssize_t arch_cpu_probe(const char *, size_t);
+extern ssize_t arch_cpu_release(const char *, size_t);
#endif
struct notifier_block;

___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev