Re: Improperly handled case in arch/mips/au1000/common/time.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2003-04-22 at 15:56, Jun Sun wrote:
> I think this is a good example to show benefit of code sharing.
> There is no good reason for au1x00 boards of not using new time.c.
> You get to write less board code, fewer bugs and future proof.

The I didn't use the generic time.c back then is power management. The
CP0 counter sleeps when using the 'wait' instruction, so in that case
you have to use a different counter with a rather poor resolution. The
modifications I had to make were such that they couldn't go in the
generic time.c. But that area definitely needs to be revisited and
cleaned up.

Pete

> Jun
> 
> On Tue, Apr 22, 2003 at 12:54:50PM -0700, Jeff Baitis wrote:
> > Pete:
> > 
> > While struggling to get Linux up on Evolution's custom board based on the
> > Au1500, I discovered a poorly handled case in time.c; null interrupts are
> > handled should not affect the local IRQ count. (if the local IRQ count is not
> > decremented, tests for in_irq() fail.)
> > 
> > Thanks for taking a look at my patch!
> > 
> > -Jeff
> > 
> > Index: time.c
> > ===================================================================
> > RCS file: /home/cvs/linux/arch/mips/au1000/common/time.c,v
> > retrieving revision 1.5.2.10
> > diff -u -r1.5.2.10 time.c
> > --- time.c      25 Mar 2003 14:30:19 -0000      1.5.2.10
> > +++ time.c      22 Apr 2003 19:47:24 -0000
> > @@ -114,6 +114,7 @@
> >         return;
> >  
> >  null:
> > +       irq_exit(cpu, irq);
> >         ack_r4ktimer(0);
> >  }
> > 
> > 
> 
> 



[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux