From owner-freebsd-x11@FreeBSD.ORG Thu Oct 2 22:41:24 2008 Return-Path: Delivered-To: freebsd-x11@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 13D4410656B2 for ; Thu, 2 Oct 2008 22:41:24 +0000 (UTC) (envelope-from onemda@gmail.com) Received: from rv-out-0506.google.com (rv-out-0506.google.com [209.85.198.224]) by mx1.freebsd.org (Postfix) with ESMTP id D37508FC23 for ; Thu, 2 Oct 2008 22:41:23 +0000 (UTC) (envelope-from onemda@gmail.com) Received: by rv-out-0506.google.com with SMTP id b25so1185312rvf.43 for ; Thu, 02 Oct 2008 15:41:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; 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; bh=hU3wbeE/+75R/mZ8EwvLyQVeQu+x/FANIT2rImbagmk=; b=hzsZI3uur7VpKaclmBT4PpO0ZikvEaP6hD/1MoO1HmAg1eqqteXkAEJ9Fy7T0WzOJD ji7jvf478Pmcuau8F2Hd5km6W5nQqOK0O7VPBriTJpRrO2SykZqtHk/T5mXUg6SVj0cs Fm0a+GLaOrCx9F+5LmllBFQIVEv70mv1jn/A4= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:cc:in-reply-to:mime-version :content-type:content-transfer-encoding:content-disposition :references; b=TwGi+gjZtFwiZncUIo+eQr0223XbRHeLLhRPWBoawaR0cK4aUAwPBeyBnUS1Xof2kj f9j/+AI7UhFl11KSbpCt+9vZk9daWa7DIg+rngQ4wlGHR+avLyYr0otZi+dHEA5XiepT 2zEHp1+Mq3WIrEU+gm3nlt6IOI6xbpSsXobF8= Received: by 10.140.136.6 with SMTP id j6mr126368rvd.231.1222987283221; Thu, 02 Oct 2008 15:41:23 -0700 (PDT) Received: by 10.141.189.15 with HTTP; Thu, 2 Oct 2008 15:41:23 -0700 (PDT) Message-ID: <3a142e750810021541t622f5c7eoa9dc6be6eca34d8@mail.gmail.com> Date: Fri, 3 Oct 2008 00:41:23 +0200 From: "Paul B. Mahol" To: "Robert Noland" In-Reply-To: <1222984357.12046.48.camel@squirrel.corp.cox.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <1222923143.1684.58.camel@wombat.2hip.net> <3a142e750810021338m304e773mdef29261f655ba04@mail.gmail.com> <1222981799.12046.41.camel@squirrel.corp.cox.com> <3a142e750810021416s4f03d475u2620ed2d74d40d73@mail.gmail.com> <1222982990.12046.46.camel@squirrel.corp.cox.com> <3a142e750810021438x3243a313j131d1411181f4fe9@mail.gmail.com> <3a142e750810021447m7e0ba1e4v2d3d8bb676dac5c3@mail.gmail.com> <1222984357.12046.48.camel@squirrel.corp.cox.com> Cc: freebsd-x11 Subject: Re: latest drm patches X-BeenThere: freebsd-x11@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: X11 on FreeBSD -- maintaining and support List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 02 Oct 2008 22:41:24 -0000 On 10/2/08, Robert Noland wrote: > On Thu, 2008-10-02 at 23:47 +0200, Paul B. Mahol wrote: >> On 10/2/08, Paul B. Mahol wrote: >> > On 10/2/08, Robert Noland wrote: >> >> On Thu, 2008-10-02 at 23:16 +0200, Paul B. Mahol wrote: >> >>> On 10/2/08, Robert Noland wrote: >> >>> > On Thu, 2008-10-02 at 22:38 +0200, Paul B. Mahol wrote: >> >>> >> On 10/2/08, Robert Noland wrote: >> >>> >> > I have made new patch sets for both -CURRENT and -STABLE. They >> >>> >> > are >> >>> >> > located at: >> >>> >> > >> >>> >> > http://people.freebsd.org/~rnoland/drm-update-7-100108.patch.bz2 >> >>> >> > http://people.freebsd.org/~rnoland/drm-update-8-100108.patch.bz2 >> >>> >> > >> >>> >> > Note that if your are using RELENG_7, you will need to be very >> >>> >> > current. >> >>> >> > i.e. on or after 2008-09-29 16:20:13 -0400. CURRENT should be at >> >>> >> > least >> >>> >> > 2008-09-20 15:56:02 -0400. >> >>> >> > >> >>> >> > This is a re-sync to git master, which seems to address many of >> >>> >> > the >> >>> >> > issues with the intel chipsets < 965. I'm still not certain >> >>> >> > exactly >> >>> >> > which change fixes them unfortunately. This update contains a >> >>> >> > lot >> >>> >> > of >> >>> >> > code cleanup and is post gem merge (no, we don't have gem >> >>> >> > support). >> >>> >> > It >> >>> >> > should prove much easier to read the code now. A lot of thanks >> >>> >> > goes >> >>> >> > to >> >>> >> > vehemens for that work. I have adapted the code to use cdevpriv >> >>> >> > for >> >>> >> > tracking per open file data, which is the reason that you need >> >>> >> > really >> >>> >> > current bits to use this patch. That alleviates the old ugly >> >>> >> > hack >> >>> >> > that >> >>> >> > we used to try and accomplish the task and helped to clean up the >> >>> >> > open / >> >>> >> > close behavior a good bit. This also replaces the hack that was >> >>> >> > put >> >>> >> > in >> >>> >> > place a year or so ago to prevent radeons from locking up with >> >>> >> > AIGLX >> >>> >> > enabled. I have had a couple of radeon testers report that it >> >>> >> > still >> >>> >> > works as expected, though I no longer have radeon hardware to >> >>> >> > test >> >>> >> > with >> >>> >> > myself. Other various fixes from the linux crew and Intel, many >> >>> >> > of >> >>> >> > which are muddled in with the gem merge. >> >>> >> > >> >>> >> > I am planning to push this into CURRENT pretty soon, possibly >> >>> >> > even >> >>> >> > sometime tomorrow once I have a chance to discuss with a few >> >>> >> > others. >> >>> >> >> >>> >> Wow, something is broken again (glxgears doenst work again). I >> >>> >> checked >> >>> >> twice (with two versions of agp) >> >>> > >> >>> > Are you sure that your -CURRENT meets the requirements that I >> >>> > stated? >> >>> >> >>> FreeBSD dhcppc1 8.0-CURRENT FreeBSD 8.0-CURRENT #6: Sun Sep 28 >> >>> 11:46:55 CEST 2008 >> >> >> >> Hrm, ok... You say that git from yesterday is working. I pushed one of >> >> the 2 differences last night, so let's try this. >> >> >> >> cd into your git directory >> >> git pull >> >> cd bsd-core && make clean && make all install >> >> >> >> Hopefully that will tell us which one has the issue. >> > >> > OK: >> > >> >> git pull >> > remote: Counting objects: 7, done. >> > remote: Compressing objects: 100% (4/4), done. >> > remote: Total 4 (delta 3), reused 0 (delta 0) >> > Unpacking objects: 100% (4/4), done. >> > From git://anongit.freedesktop.org/git/mesa/drm >> > 073cb5e..81952c7 master -> origin/master >> > Updating 073cb5e..81952c7 >> > Fast forward >> > bsd-core/drm_vm.c | 21 ++++++++++++--------- >> > 1 files changed, 12 insertions(+), 9 deletions(-) >> > >> > Let we see would this change break something :) ... >> >> It did not, DRI works again. > > Sigh... ok, please try applying the following patch in REVERSE to your > kernel tree. > > http://people.freebsd.org/~rnoland/0001--FreeBSD-This-eliminates-a-potential-panic-but-can.patch Bingo, reversing that patch fixed problem. I reversed it on top of patch from this thread. I'm interested what comit really fixed drm module unloading.