Date: Mon, 9 Mar 2009 13:12:48 +0000 (UTC) From: Robert Watson <rwatson@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r189573 - head/sys/sys Message-ID: <200903091312.n29DCmKw027132@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: rwatson Date: Mon Mar 9 13:12:48 2009 New Revision: 189573 URL: http://svn.freebsd.org/changeset/base/189573 Log: Use a u_int for p_lock instead of a char: this avoids a (somewhat unlikely but not impossible given modern thread counts) wrap-around, and the compiler was padding it out to an int (at least) anyway. MFC after: 3 days (but confirm ABI impact) Modified: head/sys/sys/proc.h Modified: head/sys/sys/proc.h ============================================================================== --- head/sys/sys/proc.h Mon Mar 9 13:11:16 2009 (r189572) +++ head/sys/sys/proc.h Mon Mar 9 13:12:48 2009 (r189573) @@ -491,7 +491,7 @@ struct proc { struct vnode *p_tracevp; /* (c + o) Trace to vnode. */ struct ucred *p_tracecred; /* (o) Credentials to trace with. */ struct vnode *p_textvp; /* (b) Vnode of executable. */ - char p_lock; /* (c) Proclock (prevent swap) count. */ + u_int p_lock; /* (c) Proclock (prevent swap) count. */ struct sigiolst p_sigiolst; /* (c) List of sigio sources. */ int p_sigparent; /* (c) Signal to parent on exit. */ int p_sig; /* (n) For core dump/debugger XXX. */
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200903091312.n29DCmKw027132>