RTOS/rtems
1
0
Fork 0
forked from Imagelibrary/rtems
Code Pull Requests Activity

sparc: Fix SMP context switch

Browse Source 
The value of register %g1 was overwritten in a delay slot.  Use
additional %g4 register.

Bug introduced by 258ad71e96.
This commit is contained in:
Sebastian Huber
2016-03-17 13:30:54 +01:00
parent d731bc6fe7
commit d64f2e7a85
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
c/src/lib/libbsp/sparc/shared/irq_asm.S
View File

@@ -236,21 +236,21 @@ check_is_executing:
! We may have a new heir ! We may have a new heir
! Read the executing and heir ! Read the executing and heir
ld [%g6 + PER_CPU_OFFSET_EXECUTING], %g1 ld [%g6 + PER_CPU_OFFSET_EXECUTING], %g2
ld [%g6 + PER_CPU_OFFSET_HEIR], %g2 ld [%g6 + PER_CPU_OFFSET_HEIR], %g4
! Update the executing only if necessary to avoid cache line ! Update the executing only if necessary to avoid cache line
! monopolization. ! monopolization.
cmp %g1, %g2 cmp %g2, %g4
beq try_update_is_executing beq try_update_is_executing
mov 1, %g1 mov 1, %g1
! Calculate the heir context pointer ! Calculate the heir context pointer
sub %o1, %g1, %g1 sub %o1, %g2, %g2
add %g1, %g2, %o1 add %g2, %g4, %o1
! Update the executing ! Update the executing
st %g2, [%g6 + PER_CPU_OFFSET_EXECUTING] st %g4, [%g6 + PER_CPU_OFFSET_EXECUTING]
ba try_update_is_executing ba try_update_is_executing
mov 1, %g1 mov 1, %g1