From owner-cvs-all@FreeBSD.ORG Mon Jul 23 21:20:20 2007 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 5A2C016A420 for ; Mon, 23 Jul 2007 21:20:20 +0000 (UTC) (envelope-from mureninc@gmail.com) Received: from wx-out-0506.google.com (wx-out-0506.google.com [66.249.82.227]) by mx1.freebsd.org (Postfix) with ESMTP id 0F60F13C467 for ; Mon, 23 Jul 2007 21:20:19 +0000 (UTC) (envelope-from mureninc@gmail.com) Received: by wx-out-0506.google.com with SMTP id i29so1449698wxd for ; Mon, 23 Jul 2007 14:20:19 -0700 (PDT) DKIM-Signature: a=rsa-sha1; c=relaxed/relaxed; d=gmail.com; s=beta; h=domainkey-signature:received:received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=Bc94jlN93S2xatktho0QPiWCqK0dYBZU2fpS2BjCV0O5XnBRKX5gkmb9fFTegGqbtNi8bEcEAvtTDTn4OXl13+HUGG3Kh/YG4XaqUfp3coPu9tsxKjy6VlhKmx6Vd0IiBYmln2sungVmcEZ7EWquu8PvL6qbL44IuGYp1Ox4HbA= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=A5uJrkcVV253SQmpNyP+Nb/bo5z6zXODXnnLSCmw25NxxvAEzAuNC+/7ZhFsdlKIC3DjyP2JeCPfaXeePZJcaZI0LlrJ7ioX6CDnAywTtHk4E3nuT/fk4ABTZ7r2LldOxjQrbkZYu4v0Jx5v29lV1ikg3tqFGAr54lXxt2lYxvc= Received: by 10.90.97.11 with SMTP id u11mr2579141agb.1185225618431; Mon, 23 Jul 2007 14:20:18 -0700 (PDT) Received: by 10.90.78.10 with HTTP; Mon, 23 Jul 2007 14:20:18 -0700 (PDT) Message-ID: Date: Mon, 23 Jul 2007 17:20:18 -0400 From: "Constantine A. Murenin" To: "Doug Barton" In-Reply-To: <46A515B9.6060608@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <200707231452.l6NEqMot074554@repoman.freebsd.org> <46A4F893.20609@FreeBSD.org> <46A515B9.6060608@FreeBSD.org> Cc: Marcus Alves Grando , Attilio Rao , src-committers@freebsd.org, cvs-all@freebsd.org, cvs-src@freebsd.org Subject: Re: cvs commit: src/sys/kern kern_kse.c kern_thread.c src/sys/sys proc.h X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 23 Jul 2007 21:20:20 -0000 On 23/07/07, Doug Barton wrote: > Marcus Alves Grando wrote: > > Attilio Rao wrote: > >> attilio 2007-07-23 14:52:22 UTC > >> > >> FreeBSD src repository > >> > >> Modified files: > >> sys/kern kern_kse.c kern_thread.c > >> sys/sys proc.h Log: > >> Actually, KSE kernel bits locking is broken and can lead likely to > >> dangerous races. > >> Fix this problems adding correct locking for the members of 'struct > >> kse_upcall' and other struct proc/struct thread related members. > >> For the moment, just leave ku_mflag and ku_flags "lazy" locked. > >> While here, cleanup the code removing the function kse_GC() (unused), > >> and merging upcall_link(), upcall_unlink(), upcall_stash() in their > >> respective callers (static functions, very short and only called in one > >> place). > >> Reported by: pav > >> Tested by: pav (on some pointyhat cluster nodes) > >> Approved by: jeff > >> Approved by: re > >> Sponsorized by: NGX Italy (http://www.ngx.it) > >> Revision Changes Path > >> 1.232 +69 -82 src/sys/kern/kern_kse.c > >> 1.251 +2 -0 src/sys/kern/kern_thread.c > >> 1.488 +0 -1 src/sys/sys/proc.h > >> _______________________________________________ > >> cvs-src@freebsd.org mailing list > >> http://lists.freebsd.org/mailman/listinfo/cvs-src > >> To unsubscribe, send any mail to "cvs-src-unsubscribe@freebsd.org" > >> > >> ------------------------------------------------------ > >> http://www.FreeBSD.org/cgi/cvsweb.cgi/src/sys/kern/kern_kse.c.diff?&r1=1.231&r2=1.232 > >> > >> http://www.FreeBSD.org/cgi/cvsweb.cgi/src/sys/kern/kern_thread.c.diff?&r1=1.250&r2=1.251 > >> > >> http://www.FreeBSD.org/cgi/cvsweb.cgi/src/sys/sys/proc.h.diff?&r1=1.487&r2=1.488 > >> > >> > > > > Wno-pointer-sign -fformat-extensions -nostdinc -I. -I/usr/src/sys > > -I/usr/src/sys/contrib/altq -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS > > -include opt_global.h -fno-common -finline-limit=8000 --param > > inline-unit-growth=100 --param large-function-growth=1000 > > -mno-align-long-strings -mpreferred-stack-boundary=2 -mno-mmx > > -mno-3dnow -mno-sse -mno-sse2 -mno-sse3 -ffreestanding -Werror > > /usr/src/sys/kern/kern_kse.c > > cc1: warnings being treated as errors > > /usr/src/sys/kern/kern_kse.c:87: warning: 'upcall_free' defined but not > > used > > *** Error code 1 > > Me Too. Removing the prototype and the function definition got the > kernel to compile, but I'm a little scared to boot it. :) Apparently, it should be enough to remove the "static" keyword from the prototype, in case you want a quick-and-dirty fix. :) After completely removing the prototype and the function definition, kernel does compile and boot without any problems. Cheers, Constantine.