This is the mail archive of the
ecos-patches@sourceware.org
mailing list for the eCos project.
Re: FIFO DSRs... the 2nd
- From: Nick Garnett <nickg at ecoscentric dot com>
- To: Gary Thomas <gary at mlbassoc dot com>
- Cc: ecos-patches at sources dot redhat dot com
- Date: 11 Aug 2006 11:31:48 +0100
- Subject: Re: FIFO DSRs... the 2nd
- References: <004b01c6589d$006a3cf0$8262fea9@nullnullsix> <87bqvac5bw.fsf@javad.com> <m3wt9fliwf.fsf@xl5.calivar.com> <44DC533D.3040502@mlbassoc.com>
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