Re: One more boobytrap needed for 2.2.15pre ?

[Andris Pavenis]

On Wed, 16 Feb 2000, [e-mail] wrote:
> Zitiere Andris Pavenis <[e-mail]>:
> > 
> > Applied patch, built kernel and got error messages 
> > 	\'do_IRQ: bad state change in bh\'  
> > in average once per minute in first about 10 minutes 
> minutes uptime:
> 
> That could be the bug. IŽll post another patch this 
> evening:
> modify run_bottom_halves() in kernel/softirq.c, and 
> check which BH handler calls set_current_state().
> 

Tried to do something similar:
	set some global variable which is non 0 if in do_IRQ()
	copied last boobytrap to kernel/softint.c, but checked only when called
		from do_IRQ(), but didn't change current->state back there

Results: got error messages about 2 bh_handlers (about same number for each):
	tqueue_bh
	console_bh

One note: I got more messages from do_IRQ() than from run_botom_halves()
	(10 from run_botom_halves,  14 from do_IRQ). I added checks
	around (*bh)() only.

Andris