Subject: Re: Patch: Generic TODR for ARM ports
To: Garrett D'Amore <garrett_damore@tadpole.com>
From: Garrett D'Amore <garrett_damore@tadpole.com>
List: port-hpcarm
Date: 09/18/2006 13:37:00
Peter Postma wrote:
> On Sun, Sep 17, 2006 at 07:37:26PM -0700, Garrett D'Amore wrote:
>   
>> Peter Postma wrote:
>>     
>>> On Sun, Sep 17, 2006 at 09:15:55AM -0700, Garrett D'Amore wrote:
>>>       
>>>> The double selection messages suggest that you need to update the kernel
>>>> from CVS.  A change was made to todr to prevent tc_windup() from getting
>>>> called with interrupts enabled -- I think that is what has happened
>>>> here.  (You should not see double messages like this.)
>>>>     
>>>>         
>>> I believe that I'm up to date :-) which revisions should I have?
>>>   
>>>       
>> 1.12 of kern_tc.c is what you need.
>>
>> This version calls tc_windup() inside splclock().  Earlier versions didn't.
>>
>>     
>
> I'm at 1.12. I debugged a bit and it seems that when I change the splclock()
> to splhigh(), the double messages are gone. So this might be a hpcarm
> specific problem.
>
>   

Hmm... maybe splclock() isn't really blocking the clock interrupt?  It
certainly sounds like an hpcarm problem.

-- 
Garrett D'Amore, Principal Software Engineer
Tadpole Computer / Computing Technologies Division,
General Dynamics C4 Systems
http://www.tadpolecomputer.com/
Phone: 951 325-2134  Fax: 951 325-2191