Date: Tue, 29 Jun 2010 16:32:52 +0530 From: "Jayachandran C." <c.jayachandran@gmail.com> To: Luiz Otavio O Souza <lists.br@gmail.com> Cc: freebsd-mips@freebsd.org Subject: Re: Merging 64 bit changes to -HEAD Message-ID: <AANLkTilQIqF4FCfgLdVcKdcsAUVjCmr89Lu0TEXUFdYN@mail.gmail.com> In-Reply-To: <C6D73C96-3640-4502-A9D7-B3597E37E80A@gmail.com> References: <AANLkTik8jFkB7FTIIhyjalkfv1c0yXqse57Jzz527uf_@mail.gmail.com> <897604F6-95C4-49A8-B11F-277A74C8DBAE@gmail.com> <AANLkTilfW_zOFKuIa0gJ3ahTo-vGC1VNk99a1H24uFRq@mail.gmail.com> <AANLkTil78NFxH016C7MntD8L3d4rFlCudJ0Lv22L0KCb@mail.gmail.com> <3C0AEF9B-AE0C-4459-A4E1-2C8C30C10FD6@gmail.com> <AANLkTint7Hyf79EH29OLsIfreQRd7dQMdvX9wRq4v_yG@mail.gmail.com> <C6D73C96-3640-4502-A9D7-B3597E37E80A@gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--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 <lists.br@gmail.com> 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. >> <try.diff> > > 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--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?AANLkTilQIqF4FCfgLdVcKdcsAUVjCmr89Lu0TEXUFdYN>