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/context.S | |
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/context.S')
-rw-r--r-- | arch/blackfin/include/asm/context.S | 39 |
1 files changed, 39 insertions, 0 deletions
diff --git a/arch/blackfin/include/asm/context.S b/arch/blackfin/include/asm/context.S index 9ce21f68e914..16561ab18b38 100644 --- a/arch/blackfin/include/asm/context.S +++ b/arch/blackfin/include/asm/context.S | |||
@@ -357,3 +357,42 @@ | |||
357 | SYSCFG = [sp++]; | 357 | SYSCFG = [sp++]; |
358 | csync; | 358 | csync; |
359 | .endm | 359 | .endm |
360 | |||
361 | .macro save_context_cplb | ||
362 | [--sp] = (R7:0, P5:0); | ||
363 | [--sp] = fp; | ||
364 | |||
365 | [--sp] = a0.x; | ||
366 | [--sp] = a0.w; | ||
367 | [--sp] = a1.x; | ||
368 | [--sp] = a1.w; | ||
369 | |||
370 | [--sp] = LC0; | ||
371 | [--sp] = LC1; | ||
372 | [--sp] = LT0; | ||
373 | [--sp] = LT1; | ||
374 | [--sp] = LB0; | ||
375 | [--sp] = LB1; | ||
376 | |||
377 | [--sp] = RETS; | ||
378 | .endm | ||
379 | |||
380 | .macro restore_context_cplb | ||
381 | RETS = [sp++]; | ||
382 | |||
383 | LB1 = [sp++]; | ||
384 | LB0 = [sp++]; | ||
385 | LT1 = [sp++]; | ||
386 | LT0 = [sp++]; | ||
387 | LC1 = [sp++]; | ||
388 | LC0 = [sp++]; | ||
389 | |||
390 | a1.w = [sp++]; | ||
391 | a1.x = [sp++]; | ||
392 | a0.w = [sp++]; | ||
393 | a0.x = [sp++]; | ||
394 | |||
395 | fp = [sp++]; | ||
396 | |||
397 | (R7:0, P5:0) = [SP++]; | ||
398 | .endm | ||