diff options
author | Bernd Schmidt <bernds_cb1@t-online.de> | 2009-01-07 10:14:38 -0500 |
---|---|---|
committer | Bryan Wu <cooloney@kernel.org> | 2009-01-07 10:14:38 -0500 |
commit | dbdf20db537a5369c65330f878ad4905020a8bfa (patch) | |
tree | c7fa553755e2d75a6e98d3f32fbe41fab9f72609 /arch/blackfin/include/asm/cplb-mpu.h | |
parent | 6651ece9e257302ee695ee76e69a4427f7033235 (diff) |
Blackfin arch: Faster C implementation of no-MPU CPLB handler
This is a mixture ofcMichael McTernan's patch and the existing cplb-mpu code.
We ditch the old cplb-nompu implementation, which is a good example of
why a good algorithm in a HLL is preferrable to a bad algorithm written in
assembly. Rather than try to construct a table of all posible CPLBs and
search it, we just create a (smaller) table of memory regions and
their attributes. Some of the data structures are now unified for both
the mpu and nompu cases. A lot of needless complexity in cplbinit.c is
removed.
Further optimizations:
* compile cplbmgr.c with a lot of -ffixed-reg options, and omit saving
these registers on the stack when entering a CPLB exception.
* lose cli/nop/nop/sti sequences for some workarounds - these don't
* make
sense in an exception context
Additional code unification should be possible after this.
[Mike Frysinger <vapier.adi@gmail.com>:
- convert CPP if statements to C if statements
- remove redundant statements
- use a do...while loop rather than a for loop to get slightly better
optimization and to avoid gcc "may be used uninitialized" warnings ...
we know that the [id]cplb_nr_bounds variables will never be 0, so this
is OK
- the no-mpu code was the last user of MAX_MEM_SIZE and with that rewritten,
we can punt it
- add some BUG_ON() checks to make sure we dont overflow the small
cplb_bounds array
- add i/d cplb entries for the bootrom because there is functions/data in
there we want to access
- we do not need a NULL trailing entry as any time we access the bounds
arrays, we use the nr_bounds variable
]
Signed-off-by: Michael McTernan <mmcternan@airvana.com>
Signed-off-by: Mike Frysinger <vapier.adi@gmail.com>
Signed-off-by: Bernd Schmidt <bernds_cb1@t-online.de>
Signed-off-by: Bryan Wu <cooloney@kernel.org>
Diffstat (limited to 'arch/blackfin/include/asm/cplb-mpu.h')
-rw-r--r-- | arch/blackfin/include/asm/cplb-mpu.h | 62 |
1 files changed, 0 insertions, 62 deletions
diff --git a/arch/blackfin/include/asm/cplb-mpu.h b/arch/blackfin/include/asm/cplb-mpu.h deleted file mode 100644 index 80680ad7a378..000000000000 --- a/arch/blackfin/include/asm/cplb-mpu.h +++ /dev/null | |||
@@ -1,62 +0,0 @@ | |||
1 | /* | ||
2 | * File: include/asm-blackfin/cplbinit.h | ||
3 | * Based on: | ||
4 | * Author: | ||
5 | * | ||
6 | * Created: | ||
7 | * Description: | ||
8 | * | ||
9 | * Modified: | ||
10 | * Copyright 2004-2006 Analog Devices Inc. | ||
11 | * | ||
12 | * Bugs: Enter bugs at http://blackfin.uclinux.org/ | ||
13 | * | ||
14 | * This program is free software; you can redistribute it and/or modify | ||
15 | * it under the terms of the GNU General Public License as published by | ||
16 | * the Free Software Foundation; either version 2 of the License, or | ||
17 | * (at your option) any later version. | ||
18 | * | ||
19 | * This program is distributed in the hope that it will be useful, | ||
20 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
21 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
22 | * GNU General Public License for more details. | ||
23 | * | ||
24 | * You should have received a copy of the GNU General Public License | ||
25 | * along with this program; if not, see the file COPYING, or write | ||
26 | * to the Free Software Foundation, Inc., | ||
27 | * 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA | ||
28 | */ | ||
29 | #ifndef __ASM_BFIN_CPLB_MPU_H | ||
30 | #define __ASM_BFIN_CPLB_MPU_H | ||
31 | #include <linux/threads.h> | ||
32 | |||
33 | struct cplb_entry { | ||
34 | unsigned long data, addr; | ||
35 | }; | ||
36 | |||
37 | struct mem_region { | ||
38 | unsigned long start, end; | ||
39 | unsigned long dcplb_data; | ||
40 | unsigned long icplb_data; | ||
41 | }; | ||
42 | |||
43 | extern struct cplb_entry dcplb_tbl[NR_CPUS][MAX_CPLBS]; | ||
44 | extern struct cplb_entry icplb_tbl[NR_CPUS][MAX_CPLBS]; | ||
45 | extern int first_switched_icplb; | ||
46 | extern int first_mask_dcplb; | ||
47 | extern int first_switched_dcplb; | ||
48 | |||
49 | extern int nr_dcplb_miss[], nr_icplb_miss[], nr_icplb_supv_miss[]; | ||
50 | extern int nr_dcplb_prot[], nr_cplb_flush[]; | ||
51 | |||
52 | extern int page_mask_order; | ||
53 | extern int page_mask_nelts; | ||
54 | |||
55 | extern unsigned long *current_rwx_mask[NR_CPUS]; | ||
56 | |||
57 | extern void flush_switched_cplbs(unsigned int); | ||
58 | extern void set_mask_dcplbs(unsigned long *, unsigned int); | ||
59 | |||
60 | extern void __noreturn panic_cplb_error(int seqstat, struct pt_regs *); | ||
61 | |||
62 | #endif /* __ASM_BFIN_CPLB_MPU_H */ | ||