PERFORCE change 108941 for review

John Baldwin jhb at freebsd.org
Wed Nov 1 23:05:39 UTC 2006


On Wednesday 01 November 2006 14:58, Roman Divacky wrote:
> http://perforce.freebsd.org/chv.cgi?CH=108941
> 
> Change 108941 by rdivacky at rdivacky_witten on 2006/11/01 19:57:42
> 
> 	Lock proc in gettid.
> 
> Affected files ...
> 
> .. //depot/projects/linuxolator/src/sys/compat/linux/linux_misc.c#22 edit
> 
> Differences ...
> 
> ==== //depot/projects/linuxolator/src/sys/compat/linux/linux_misc.c#22 
(text+ko) ====
> 
> @@ -1408,7 +1408,9 @@
>  		printf(ARGS(gettid, ""));
>  #endif
>  
> +	PROC_LOCK(td->td_proc);
>  	td->td_retval[0] = td->td_proc->p_pid;
> +	PROC_UNLOCK(td->td_proc);
>  	return (0);
>  }

You don't need this.  p_pid is read-only after fork().

-- 
John Baldwin


More information about the p4-projects mailing list