CPU/hotplug: Clarify description of __cpuhp_setup_state() return value
authorBoris Ostrovsky <boris.ostrovsky@oracle.com>
Thu, 15 Dec 2016 15:00:57 +0000 (10:00 -0500)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 21 Dec 2016 01:52:51 +0000 (02:52 +0100)
When ivoked with CPUHP_AP_ONLINE_DYN state __cpuhp_setup_state()
is expected to return positive value which is the hotplug state that
the routine assigns.

Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
kernel/cpu.c

index 29de1a9352c005c0d4808d2f842c15d29df1b2f8..1fca8b716ec1059cf4a8df5c6f420a43c6a79003 100644 (file)
@@ -1586,7 +1586,11 @@ EXPORT_SYMBOL_GPL(__cpuhp_state_add_instance);
  * @startup:   startup callback function
  * @teardown:  teardown callback function
  *
- * Returns 0 if successful, otherwise a proper error code
+ * Returns:
+ *   On success:
+ *      Positive state number if @state is CPUHP_AP_ONLINE_DYN
+ *      0 for all other states
+ *   On failure: proper (negative) error code
  */
 int __cpuhp_setup_state(enum cpuhp_state state,
                        const char *name, bool invoke,