Re: [PATCH 01/27] smpboot: Provide a generic method to boot secondary processors

From
Srivatsa S. Bhat <hidden>
Date
2012-06-01 22:42:24
Also in
lkml
Reply to
Sam Ravnborg, 2012-06-01
On 06/01/2012 10:23 PM, Sam Ravnborg wrote:
quoted
+
+
+/* Implement the following functions in your architecture, as appropriate. */
+
+/**
+ * __cpu_pre_starting()
+ *
+ * Implement whatever you need to do before the CPU_STARTING notifiers are
+ * invoked. Note that the CPU_STARTING callbacks run *on* the cpu that is
+ * coming up. So that cpu better be prepared! IOW, implement all the early
+ * boot/init code for the cpu here. And do NOT enable interrupts.
+ */
+#ifndef __cpu_pre_starting
+void __weak __cpu_pre_starting(void *arg) {}
+#endif
I miss the prototype for this in a header?

Prototype is not really necessary for this. Hence not added.
And the comment maybe belong in the header - not in the implementation?

I think having the comment near the implementation itself works better, just
like how it is done at other places in the kernel.

Regards,
Srivatsa S. Bhat
Thread (81 messages) 81 messages, 12 authors, last reply 2012-06-05
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help