From owner-freebsd-current@FreeBSD.ORG Sun Nov 25 17:24:02 2007 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id BF83616A417 for ; Sun, 25 Nov 2007 17:24:02 +0000 (UTC) (envelope-from vovkasm@gmail.com) Received: from rv-out-0910.google.com (rv-out-0910.google.com [209.85.198.188]) by mx1.freebsd.org (Postfix) with ESMTP id 914BB13C458 for ; Sun, 25 Nov 2007 17:24:02 +0000 (UTC) (envelope-from vovkasm@gmail.com) Received: by rv-out-0910.google.com with SMTP id l15so349748rvb for ; Sun, 25 Nov 2007 09:24:01 -0800 (PST) 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=EWz1dQXWA0yJ8PSGxDRsCswhXKG8bgwTWXoi9t/0rQI=; b=v81hsWe65VzL64vxxu7k6vJd+vLPIXvNawSqb8lwXcdpBrlMW3JO864fBEF5sk9Mssc0Xaj/GPURLiTytMRwVnNKuDw1OVuoxTA1QSHh/NCO+DsLS0bMyv3AFDkfDZXpcG5MpyF/s7OEjYNGttCrelRa+8FyEPs1zCfs/oXD69M= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=Ek3RcrbCcO/tjdEdZZLW6aCUi3eY16qXAPG+YH4IERhTMNBJFGbf8Sb8blXZUztUeHUoEk29LqlzkmaVjM4kH1qM50Wgw0YC9fmJH6pkCU8FpbRJ9Wy5zYGrhAR58H4SBHg4+zBBunsJal0y7T+0d5d0TqxWR+bLIHdTpzUkYIA= Received: by 10.141.74.17 with SMTP id b17mr695094rvl.1196011441443; Sun, 25 Nov 2007 09:24:01 -0800 (PST) Received: by 10.141.40.1 with HTTP; Sun, 25 Nov 2007 09:24:01 -0800 (PST) Message-ID: Date: Sun, 25 Nov 2007 20:24:01 +0300 From: "Vladimir Timofeev" To: "Lionel Flandrin" In-Reply-To: <86fxyuuzew.fsf@simias.hd.free.fr> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <86lk9fetff.fsf@simias.hd.free.fr> <20071103160137.GA1182@hurx.thc> <86oddi3h0o.fsf@simias.hd.free.fr> <86fxyuuzew.fsf@simias.hd.free.fr> Cc: Eric Anholt , freebsd-current@freebsd.org Subject: Re: X crash with radeon driver - AGP broken? 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: Sun, 25 Nov 2007 17:24:02 -0000 On Nov 25, 2007 6:18 PM, Lionel Flandrin wrote: > "Vladimir Timofeev" writes: > > > Hi, if u have nforce2 chipset on motheboard side, > > I do > > > try this patch... > > with this I can run X, but not test more... > > > > How to patch: > > cd /usr/src > > patch < latest-agp_nvidia.c.diff > > > > my conf: Abit NF7 motheboard, Radeon 9200 video, fbsd 7.0-BETA3, X > > Server 1.4.0 (from /var/log/Xorg.0.log) > > > > I have a zenith chaintech motherboard and a radeon 9600 pro, same > fbsd/X version, and indeed, it works, thanks a lot! I can finally use my > two screens again! > > I played with xrandr, glxgears and some 3d apps (blender for instance) > and it seems to work perfectly. Will this patch be commited to the main > tree ? May be, if this works and doing right things ;-) After some code cleanup and revision... I'am not commiter anything... may be Eric Anholt? > > Also, I'm curious, what was the problem in the first place ? I must > say I have no clue what this patch is doing (excepted it doesn't seem > too nasty :). Story of this patch is short. I take this kernel crash after upgrade to current (now releng 7). May be gcc4 change 'volatile' behaviour or anything else... I start studing code differences in linux and fbsd agp drivers code. In this patch for agp_nvidia_flush_tlb I realize two things: 1. get out volatile temporary variable to switch off potentialy compiler optimizations (in linux code this variable absent) 2. ensure that we read no more than allocated entries in agp gatt structure (in linux and current realization this read 33(magic?) entries) But it only my guesses. (I'am not c programmer, only know syntax ;-) > > Thanks again, > -- > Lionel >