> Yes, this is correct. However it had me looking at that code and pretty > much everything else is completely wrong :-) > > That is, its functionally correct (probably), but the function name is > not descriptive of what the function does and the comment is just plain > wrong. > > Also, since both functions are only used in core.c we should probably > move them there. I'm not sure I understood it completely. What do you mean for wrong? Will CONFIG_SMP a meaningless check here? How about moving 'prepare_lock_switch' code from sched.h to prepare_task_switch in core.c? And about the comment in 'prepare_lock_switch', I can replace it to "Set on_cpu to 1 during the context switch will lock the processes on the cpu" > Do you think you can fix all that as well? Yeah absolutely, I just might need a few more comprehension on it. -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
