lkml.org 
[lkml]   [2006]   [Nov]   [14]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [2.6 patch] arch/i386/kernel/io_apic.c: handle a negative return value
FromIngo Molnar <>
DateTue, 14 Nov 2006 07:46:52 +0100
On Mon, 2006-11-13 at 16:42 -0800, Andrew Morton wrote:
>         pin  = find_isa_irq_pin(8, mp_INT);
> +       if (pin == -1) {
> +               printk(KERN_ERR "unlock_ExtINT_logic:
> find_isa_irq_pin()
> +                               "failed\n");
> +               return;
> +       }
>         apic = find_isa_irq_apic(8, mp_INT);
> -       if (pin == -1)
> +       if (apic == -1) {
> +               printk(KERN_ERR "unlock_ExtINT_logic:
> find_isa_irq_apic()
> +                               "failed\n");
>                 return;
> +       } 

as i mentioned it in my mail yesterday, if find_isa_irq_apic() returns
-1 then find_isa_irq_pin() has to return -1 too. But this is obscure and
needs to be documented at least - and your patch is good for
documentation purposes too :-)

Acked-by: Ingo Molnar <mingo@redhat.com>

	Ingo

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

\
 
 \ /
  Last update: 2006-11-14 06:51    [from the cache]
©2003-2008