Re: [PATCH 5/9] MIPS: dump_tlb: Take global bit into account

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

 



On Mon, 18 May 2015, James Hogan wrote:

> >> +		if (!(entrylo0 & 1) && (entryhi & 0xff) != asid)
> > 
> >  Hmm, it looks like r3k_dump_tlb.c will need a similar update.  I suggest 
> 
> Yes, quite possibly. Would you be happy to test such a patch (assuming
> you have r3000 hardware available)? Patch 1 should allow the code to be
> easily triggered.

 I'll test the change when you have it, no problem with that.

> > using _PAGE_GLOBAL and ASID_MASK rather than hardcoded 1 and 0xff.
> 
> Yeh, as you mentioned these describe the PTE rather than what goes in
> EntryLo. Perhaps it makes sense to have a few more TLB dependent
> definitions in mipsregs.h (patch 7 already adds a couple for RI/XI bits).

 I think so.  Hardcoded magic values are a pain to track down when you 
need to do so.

 Will you be able to add MIPS_ENTRYLO_G and R3K_ENTRYLO_G, etc. macros 
then please?  I think you can reorder all mipsregs.h changes ahead of the 
series as a single patch.

 Thanks,

  Maciej





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

  Powered by Linux