diff options
author | Joel Sherrill <joel.sherrill@OARcorp.com> | 2007-09-12 15:23:44 +0000 |
---|---|---|
committer | Joel Sherrill <joel.sherrill@OARcorp.com> | 2007-09-12 15:23:44 +0000 |
commit | f93630db63c411300bd638e7406c45ca06431741 (patch) | |
tree | 2681b1d89dd36b582125d481a593261d663efc0b /c/src/lib/libcpu/powerpc/mpc8xx | |
parent | 2007-09-12 Joel Sherrill <joel.sherrill@OARcorp.com> (diff) | |
download | rtems-f93630db63c411300bd638e7406c45ca06431741.tar.bz2 |
2007-09-12 Joel Sherrill <joel.sherrill@OARcorp.com>
PR 1257/bsps
* mpc5xx/exceptions/raw_exception.c, mpc5xx/irq/irq.c,
mpc6xx/exceptions/raw_exception.c,
mpc8260/exceptions/raw_exception.c,
mpc8xx/exceptions/raw_exception.c, new-exceptions/raw_exception.c,
ppc403/ictrl/ictrl.c, ppc403/irq/ictrl.c: Code outside of cpukit
should use the public API for
rtems_interrupt_disable/rtems_interrupt_enable. By bypassing the
public API and directly accessing _CPU_ISR_Disable and
_CPU_ISR_Enable, they were bypassing the compiler memory barrier
directive which could lead to problems. This patch also changes the
type of the variable passed into these routines and addresses minor
style issues.
Diffstat (limited to 'c/src/lib/libcpu/powerpc/mpc8xx')
-rw-r--r-- | c/src/lib/libcpu/powerpc/mpc8xx/exceptions/raw_exception.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/c/src/lib/libcpu/powerpc/mpc8xx/exceptions/raw_exception.c b/c/src/lib/libcpu/powerpc/mpc8xx/exceptions/raw_exception.c index 3b338d1f56..3dd28d9a89 100644 --- a/c/src/lib/libcpu/powerpc/mpc8xx/exceptions/raw_exception.c +++ b/c/src/lib/libcpu/powerpc/mpc8xx/exceptions/raw_exception.c @@ -82,7 +82,7 @@ int mpc8xx_vector_is_valid(rtems_vector vector) int mpc8xx_set_exception (const rtems_raw_except_connect_data* except) { - unsigned int level; + rtems_interrupt_level level; if (!mpc8xx_vector_is_valid(except->exceptIndex)) { return 0; @@ -98,7 +98,7 @@ int mpc8xx_set_exception (const rtems_raw_except_connect_data* except) return 0; } - _CPU_ISR_Disable(level); + rtems_interrupt_disable(level); raw_except_table [except->exceptIndex] = *except; codemove((void*)mpc8xx_get_vector_addr(except->exceptIndex), @@ -107,7 +107,7 @@ int mpc8xx_set_exception (const rtems_raw_except_connect_data* except) PPC_CACHE_ALIGNMENT); except->on(except); - _CPU_ISR_Enable(level); + rtems_interrupt_enable(level); return 1; } @@ -124,7 +124,7 @@ int mpc8xx_get_current_exception (rtems_raw_except_connect_data* except) int mpc8xx_delete_exception (const rtems_raw_except_connect_data* except) { - unsigned int level; + rtems_interrupt_level level; if (!mpc8xx_vector_is_valid(except->exceptIndex)){ return 0; @@ -141,7 +141,7 @@ int mpc8xx_delete_exception (const rtems_raw_except_connect_data* except) except->hdl.raw_hdl_size)) { return 0; } - _CPU_ISR_Disable(level); + rtems_interrupt_disable(level); except->off(except); codemove((void*)mpc8xx_get_vector_addr(except->exceptIndex), @@ -153,7 +153,7 @@ int mpc8xx_delete_exception (const rtems_raw_except_connect_data* except) raw_except_table[except->exceptIndex] = default_raw_except_entry; raw_except_table[except->exceptIndex].exceptIndex = except->exceptIndex; - _CPU_ISR_Enable(level); + rtems_interrupt_enable(level); return 1; } @@ -163,8 +163,8 @@ int mpc8xx_delete_exception (const rtems_raw_except_connect_data* except) */ int mpc8xx_init_exceptions (rtems_raw_except_global_settings* config) { - unsigned i; - unsigned int level; + int i; + rtems_interrupt_level level; /* * store various accelerators @@ -173,7 +173,7 @@ int mpc8xx_init_exceptions (rtems_raw_except_global_settings* config) local_settings = config; default_raw_except_entry = config->defaultRawEntry; - _CPU_ISR_Disable(level); + rtems_interrupt_disable(level); for (i=0; i <= LAST_VALID_EXC; i++) { if (!mpc8xx_vector_is_valid(i)){ @@ -190,7 +190,7 @@ int mpc8xx_init_exceptions (rtems_raw_except_global_settings* config) raw_except_table[i].off(&raw_except_table[i]); } } - _CPU_ISR_Enable(level); + rtems_interrupt_enable(level); return 1; } |