From owner-freebsd-current@FreeBSD.ORG Tue Aug 15 20:48:25 2006 Return-Path: X-Original-To: current@freebsd.org Delivered-To: freebsd-current@FreeBSD.ORG Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 6BB6016A4DF; Tue, 15 Aug 2006 20:48:25 +0000 (UTC) (envelope-from xdivac02@stud.fit.vutbr.cz) Received: from eva.fit.vutbr.cz (eva.fit.vutbr.cz [147.229.10.14]) by mx1.FreeBSD.org (Postfix) with ESMTP id B9FB043D49; Tue, 15 Aug 2006 20:48:23 +0000 (GMT) (envelope-from xdivac02@stud.fit.vutbr.cz) Received: from eva.fit.vutbr.cz (localhost [127.0.0.1]) by eva.fit.vutbr.cz (envelope-from xdivac02@eva.fit.vutbr.cz) (8.13.7/8.13.7) with ESMTP id k7FKmHQr076941 (version=TLSv1/SSLv3 cipher=EDH-RSA-DES-CBC3-SHA bits=168 verify=NO); Tue, 15 Aug 2006 22:48:17 +0200 (CEST) Received: (from xdivac02@localhost) by eva.fit.vutbr.cz (8.13.7/8.13.3/Submit) id k7FKmHiH076940; Tue, 15 Aug 2006 22:48:17 +0200 (CEST) Date: Tue, 15 Aug 2006 22:48:17 +0200 From: Divacky Roman To: Kip Macy Message-ID: <20060815204817.GA75819@stud.fit.vutbr.cz> References: <20060815141151.15ae4349@Magellan.Leidinger.net> <44E1BD03.2030402@FreeBSD.org> <20060815144625.362bf376@Magellan.Leidinger.net> <44E1C3E4.7080508@FreeBSD.org> <20060815153451.604d16f1@Magellan.Leidinger.net> <20060815130418.I18116@demos.bsdclusters.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20060815130418.I18116@demos.bsdclusters.com> User-Agent: Mutt/1.4.2i X-Scanned-By: MIMEDefang 2.54 on 147.229.10.14 Cc: Alexander Leidinger , Suleiman Souhlal , current@freebsd.org Subject: Re: HEADS-UP: starting to commit linuxolator (SoC 2006) changes... X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 15 Aug 2006 20:48:25 -0000 On Tue, Aug 15, 2006 at 01:04:52PM -0700, Kip Macy wrote: > What does testing buy us when you've already admitted that futexes et > al. are broken? the futexes are not 100% working which is different from broken. it passes my pthread tests and its able to run quite complicated programs (firefox, opera) the problem is (for example) with realplay which hangs waiting on a futex when I attempt to play a file. I noticed that this behaviour changes based upon what pid I copy out to the parent thread (when I copy linux-tid ff/opera works and realplay not, when I copy linux-pid ff is broken, realplay works). if anyone has an idea what can cause this feel free to tell me. the point is - other people might see what the bug is. I dont see it. what commiting this code into src means? 1) more eyeballs to review 2) coverity run - I strongly suggest extending coverity to run on some p4 branches 3) some panics etc. might be revealed 4) the commit is almost 100% self-contained and doesnt break anything while I agree that Alexander could wait 2-3 days in general case I also know his reasons why he commited it so fast - his vacation ends and he wanted to devote today to integrating this stuff into the cvs. On the other hand I've seen too many patches which got never integrated that I am glad this one is different. my 2 cents Roman Divacky (the author of the patch in question)