PERFORCE change 99751 for review

Paolo Pisati p.pisati at oltrelinux.com
Wed Jun 21 13:13:36 UTC 2006


On Wed, Jun 21, 2006 at 12:14:29PM +0000, Paolo Pisati wrote:
> http://perforce.freebsd.org/chv.cgi?CH=99751
> 
> Change 99751 by piso at piso_newluxor on 2006/06/21 12:14:00
> 
> 	Updated a bit to reflect work status
> 
> Affected files ...
> 
> .. //depot/projects/soc2006/intr_filter/arm/arm/intr.c#2 edit
> .. //depot/projects/soc2006/intr_filter/notes#2 edit
> 
> Differences ...
> 
> ==== //depot/projects/soc2006/intr_filter/arm/arm/intr.c#2 (text+ko) ====
> 
> @@ -99,10 +99,9 @@
>  }
>  
>  void
> -arm_handler_execute(struct trapframe *frame, int irqnb)
> +arm_handler_execute(struct trapframe *frame, int irqnb __unused)
>  {
>  	struct intr_event *event;
> -	struct intr_handler *ih;
>  	struct thread *td = curthread;
>  	int i, thread;
>  
> @@ -115,14 +114,7 @@
>  			continue;
>  
>  		/* Execute fast handlers. */
> -		thread = 0;
> -		TAILQ_FOREACH(ih, &event->ie_handlers, ih_next) {
> -			if (!(ih->ih_flags & IH_FAST))
> -				thread = 1;
> -			else
> -				ih->ih_handler(ih->ih_argument ?
> -				    ih->ih_argument : frame);
> -		}
> +		thread = intr_filter_loop(event, frame);
>  
>  		/* Schedule thread if needed. */
>  		if (thread)
> 

For some reason, the server had this change pending, and
when i submitted an update to the todo list, this stale modification
crept in... the odd thing is that i was sure i commited it some
days ago...

-- 

Paolo

Piso's first law: nothing works as expected!


More information about the p4-projects mailing list