diff options
| author | Ingo Molnar <mingo@elte.hu> | 2002-08-15 02:48:18 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@home.transmeta.com> | 2002-08-15 02:48:18 -0700 |
| commit | 496084cb8bb78efc5ecc1dca593386d46bfe994b (patch) | |
| tree | 097d6d0e636bf2fbbacb31e177f2d1af43d2a539 /kernel/fork.c | |
| parent | eb2e58fd0b547556d4eb4fa29d575d85d2be281f (diff) | |
[PATCH] thread management - take three
you have applied my independent-pointer patch already, but i think your
CLEARTID variant is the most elegant solution: it reuses a clone argument,
thus reduces the number of arguments and it's also a nice conceptual pair
to the existing SETTID call. And the TID field can be used as a 'usage'
field as well, because the TID (PID) can never be 0, reducing the number
of fields in the TCB. And we can change the userspace locking code to use
the TID field no problem.
Diffstat (limited to 'kernel/fork.c')
| -rw-r--r-- | kernel/fork.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/fork.c b/kernel/fork.c index 65d6d40073d9..d577f3077e73 100644 --- a/kernel/fork.c +++ b/kernel/fork.c @@ -370,12 +370,12 @@ void mm_release(void) tsk->vfork_done = NULL; complete(vfork_done); } - if (tsk->user_vm_lock) + if (tsk->user_tid) /* * We dont check the error code - if userspace has * not set up a proper pointer then tough luck. */ - put_user(0UL, tsk->user_vm_lock); + put_user(0UL, tsk->user_tid); } static int copy_mm(unsigned long clone_flags, struct task_struct * tsk) |