From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Fri, 13 May 2011 19:17:54 +0200 (CEST) Received: from mail3.caviumnetworks.com ([12.108.191.235]:16612 "EHLO mail3.caviumnetworks.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S1491870Ab1EMRRs (ORCPT ); Fri, 13 May 2011 19:17:48 +0200 Received: from caexch01.caveonetworks.com (Not Verified[192.168.16.9]) by mail3.caviumnetworks.com with MailMarshal (v6,7,2,8378) id ; Fri, 13 May 2011 10:18:49 -0700 Received: from caexch01.caveonetworks.com ([192.168.16.9]) by caexch01.caveonetworks.com with Microsoft SMTPSVC(6.0.3790.4675); Fri, 13 May 2011 10:17:46 -0700 Received: from dd1.caveonetworks.com ([12.108.191.236]) by caexch01.caveonetworks.com over TLS secured channel with Microsoft SMTPSVC(6.0.3790.4675); Fri, 13 May 2011 10:17:46 -0700 Message-ID: <4DCD67BA.5090002@caviumnetworks.com> Date: Fri, 13 May 2011 10:17:46 -0700 From: David Daney User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.15) Gecko/20101027 Fedora/3.0.10-1.fc12 Thunderbird/3.0.10 MIME-Version: 1.0 To: Ralf Baechle CC: Kevin Cernekee , linux-mips@linux-mips.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/4] MIPS: Replace _PAGE_READ with _PAGE_NO_READ References: <7aa38c32b7748a95e814e5bb0583f967@localhost> <20110513150707.GA26389@linux-mips.org> <20110513155605.GA30674@linux-mips.org> In-Reply-To: <20110513155605.GA30674@linux-mips.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 13 May 2011 17:17:46.0386 (UTC) FILETIME=[AD643F20:01CC1191] Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 30012 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: ddaney@caviumnetworks.com Precedence: bulk X-list: linux-mips On 05/13/2011 08:56 AM, Ralf Baechle wrote: > On Fri, May 13, 2011 at 08:46:18AM -0700, Kevin Cernekee wrote: > >>>> On hardware that does not support RI/XI, EntryLo bits 31:30 / 63:62 will >>>> remain unset and RI/XI permissions will not be enforced. >>> >>> Nice idea but it breaks on 64-bit hardware running 32-bit kernels. On >>> those the RI/XI bits written to c0_entrylo0/1 31:30 will be interpreted as >>> physical address bits 37:36. >> >> Hmm, are you sure? (Unfortunately I do not have a 64-bit machine to >> test it on.) >> >> I did not touch David's existing build_update_entries(), which makes a >> point not to set the RI/XI bits when the RIXI feature is disabled: >> >> if (kernel_uses_smartmips_rixi) { >> UASM_i_SRL(p, tmp, tmp, ilog2(_PAGE_NO_EXEC)); >> UASM_i_SRL(p, ptep, ptep, ilog2(_PAGE_NO_EXEC)); >> UASM_i_ROTR(p, tmp, tmp, ilog2(_PAGE_GLOBAL) - >> ilog2(_PAGE_NO_EXEC)); >> if (r4k_250MHZhwbug()) >> UASM_i_MTC0(p, 0, C0_ENTRYLO0); >> UASM_i_MTC0(p, tmp, C0_ENTRYLO0); /* load it */ >> UASM_i_ROTR(p, ptep, ptep, ilog2(_PAGE_GLOBAL) - >> ilog2(_PAGE_NO_EXEC)); >> } else { >> UASM_i_SRL(p, tmp, tmp, ilog2(_PAGE_GLOBAL)); /* >> convert to entrylo0 */ >> if (r4k_250MHZhwbug()) >> UASM_i_MTC0(p, 0, C0_ENTRYLO0); >> UASM_i_MTC0(p, tmp, C0_ENTRYLO0); /* load it */ >> UASM_i_SRL(p, ptep, ptep, ilog2(_PAGE_GLOBAL)); /* >> convert to entrylo1 */ >> if (r45k_bvahwbug()) >> uasm_i_mfc0(p, tmp, C0_INDEX); >> } >> >> If RIXI is enabled, it shifts the SW bits off the end of the register, >> then rotates the RI/XI bits into place. >> >> If RIXI is disabled, it shifts the SW bits + RI/XI bits off the end of >> the register. It should not be setting bits 31:30 or 63:62, ever. >> >> (A side issue here is that ROTR is a MIPS R2 instruction, so we could >> never remove the old handler and use the RIXI version of the TLB >> handler on an R1 machine.) >> >> If setting EntryLo bits 31:30 for RI/XI is illegal on a 64-bit system >> running a 32-bit kernel, I suspect we will have a problem with the >> existing RIXI TLB update code, regardless of whether my changes are >> applied. > > I'm not totally certain with my explanation but it seemed like a good > working hypothesis. Jayachandran C. bisected this morning's linux-queue > on his Netlogic XLR which is MIPS64 R1 and found this comment causing > the problem. He should dump out the TLB Refill handler with the patch set applied. Perhaps it is erroneously getting in RI/XI mode. Put #define DEBUG 1 as the first line of tlbex.c, then boot the kernel with 'debug' on the command line. Also it is best to have early printk enabled so you can see the dump. David Daney.