Re: [PATCH 2.6.11-rc3-bk4] arch/i386/kernel/pci/irq.c: Wrong message output

From: Greg KH
Date: Tue Feb 08 2005 - 00:36:54 EST


On Mon, Feb 07, 2005 at 09:06:18PM -0800, Mark F. Haigh wrote:
>
> (Same basic problem I just reported in a seperate thread against 2.4.29-bk8)
>
> The following has been reported in the wild for kernel 2.6.8-24:
>
> PCI: Enabling device 0000:00:05.0 (0000 -> 0002)
> PCI: No IRQ known for interrupt pin @ of device 0000:00:05.0. Probably
> buggy MP table.
>
> It should read "No IRQ known for interrupt pin A", but the 'pin'
> variable has already been decremented (from 1 to 0), so the line:
>
> printk(KERN_WARNING "PCI: No IRQ known for interrupt pin %c of device
> %s.%s\n", 'A' + pin - 1, dev->slot_name, msg);
>
> causes "pin @" to be output, because 'A' + 0 - 1 == '@'.
>
> The supplied patch should fix it. It also removes a redundant check for
> a nonzero pin.
>
>
> Mark F. Haigh
> Mark.Haigh@xxxxxxxxxxxxxx
>

> --- arch/i386/pci/irq.c.orig 2005-02-07 20:40:58.140856536 -0800
> +++ arch/i386/pci/irq.c 2005-02-07 20:46:06.713946296 -0800

Can you resend this so it can be applied with -p1 to patch, and a
Signed-off-by: line?

thanks,

greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/