On Mon, 5 May 2008, Andi Kleen wrote:
quoted text
> Thomas Gleixner wrote:
> > On Fri, 2 May 2008, Andi Kleen wrote:
> >> +static void stack_overflow(void)
> >> +{
> >> +	printk("low stack detected by irq handler\n");
> > 
> > Needs a KERN_ERR
> 
> Just moving code. If there is one added it should be in another patch.

Err, you are not moving code. The printk is pretty different and
adding the KERN_xx in the same go is nothing which makes the patch
harder to understand.

quoted text
> Besides if anything it's a KERN_WARN I guess.

KERN_WARN is fine, even if I consider a stack overflow as an error.

quoted text
> > 
> >> +		/* Execute warning on interrupt stack */
> >> +		if (unlikely(overflow))
> >> +			call_on_stack2(stack_overflow, isp, 0, 0);
> >> +
> >> +		call_on_stack2(desc->handle_irq, isp, irq, desc);
> > 
> > arch/x86/kernel/irq_32.c:148: warning: passing argument 2 of