FIFO DSRs... the 2nd

[Nick Garnett]

Gary Thomas <gary@mlbassoc.com> writes:

> 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?

Didn't notice that. Something experimental that got left around. Now
reversed.


-- 
Nick Garnett                                 eCos Kernel Architect
http://www.ecoscentric.com            The eCos and RedBoot experts