lkml.org 
[lkml]   [2013]   [Nov]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 01/14] sched: add sched_class->task_dead.
On 11/12/2013 06:19 PM, Steven Rostedt wrote:
> On Thu, 7 Nov 2013 14:43:35 +0100
> Juri Lelli <juri.lelli@gmail.com> wrote:
>
>> From: Dario Faggioli <raistlin@linux.it>
>>
>> Add a new function to the scheduling class interface. It is called
>> at the end of a context switch, if the prev task is in TASK_DEAD state.
>>
>> It might be useful for the scheduling classes that want to be notified
>> when one of their task dies, e.g. to perform some cleanup actions.
>
> Nit. s/task/tasks/
>

Amended, thanks!

Best,

- Juri

>>
>> Signed-off-by: Dario Faggioli <raistlin@linux.it>
>> Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
>> ---
>> kernel/sched/core.c | 3 +++
>> kernel/sched/sched.h | 1 +
>> 2 files changed, 4 insertions(+)
>>
>> diff --git a/kernel/sched/core.c b/kernel/sched/core.c
>> index 5ac63c9..850a02c 100644
>> --- a/kernel/sched/core.c
>> +++ b/kernel/sched/core.c
>> @@ -1890,6 +1890,9 @@ static void finish_task_switch(struct rq *rq, struct task_struct *prev)
>> if (mm)
>> mmdrop(mm);
>> if (unlikely(prev_state == TASK_DEAD)) {
>> + if (prev->sched_class->task_dead)
>> + prev->sched_class->task_dead(prev);
>> +
>> /*
>> * Remove function-return probe instances associated with this
>> * task and put them back on the free list.
>> diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h
>> index b3c5653..64eda5c 100644
>> --- a/kernel/sched/sched.h
>> +++ b/kernel/sched/sched.h
>> @@ -992,6 +992,7 @@ struct sched_class {
>> void (*set_curr_task) (struct rq *rq);
>> void (*task_tick) (struct rq *rq, struct task_struct *p, int queued);
>> void (*task_fork) (struct task_struct *p);
>> + void (*task_dead) (struct task_struct *p);
>>
>> void (*switched_from) (struct rq *this_rq, struct task_struct *task);
>> void (*switched_to) (struct rq *this_rq, struct task_struct *task);
>


\
 
 \ /
  Last update: 2013-11-12 20:41    [W:0.152 / U:0.076 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site