Re: [PATCH 1/4] MIPS: Replace _PAGE_READ with _PAGE_NO_READ

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.



[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux