From owner-freebsd-mips@FreeBSD.ORG Wed Jun 30 17:08:26 2010 Return-Path: Delivered-To: freebsd-mips@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 6919B106566C; Wed, 30 Jun 2010 17:08:26 +0000 (UTC) (envelope-from lists.br@gmail.com) Received: from mail-gy0-f182.google.com (mail-gy0-f182.google.com [209.85.160.182]) by mx1.freebsd.org (Postfix) with ESMTP id E99E28FC19; Wed, 30 Jun 2010 17:08:25 +0000 (UTC) Received: by gyf3 with SMTP id 3so709551gyf.13 for ; Wed, 30 Jun 2010 10:08:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:subject:mime-version :content-type:from:in-reply-to:date:cc:content-transfer-encoding :message-id:references:to:x-mailer; bh=rAYiZjSqZwq25TYYFcR65zcrSVzX9aHdMz6o5/QcLGs=; b=Bpj2ulQQ9AMKlAYVoS2PPkDzDCGNVW4zi+IH60heui0F05LhnGAb2aHLUkyPn5X+ga W4qhTdDcvAsnqdjYxvOy8U4JenCWiVSUrRza1MFzVvtLPevI6eh1p2G9GnLalsRfpPMN q7nJDFDoKnyJ0u8DxmqfMgTIfZiIDKnMTS6xk= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=subject:mime-version:content-type:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to:x-mailer; b=tOgMkTLvrzsCFGFlBe3m/YEJJJyTal7LCMoWGQr8ZW7cTJSyLCzSO1BBxIm/SmK9yq U8vb6hdGPUB+b3mHkevUUkxEM0wDPO7to+2ezb7DUeOjFu67DmH2rh2GzD2QCZI93DDF wt6UpWtcdP4stKgvcNJXZKjmCANaRk3X/9uJE= Received: by 10.229.241.207 with SMTP id lf15mr5339886qcb.134.1277917699884; Wed, 30 Jun 2010 10:08:19 -0700 (PDT) Received: from [192.168.0.86] ([187.39.15.144]) by mx.google.com with ESMTPS id c35sm28493693qco.0.2010.06.30.10.08.16 (version=TLSv1/SSLv3 cipher=RC4-MD5); Wed, 30 Jun 2010 10:08:18 -0700 (PDT) Mime-Version: 1.0 (Apple Message framework v1081) Content-Type: text/plain; charset=us-ascii From: Luiz Otavio O Souza In-Reply-To: Date: Wed, 30 Jun 2010 14:08:13 -0300 Content-Transfer-Encoding: quoted-printable Message-Id: References: <897604F6-95C4-49A8-B11F-277A74C8DBAE@gmail.com> <3C0AEF9B-AE0C-4459-A4E1-2C8C30C10FD6@gmail.com> <25B9A19D-0A6B-4731-8FB1-A2C6722F0E9C@gmail.com> To: Jayachandran C. X-Mailer: Apple Mail (2.1081) Cc: freebsd-mips@freebsd.org Subject: Re: Merging 64 bit changes to -HEAD X-BeenThere: freebsd-mips@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting FreeBSD to MIPS List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 30 Jun 2010 17:08:26 -0000 On Jun 30, 2010, at 9:57 AM, Jayachandran C. wrote: > On Tue, Jun 29, 2010 at 10:32 PM, Luiz Otavio O Souza > wrote: >>=20 >> On Jun 29, 2010, at 8:02 AM, Jayachandran C. wrote: >>=20 >>> On Tue, Jun 29, 2010 at 2:28 AM, Luiz Otavio O Souza = wrote: >>>>> Thanks for the the update. Looks like pmap_map for kernel is = failing, >>>>> may be the new tlb_update code causes this. Can you apply the >>>>> attached patch and see if the problem still persists, it replaces = the >>>>> new tlb_update code with the older version. >>>>>=20 >>>>> Obviously not a fix, but if we can narrow it down to this = function, >>>>> fixing will be easier. >>>>>=20 >>>>> JC. >>>>> >>>>=20 >>>> JC, >>>>=20 >>>> This fix the problem ! Thanks ! Now, at least, you know where to = look :) >>>=20 >>> The new tlb_update does not seem to update the tlb entry if the tlbp >>> fails. Here's a patch that should make the new function behave like >>> the older one. The patch is in attached file 'tlb-update.diff'. >>>=20 >>> If that does not work, I'm not sure what the issue is. You could = also >>> try try the nop-change.diff attached. It tries to switch the ssnop >>> used for delay in the new code with 'nop' which was used by the old >>> code. >>>=20 >>> Thanks, >>> JC. >>> >>=20 >> JC, >>=20 >> The nop-change seems to have no effect at all and with the tlb-update = patch the kernel apparently crash at bzero(), here is the dmesg with = TRAP_DEBUG enabled: >>=20 >> http://mips.pastebin.com/jydPvJ20 >>=20 >> So hopefully you are on the right track and this may be something = obvious to you. >=20 > Not yet :) I really hoped the earlier change would fix it. The number > of nop does not seem to be the issue as it is higher in the C code > than the assembly. >=20 > Can you try the attached patch (try.diff) - this re-implements the > assembly code functionality almost in the same way in C. This really > should work, given that the patch which made it assembly worked... >=20 > If that works can you see if the second attached patch works, this > fixes a potential problem (ie, we should be masking 13bits for TLBHI). >=20 > Both patches should apply directly to SVN (not dependent on each > other, or on previous patches) >=20 > Thanks again, > JC. > JC, The try.diff works with or without the pte.h change (at least for a = simple boot) and the pte.h change does nothing without the try.diff. Thanks for the promptly fix :) Luiz