From owner-freebsd-ppc@FreeBSD.ORG  Sun Jul  8 20:55:52 2012
Return-Path: <owner-freebsd-ppc@FreeBSD.ORG>
Delivered-To: freebsd-ppc@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id AF8C7106566C;
	Sun,  8 Jul 2012 20:55:52 +0000 (UTC)
	(envelope-from andreast-list@fgznet.ch)
Received: from smtp.fgznet.ch (mail.fgznet.ch [81.92.96.47])
	by mx1.freebsd.org (Postfix) with ESMTP id 466C28FC12;
	Sun,  8 Jul 2012 20:55:51 +0000 (UTC)
Received: from deuterium.andreas.nets (dhclient-91-190-14-19.flashcable.ch
	[91.190.14.19])
	by smtp.fgznet.ch (8.13.8/8.13.8/Submit_SMTPAUTH) with ESMTP id
	q68KthHL045277; Sun, 8 Jul 2012 22:55:44 +0200 (CEST)
	(envelope-from andreast-list@fgznet.ch)
Message-ID: <4FF9F3CF.6050608@fgznet.ch>
Date: Sun, 08 Jul 2012 22:55:43 +0200
From: Andreas Tobler <andreast-list@fgznet.ch>
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.5;
	rv:13.0) Gecko/20120601 Thunderbird/13.0
MIME-Version: 1.0
To: Alan Cox <alc@rice.edu>
References: <20120707102004.3e874201@narn.knownspace>
	<4FF87446.2090903@rice.edu>
In-Reply-To: <4FF87446.2090903@rice.edu>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
X-Scanned-By: MIMEDefang 2.64 on 81.92.96.47
Cc: alc@freebsd.org, Justin Hibbits <chmeeedalf@gmail.com>,
	freebsd-ppc@freebsd.org
Subject: Re: Panic with latest pmap lock changes.
X-BeenThere: freebsd-ppc@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Porting FreeBSD to the PowerPC <freebsd-ppc.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-ppc>,
	<mailto:freebsd-ppc-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-ppc>
List-Post: <mailto:freebsd-ppc@freebsd.org>
List-Help: <mailto:freebsd-ppc-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-ppc>,
	<mailto:freebsd-ppc-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 08 Jul 2012 20:55:52 -0000

On 07.07.12 19:39, Alan Cox wrote:
> On 07/07/2012 09:20, Justin Hibbits wrote:
>> Looks like I spoke too soon about the pmap lock changes working on my
>> G4.  After about 24 hours of uptime, it panicked with the following:
>>
>> _rw_wlock_hard: recursing but non-recursive rw pmap pv global
>> @ /home/chmeee/freebsd/src/sys/powerpc/aim/mmu_oea.c:2301
>>
>> I think the attached patch should fix it (Untested, except for
>> compiling).
>>
>
> Ugh.  Sorry.
>
> The attached patch eliminates the lock recursion.  While I was doing
> that, I noticed that the pmap_ts_referenced() implementations on powerpc
> have the wrong return type.  Oddly, the comments in mmu_if.h have the
> return type correct, but the code two or three lines later has it wrong.

Fyi, I'm building world with the patch mentioned in this thread and the 
kernel updated to 238258. (G5-SMP 32-bit)

So far it looks promising.

Before 238258 I got reliable machine locks/freeze w/o any idea what was 
happening.

If I reverted mmu_oea.c back to 238158, one before the commit from you 
Alan, I was able to get a successful full world/kernel build cycle.
Yes, I confirmed that it booted, but I was not able to run a full 
world/kernel cycle since I lost my GEOM_APM config ;).

Anyway, as said, it looks promising and it will take some hours to complete.

Thanks,
Andreas