]> err.no Git - linux-2.6/commitdiff
[Blackfin] arch: fix bug - a crash on bootup with CONFIG_MPU on the BF548
authorBernd Schmidt <bernds_cb1@t-online.de>
Wed, 23 Apr 2008 18:56:36 +0000 (02:56 +0800)
committerBryan Wu <cooloney@kernel.org>
Wed, 23 Apr 2008 18:56:36 +0000 (02:56 +0800)
The function flush_switched_dcplbs was clearing the CPLB entries covering
the process permission bitmasks.  This means that the sequence

flush_switched_dcplbs ();
        set_mask_dcplbs(mm->context.page_rwx_mask);

has a problem: if kernel code (such as an interrupt) causes a CPLB miss before
set_mask_dcplbs completes, the CPLB handler function causes a double fault,
with an instantaneous reboot.

This bug fix is dedicated to Michael Hennerich, the only person in the world
capable of providing working JTAG hardware.

Signed-off-by: Bernd Schmidt <bernds_cb1@t-online.de>
Signed-off-by: Bryan Wu <cooloney@kernel.org>
arch/blackfin/kernel/cplb-mpu/cplbmgr.c

index 6f1c053903c4a71df3a7e4fd4517153657148971..f23bba5c6e63469f8f1c2c168206c221ad04951b 100644 (file)
@@ -307,7 +307,7 @@ void flush_switched_cplbs(void)
        enable_icplb();
 
        disable_dcplb();
-       for (i = first_mask_dcplb; i < MAX_CPLBS; i++) {
+       for (i = first_switched_dcplb; i < MAX_CPLBS; i++) {
                dcplb_tbl[i].data = 0;
                bfin_write32(DCPLB_DATA0 + i * 4, 0);
        }