[RFC] don't create cpu/online sysfs file
Rusty Russell
rusty at rustcorp.com.au
Mon Jun 7 09:08:18 EST 2004
On Sat, 2004-06-05 at 08:32, Dave Hansen wrote:
> On Fri, 2004-06-04 at 15:06, Joel Schopp wrote:
> > I might change cpu_is_hotpluggable to __cpu_is_hotpluggable to help
> > indicate it is a platform specific function.
>
> Yeah, that naming is a lot more consistent with the other functions in
> the arch-specific cpu hotplug code. New version attached.
The purpose of __ is to indicate a low-level function which requires
special knowledge to call (usually a lock). The cpu hotplug code is a
little odd, but way back when, it had cpu_disable() and cpu_die(), so
the arch-specific parts which actually did the work were called
__cpu_disable() and __cpu_die(). The names stuck.
There's nothing special required to call cpu_is_hotpluggable(), so it's
wrong to prepend __.
Sorry for the inconsistency,
Rusty.
--
Anyone who quotes me in their signature is an idiot -- Rusty Russell
** Sent via the linuxppc64-dev mail list. See http://lists.linuxppc.org/
More information about the Linuxppc64-dev
mailing list