Gary Thomas <[EMAIL PROTECTED]> writes: > Nick Garnett wrote: > > Sergei Organov <[EMAIL PROTECTED]> 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
