Re: FIFO DSRs... the 2nd

[Gary Thomas <gary at mlbassoc dot com>]

Nick Garnett wrote:
> Sergei Organov <osv@javad.com> writes:
>
> > Attached is modified patch that implements FIFO scheduling of DSRs and
> > makes it the default.
> >
> > Compared to the original patch, less files are modified, more assertions
> > are added, and FIFO variant of the call_pending_DSRs_inner() is
> > optimized both for single DSR and for multiple DSRs in the list.
> >
> > In addition, two related minor fixes to the comments in the kernel tests
> > are provided.
> >
> > Tested on ARM.
>
> This has now been checked in, the actual patch committed is attached.
>
>
>
>
>
> ------------------------------------------------------------------------
>
> Index: hal/sh/arch/current/src/context.S
> ===================================================================
> RCS file: /cvs/ecos/ecos/packages/hal/sh/arch/current/src/context.S,v
> retrieving revision 1.6
> diff -u -5 -r1.6 context.S
> --- hal/sh/arch/current/src/context.S	5 Dec 2003 17:06:23 -0000	1.6
> +++ hal/sh/arch/current/src/context.S	11 Aug 2006 09:20:18 -0000
> @@ -238,15 +238,15 @@
>          lds.l   @r0+,fpscr
>  #endif
>          
>          lds.l   @r0+,pr                 ! pr
>  
> -        mov     r3,r15                  ! update stack pointer
> -
>          mov.l   @r0+,r2                 ! SR
>          hal_cpu_int_merge r2,r0,r1      ! restore interrupt state
>  
> +        mov     r3,r15                  ! update stack pointer
> +
>          rts                             ! and return
>           nop
>  
>  #------------------------------------------------------------------------------
>  # HAL longjmp, setjmp implementations

Hmm, this doesn't seem to be related to the DSR mechanism and came
with no ChangeLog entry.

Comments?

--
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------