Date: Sat, 17 Mar 2007 11:09:11 -0800 (PST) From: Jeff Roberson <jroberson@chesapeake.net> To: Jeff Roberson <jeff@FreeBSD.org> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern sched_ule.c Message-ID: <20070317110821.I560@10.0.0.1> In-Reply-To: <200703171813.l2HIDXWI008212@repoman.freebsd.org> References: <200703171813.l2HIDXWI008212@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Any language lawyers care to comment on this? On Sat, 17 Mar 2007, Jeff Roberson wrote: > jeff 2007-03-17 18:13:33 UTC > > FreeBSD src repository > > Modified files: > sys/kern sched_ule.c > Log: > - Cast the intermediate value in priority computtion back down to > unsigned char. Weirdly, casting the 1 constant to u_char still produces > a signed integer result that is then used in the % computation. This > avoids that mess all together and causes a 0 pri to turn into 255 % 64 > as we expect. > > Reported by: kkenn (about 4 times, thanks) > > Revision Changes Path > 1.190 +1 -1 src/sys/kern/sched_ule.c >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20070317110821.I560>