[PATCH 13/16] powerpc/44x: No need to mask MSR:CE,ME or DE in _tlbil_va on 440

Josh Boyer jwboyer at linux.vnet.ibm.com
Mon Dec 15 23:25:54 EST 2008


On Mon, 15 Dec 2008 16:45:05 +1100
Benjamin Herrenschmidt <benh at kernel.crashing.org> wrote:

> The handlers for Critical, Machine Check or Debug interrupts
> will save and restore MMUCR nowadays, thus we only need to
> disable normal interrupts when invalidating TLB entries.
> 
> Signed-off-by: Benjamin Herrenschmidt <benh at kernel.crashing.org>

Acked-by: Josh Boyer <jwboyer at linux.vnet.ibm.com>

> ---
> 
>  arch/powerpc/mm/tlb_nohash_low.S |   19 ++++++++++---------
>  1 file changed, 10 insertions(+), 9 deletions(-)
> 
> --- linux-work.orig/arch/powerpc/mm/tlb_nohash_low.S	2008-12-15 13:34:57.000000000 +1100
> +++ linux-work/arch/powerpc/mm/tlb_nohash_low.S	2008-12-15 13:35:07.000000000 +1100
> @@ -75,18 +75,19 @@ _GLOBAL(_tlbil_va)
>  	mfspr	r5,SPRN_MMUCR
>  	rlwimi	r5,r4,0,24,31			/* Set TID */
> 
> -	/* We have to run the search with interrupts disabled, even critical
> -	 * and debug interrupts (in fact the only critical exceptions we have
> -	 * are debug and machine check).  Otherwise  an interrupt which causes
> -	 * a TLB miss can clobber the MMUCR between the mtspr and the tlbsx. */
> +	/* We have to run the search with interrupts disabled, otherwise
> +	 * an interrupt which causes a TLB miss can clobber the MMUCR
> +	 * between the mtspr and the tlbsx.
> +	 *
> +	 * Critical and Machine Check interrupts take care of saving
> +	 * and restoring MMUCR, so only normal interrupts have to be
> +	 * taken care of.
> +	 */
>  	mfmsr	r4
> -	lis	r6,(MSR_EE|MSR_CE|MSR_ME|MSR_DE)@ha
> -	addi	r6,r6,(MSR_EE|MSR_CE|MSR_ME|MSR_DE)@l
> -	andc	r6,r4,r6
> -	mtmsr	r6
> +	wrteei	0
>  	mtspr	SPRN_MMUCR,r5
>  	tlbsx.	r3, 0, r3
> -	mtmsr	r4
> +	wrtee	r4
>  	bne	1f
>  	sync
>  	/* There are only 64 TLB entries, so r3 < 64,



More information about the Linuxppc-dev mailing list