lkml.org 
[lkml]   [2005]   [Aug]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] Re: 2.6.12: itimer_real timers don't survive execve() any more
Andrew Morton wrote:
> Roland McGrath <roland@redhat.com> wrote:
>
>>That's wrong. It has to be done only by the last thread in the group to go.
>>Just revert Ingo's change.
>>
Hm... I was looking at 2.6.10 to figure it out. This looks more correct.

>
>
> OK..
>
> --- 25/kernel/exit.c~revert-timer-exit-cleanup Thu Aug 4 15:00:55 2005
> +++ 25-akpm/kernel/exit.c Thu Aug 4 15:01:06 2005
> @@ -829,8 +829,10 @@ fastcall NORET_TYPE void do_exit(long co
> acct_update_integrals(tsk);
> update_mem_hiwater(tsk);
> group_dead = atomic_dec_and_test(&tsk->signal->live);
> - if (group_dead)
> + if (group_dead) {
> + del_timer_sync(&tsk->signal->real_timer);
> acct_process(code);
> + }
> exit_mm(tsk);
>
> exit_sem(tsk);
> diff -puN kernel/posix-timers.c~revert-timer-exit-cleanup kernel/posix-timers.c
> --- 25/kernel/posix-timers.c~revert-timer-exit-cleanup Thu Aug 4 15:00:55 2005
> +++ 25-akpm/kernel/posix-timers.c Thu Aug 4 15:01:06 2005
> @@ -1166,7 +1166,6 @@ void exit_itimers(struct signal_struct *
> tmr = list_entry(sig->posix_timers.next, struct k_itimer, list);
> itimer_delete(tmr);
> }
> - del_timer_sync(&sig->real_timer);
> }
>
> /*
> _
>
> -
> 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/
>

--
George Anzinger george@mvista.com
HRT (High-res-timers): http://sourceforge.net/projects/high-res-timers/
-
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/

\
 
 \ /
  Last update: 2005-08-05 00:21    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site