Messages in this thread | | | Subject | Re: [PATCH] SMP signal latency fix up. | From | Mark Gross <> | Date | 07 Nov 2003 07:43:10 -0800 |
| |
On Fri, 2003-11-07 at 01:45, Ingo Molnar wrote: > On Fri, 6 Nov 2003, Mark Gross wrote: > > > } > > - success = 1; > > } > > -#ifdef CONFIG_SMP > > - else > > - if (unlikely(kick) && task_running(rq, p) && (task_cpu(p) != smp_processor_id())) > > - smp_send_reschedule(task_cpu(p)); > > -#endif > > + success = 1; > > hm, this i believe is incorrect - you've moved the 'success' case outside > of the 'real wakeup' branch. >
Yup, I was confusing myself a bit on the return symantics of try_to_wake_up, and the relationship with race between changing task->state and scheduling a task off a cpu (the "array" test while holding the rq lock.).
The feeling that this was likely wrong was eating at me all evening and then it came to me around 8pm when I was driving my son to some thing.
> to avoid races, we only want to report success if the thread has been > truly placed on the runqueue by this call. The other case (eg. changing > TASK_INTERRUPTIBLE to TASK_RUNNING) does not count as a 'wakeup'. Note > that if the task was in a non-TASK_RUNNING state then we dont have to kick > the process anyway because it's in kernel-mode and will go through the > signal return path soon. > > Ingo
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |