OpenVZ Forum


Home » Mailing lists » Devel » [RFC][PATCH] Cleanup the new thread's creation
Re: [RFC][PATCH] Cleanup the new thread's creation [message #19782 is a reply to message #19764] Sat, 25 August 2007 16:50 Go to previous messageGo to previous message
Oleg Nesterov is currently offline  Oleg Nesterov
Messages: 143
Registered: August 2006
Senior Member
On 08/24, Pavel Emelyanov wrote:
>
> The major differences of creating a new thread from creating a
> new process is that
>
> 1. newbie's tgid is set to leader's
> 2. newbie's leader is set to leader
> 3. newbie is added to leader's thread_list

(Surely, the are many other major differences, but from the pids virtualization
 POV - yes ;)

> +static void setup_new_thread(struct task_struct *thr, struct task_struct 
> *leader)
> +{
> +	thr->tgid = leader->tgid;
> +	thr->group_leader = leader;
> +	list_add_tail_rcu(&thr->thread_group, &leader->thread_group);
> +}

Imho, this name is a bit "too generic". Not that I can suggest something
better... copy_sub_thread/copy_group_leader ?

> @@ -1147,9 +1161,6 @@ static struct task_struct *copy_process(
> 	}
> 
> 	p->pid = pid_nr(pid);
> -	p->tgid = p->pid;
> -	if (clone_flags & CLONE_THREAD)
> -		p->tgid = current->tgid;

I agree, it is absoulutely not clear why should we set ->tgid here, and it
would be nice to consolidate "if (CLONE_THREAD)" checks, but do we really
need the helpers above? There are very simple, and have the only one caller.
Sometimes it is good to see what's going on without pressing C-]

Not that I against this patch, just I'm not sure it really simplifies things.
Perhaps I missed something else you have in mind.

Oleg.

_______________________________________________
Containers mailing list
Containers@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers
 
Read Message
Read Message
Read Message
Previous Topic: [PATCH] monbean v0.5
Next Topic: [PATCH] Use helpers to obtain task pid in printks (arch code)
Goto Forum:
  


Current Time: Mon Oct 20 06:39:53 GMT 2025

Total time taken to generate the page: 0.46525 seconds