Date: Fri, 18 Jan 2008 14:34:57 +0100 From: "Attilio Rao" <attilio@freebsd.org> To: "David Xu" <davidxu@freebsd.org> Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/kern kern_time.c Message-ID: <3bbf2fe10801180534y7f97f39meb0b12624e0b919a@mail.gmail.com> In-Reply-To: <200801181300.m0ID0SMi041767@repoman.freebsd.org> References: <200801181300.m0ID0SMi041767@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
2008/1/18, David Xu <davidxu@freebsd.org>: > davidxu 2008-01-18 13:00:28 UTC > > FreeBSD src repository > > Modified files: > sys/kern kern_time.c > Log: > Make sure reading td_runtime in critical section since thread may be > preempted and td_runtime will be modified. If you don't need the operation to be atomical with other operations inside the critical path (and I don't expect so) this change is pointless as our reading are always atomic. Attilio -- Peace can only be achieved by understanding - A. Einstein
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3bbf2fe10801180534y7f97f39meb0b12624e0b919a>