RE: 2.4.28-pre3: broken ips update

From: Arjan van de Ven (arjanv_at_redhat.com)
Date: 09/13/04

  • Next message: Richard B. Johnson: "Re: Fwd: Calling syscalls from x86-64 kernel results in a crash on Opteron machines"
    To: "Hammer, Jack" <Jack_Hammer@adaptec.com>
    Date:	Mon, 13 Sep 2004 16:46:25 +0200
    
    
    

    On Mon, 2004-09-13 at 16:28, Hammer, Jack wrote:
    > Marcelo,

    > --- linux.orig/drivers/scsi/ips.h Mon Sep 13 09:40:04 2004
    > +++ linux/drivers/scsi/ips.h Mon Sep 13 09:40:27 2004
    > @@ -97,7 +97,7 @@
    >
    > #if LINUX_VERSION_CODE < KERNEL_VERSION(2,5,0)
    >
    > - #ifndef irqreturn_t
    > + #if LINUX_VERSION_CODE < KERNEL_VERSION(2,4,23)
    > typedef void irqreturn_t;
    > #endif
    >

    your fix is wrong.
    you should nuke all version codes, and ONLY ifndef for IRQ_NONE and
    nothing else. That's not just cosmetics, that's also to make keep
    working with those exact vendor kernels you claim to have tested on :)

    
    

    -
    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/



  • Next message: Richard B. Johnson: "Re: Fwd: Calling syscalls from x86-64 kernel results in a crash on Opteron machines"

    Relevant Pages

    • Re: sched_domains + NUMA issue
      ... > We are seeing errors in the sched domains debug code when SMT + NUMA is ... #ifndef PENALTY_FOR_NODE_WITH_CPUS ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: SMP Panic caused by [PATCH] sched: consolidate sched domains
      ... >> This patch causes an immediate panic when the secondary processors come ... #ifndef node_to_first_cpu ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: BIOS overwritten during resume (was: Re: Asus L5D resume on battery power)
      ... > You could get subtle and reuse some other pageflag. ... #ifndef CONFIG_DISCONTIGMEM ... Nigel Cunningham ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • GNUPG - KERNEL
      ... authenticity of the codes and sent messages. ... We would make available our public keys in a servant of keys. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: Possible bug in keyboard.c (2.6.10)
      ... Nevertheless I disagree a bit. ... "raw mode" is by definition the mode ... where scan codes are passed unmodified to user space. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)