RE: [PATCH] cpqarray update

From: Wiran, Francis (francis.wiran_at_hp.com)
Date: 01/29/04

  • Next message: Thomas Davis: "Can't boot a 2.6 kernel.." 
    Date:	Thu, 29 Jan 2004 13:20:02 -0600
To: "Jeff Garzik" <jgarzik@pobox.com>

    Ok. I'll make a patch when the behaviour changes, as per Greg's:

    >Yeah, I don't really like it either, but figured it was a 2.7 task to
    >clean it up properly.

    At 2.6.1, it still returns count.

    > -----Original Message-----
    > From: Jeff Garzik [mailto:jgarzik@pobox.com]
    > Sent: Thursday, January 29, 2004 10:57 AM
    > To: Wiran, Francis
    > Cc: Greg KH; Hollis Blanchard; Marcelo Tosatti; Linux Kernel
    > Mailing List
    > Subject: Re: [PATCH] cpqarray update
    >
    >
    > Wiran, Francis wrote:
    > > check for negative value? That's odd. The
    > pci_register_driver() in my
    > > copy of 2.4.24 kernel (drivers/pci/pci.c) looks something like this:
    > >
    > > {
    > > count = 0;
    > >
    > > for ....
    > > count += foo();
    > >
    > > return count;
    > > }
    > >
    > >
    > > Or will it change in the future? The patch that I sent was
    > based on what
    > > is in the current kernel.
    >
    >
    > Correct, Greg was referring to 2.6.x behavior of
    > pci_register_driver(),
    > which changed from 2.4.x.
    >
    > Jeff
    >
    >
    >
    >
    -
    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: Thomas Davis: "Can't boot a 2.6 kernel.."

    Relevant Pages

    • RE: [PATCH] cpqarray update
      ... > To: Wiran, Francis ... > Cc: Linux Kernel Mailing List ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • RE: [PATCH] cpqarray update
      ... > To: Wiran, Francis ... > controllers found, or returns number of controllers found. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)