Re: initdata changes in 2.3.99-pre9-pre1 (was Re: bug in emu10k1 under...)

From: Jeff Garzik (jgarzik@mandrakesoft.com)
Date: Sat May 13 2000 - 15:12:20 EDT

  • Next message: f5ibh: "VFS: Busy inodes after unmount..."

    Russell King wrote:
    >
    > I'm not the culpret, but I wonder if you've got the wrong end of the stick.
    >
    > Jeff Garzik writes:
    > > __devinitdata should NOT be used unless you are certain the driver is a
    > > hotplug driver. Otherwise __initdata should be used. Similar rules for
    > > __init and __devinit.
    > >
    > > The patch in 2.3.99-pre9-pre1 blindly moves everything to __devinitdata,
    > > which causes memory wastage in non-hotplug kernels (ie. the majority of
    > > the kernels out there).
    >
    > Are you *sure* about this?
    [...]
    > Therefore, there is no difference to "non-hotplug" kernels. The only
    > difference is to proper hotplug kernels.

    I plead -ENOCAFFEINE :)

    The first paragraph is correct -- don't use __devinitdata unless the
    hardware is hotplug -- but you are correct, the second paragraph should
    state that memory is wasted only on proper hotplug kernels, the minority
    of the kernels out there.

    Regards,

            Jeff

    -- 
    Jeff Garzik              | Liberty is always dangerous, but
    Building 1024            | it is the safest thing we have.
    MandrakeSoft, Inc.       |      -- Harry Emerson Fosdick
    

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



    This archive was generated by hypermail 2b29 : Sat May 13 2000 - 15:24:30 EDT