Date: Tue, 16 Oct 2007 17:45:01 -0400 From: John Baldwin <jhb@freebsd.org> To: Olivier Houchard <cognet@ci0.org> Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/sys proc.h Message-ID: <200710161745.02448.jhb@freebsd.org> In-Reply-To: <20071016221903.GA75176@ci0.org> References: <200710161957.l9GJvaFr075367@repoman.freebsd.org> <200710161637.36950.jhb@freebsd.org> <20071016221903.GA75176@ci0.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tuesday 16 October 2007 06:19:03 pm Olivier Houchard wrote: > On Tue, Oct 16, 2007 at 04:37:36PM -0400, John Baldwin wrote: > > On Tuesday 16 October 2007 03:57:36 pm Olivier Houchard wrote: > > > cognet 2007-10-16 19:57:36 UTC > > > > > > FreeBSD src repository > > > > > > Modified files: (Branch: RELENG_6) > > > sys/sys proc.h > > > Log: > > > Make td_sqqueue explicitely signed, as we compare it with >= 0. On arm, > > > char defaults to unsigned. > > > > It probably should really be unsigned like td_tsqueue (serves the same > > purpose) FWIW, however it is currently an 'int' in HEAD. > > > > Probably changing td_sqqueue to u_char in HEAD, 7.x and 6.x would be the best > > fix. > > I didn't investigate much. I saw it was an int on -CURRENT, and this test : > MPASS(td->td_sqqueue < NR_SLEEPQS && td->td_sqqueue >= 0); > If td_squeue can be unsigned, the test is wrong. Yeah, it might be. It's meant to be an array index, so negative values really are meaningless. I'll fix it in HEAD if I get some time. -- John Baldwin
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200710161745.02448.jhb>