From owner-freebsd-mips@FreeBSD.ORG Tue Jun 29 11:02:57 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 D78F4106564A; Tue, 29 Jun 2010 11:02:57 +0000 (UTC) (envelope-from c.jayachandran@gmail.com) Received: from mail-vw0-f54.google.com (mail-vw0-f54.google.com [209.85.212.54]) by mx1.freebsd.org (Postfix) with ESMTP id 6ADDF8FC23; Tue, 29 Jun 2010 11:02:57 +0000 (UTC) Received: by vws13 with SMTP id 13so9232343vws.13 for ; Tue, 29 Jun 2010 04:02:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:received:in-reply-to :references:date:message-id:subject:from:to:cc:content-type; bh=r0lgrnVlqRDUuC28LAPlFOLzvwCfyEtlJr292Bze7uc=; b=AgW5ZzxhVmP+uahz5d7gJgpAi1Ic8pVeKJhFXZXrIkZpCk3QpptJMbcbWJsFPgBRAM kc+i99LcWhl/Nul8QlbZ7el+YdQG8gvvG4WddTEs2hKzZdavkRGbyfNX2HNN99URMEIT GrXNYEV6NNzZSshwgxVX44MfHxrmlcOSo2VvE= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=a5eQ7X4nKsYB1VVhMMjvs4X6p2eQbwBH1umpwP4NU5OWWwUL+FJRrFsWmdp3WL3ST3 KPFA1FOaUQskR9K8jUNjSiVqdzlmV7byAdDJWJ1tZYKlleLQqyFJdAa/H1elAuYFV5s/ XOhs+IAH7wPZdBc8L8IQhX/c22ngEzDb60qys= MIME-Version: 1.0 Received: by 10.220.47.79 with SMTP id m15mr3794485vcf.93.1277809372156; Tue, 29 Jun 2010 04:02:52 -0700 (PDT) Received: by 10.220.199.2 with HTTP; Tue, 29 Jun 2010 04:02:52 -0700 (PDT) In-Reply-To: References: <897604F6-95C4-49A8-B11F-277A74C8DBAE@gmail.com> <3C0AEF9B-AE0C-4459-A4E1-2C8C30C10FD6@gmail.com> Date: Tue, 29 Jun 2010 16:32:52 +0530 Message-ID: From: "Jayachandran C." To: Luiz Otavio O Souza Content-Type: multipart/mixed; boundary=0016e6470d0cb7d3aa048a292d86 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: Tue, 29 Jun 2010 11:02:57 -0000 --0016e6470d0cb7d3aa048a292d86 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On Tue, Jun 29, 2010 at 2:28 AM, Luiz Otavio O Souza w= rote: >> Thanks for the the update. Looks like pmap_map for kernel is failing, >> may be the new tlb_update code causes this. =A0Can you apply the >> attached patch and see if the problem still persists, it replaces the >> new tlb_update code with the older version. >> >> Obviously not a fix, but if we can narrow it down to this function, >> fixing will be easier. >> >> JC. >> > > JC, > > This fix the problem ! Thanks ! Now, at least, you know where to look :) 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'. 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. Thanks, JC. --0016e6470d0cb7d3aa048a292d86 Content-Type: text/plain; charset=US-ASCII; name="tlb-update.diff" Content-Disposition: attachment; filename="tlb-update.diff" Content-Transfer-Encoding: base64 X-Attachment-Id: f_gb0mqam80 SW5kZXg6IHN5cy9taXBzL21pcHMvdGxiLmMKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gc3lzL21pcHMvbWlwcy90 bGIuYwkocmV2aXNpb24gMjA5NTIxKQorKysgc3lzL21pcHMvbWlwcy90bGIuYwkod29ya2luZyBj b3B5KQpAQCAtMjE3LDE5ICsyMTcsMjEgQEAKIHZvaWQKIHRsYl91cGRhdGUoc3RydWN0IHBtYXAg KnBtYXAsIHZtX29mZnNldF90IHZhLCBwdF9lbnRyeV90IHB0ZSkKIHsKLQlyZWdpc3Rlcl90IG1h c2ssIGFzaWQ7Ci0JcmVnaXN0ZXJfdCBzOworCXB0X2VudHJ5X3Qgb3RoZXI7CisJcmVnaXN0ZXJf dCBtYXNrLCBhc2lkLCBoaSwgczsKIAlpbnQgaTsKIAogCXZhICY9IH5QQUdFX01BU0s7CisJb3Ro ZXIgPSBwdGUgJiBQVEVfRzsKIAlwdGUgJj0gflRMQkxPX1NXQklUU19NQVNLOworCWhpID0gVExC SElfRU5UUlkodmEsIHBtYXBfYXNpZChwbWFwKSk7CiAKIAlzID0gaW50cl9kaXNhYmxlKCk7CiAJ bWFzayA9IG1pcHNfcmRfcGFnZW1hc2soKTsKIAlhc2lkID0gbWlwc19yZF9lbnRyeWhpKCkgJiBU TEJISV9BU0lEX01BU0s7CiAKIAltaXBzX3dyX3BhZ2VtYXNrKDApOwotCW1pcHNfd3JfZW50cnlo aShUTEJISV9FTlRSWSh2YSwgcG1hcF9hc2lkKHBtYXApKSk7CisJbWlwc193cl9lbnRyeWhpKGhp KTsKIAl0bGJfcHJvYmUoKTsKIAlpID0gbWlwc19yZF9pbmRleCgpOwogCWlmIChpID49IDApIHsK QEAgLTI0MSw2ICsyNDMsMTcgQEAKIAkJCW1pcHNfd3JfZW50cnlsbzEocHRlKTsKIAkJfQogCQl0 bGJfd3JpdGVfaW5kZXhlZCgpOworCX0gZWxzZSB7CisJCW1pcHNfd3JfcGFnZW1hc2soMCk7CisJ CW1pcHNfd3JfZW50cnloaShoaSk7CisJCWlmICgodmEgJiBQQUdFX1NJWkUpID09IDApIHsKKwkJ CW1pcHNfd3JfZW50cnlsbzAocHRlKTsKKwkJCW1pcHNfd3JfZW50cnlsbzEob3RoZXIpOworCQl9 IGVsc2UgeworCQkJbWlwc193cl9lbnRyeWxvMChvdGhlcik7CisJCQltaXBzX3dyX2VudHJ5bG8x KHB0ZSk7CisJCX0KKwkJdGxiX3dyaXRlX3JhbmRvbSgpOwogCX0KIAogCW1pcHNfd3JfZW50cnlo aShhc2lkKTsK --0016e6470d0cb7d3aa048a292d86 Content-Type: text/plain; charset=US-ASCII; name="nop-change.diff" Content-Disposition: attachment; filename="nop-change.diff" Content-Transfer-Encoding: base64 X-Attachment-Id: f_gb0mrhru1 SW5kZXg6IHN5cy9taXBzL2luY2x1ZGUvY3B1cmVncy5oCj09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIHN5cy9taXBz L2luY2x1ZGUvY3B1cmVncy5oCShyZXZpc2lvbiAyMDk1MjEpCisrKyBzeXMvbWlwcy9pbmNsdWRl L2NwdXJlZ3MuaAkod29ya2luZyBjb3B5KQpAQCAtMTE5LDcgKzExOSw3IEBACiAgKiBQaWNrIGEg cmVhc29uYWJsZSBkZWZhdWx0IGJhc2VkIG9uIHRoZSAidHlwaWNhbCIgc3BhY2luZyBkZXNjcmli ZWQgaW4gdGhlCiAgKiAiQ1AwIEhhemFyZHMiIGNoYXB0ZXIgb2YgTUlQUyBBcmNoaXRlY3R1cmUg Qm9vayBWb2wgSUlJLgogICovCi0jZGVmaW5lCUNPUDBfU1lOQyAgc3Nub3A7IHNzbm9wOyBzc25v cDsgc3Nub3A7IHNzbm9wCisjZGVmaW5lCUNPUDBfU1lOQyAgbm9wOyBub3A7IG5vcDsgbm9wOyBu b3AKICNlbmRpZgogI2RlZmluZQlDT1AwX0hBWkFSRF9GUFVFTkFCTEUJbm9wOyBub3A7IG5vcDsg bm9wOwogCg== --0016e6470d0cb7d3aa048a292d86--