Re: multi-threaded app fails to restart

[Oren Laadan <orenl-eQaUEPhvms7ENvBUuze7eA@public.gmane.org>]

Great.
Pushed fixes to ckpt-v22-dev.

Oren.

On 07/26/2010 01:56 PM, John Paul Walters wrote:
> It works for me as well.  Thanks for your help Oren.
> 
> JP
> 
> 
> 
> On Mon, Jul 26, 2010 at 1:11 PM, Dan Smith <danms-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org> wrote:
>> OL> diff --git a/kernel/checkpoint/sys.c b/kernel/checkpoint/sys.c
>> OL> index 171c867..c5517c2 100644
>> OL> --- a/kernel/checkpoint/sys.c
>> OL> +++ b/kernel/checkpoint/sys.c
>> OL> @@ -625,8 +625,11 @@ int walk_task_subtree(struct task_struct *root,
>> OL>             }
>>
>> OL>             /* if we arrive at root again -- done */
>> OL> -           if (task == root)
>> OL> -                   break;
>> OL> +           if (task == root) {
>> OL> +                   /* if not last thread - proceed with thread */
>> OL> +                   task = root = next_thread(task);
>> OL> +                   if (thread_group_leader(task))
>> OL> +                           break;
>>
>>                } // Need to close this block
>>
>> Otherwise it seems to work for me:
>>
>> Tested-by: Dan Smith <danms-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
>>
>> --
>> Dan Smith
>> IBM Linux Technology Center
>> email: danms-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org
>>
>