Re: [PATCH] powerpc/8xx: fix regression introduced by cache coherency rewrite
From: Joakim Tjernlund <hidden>
Date: 2009-10-05 19:17:58
From: Joakim Tjernlund <hidden>
Date: 2009-10-05 19:17:58
Benjamin Herrenschmidt [off-list ref] wrote on 04/10/2009 22:28:38:
quoted
I have managed to update the TLB code to make proper use of dirty and accessed states. Advantages are: - I/D TLB Miss never needs to write to the linux pte, saving a few cyclesThat's good, that leaves us with only 40x to fix now. Also we can remove atomic updates of PTEs for all non-hash. It's pointless on those CPUs anyway.quoted
- Accessed is only set by I/D TLB Error, should be a plus when SWAP is used.No need for that neither. ISI/DSI shouldn't touch the PTE. They should just fall back to C code which takes care of it all.l
Ben, for my understanding: It seems to that the TLB Miss routines in head_32.S are less than optimal as it too touches the pte every time it hits. Would it not be better to test if ACCESSED and friends are already set and skip storing the same pte over and over again? Jocke