[lkml]   [1998]   [Sep]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: 2.1.121 breaks compilation, initmem_freed undefined

    On Sun, 13 Sep 1998, Alan Cox wrote:

    > There is a function buried in the depths of the code that can be called
    > from both modules and at init time. If its called at init time it
    > displays the penguin logo everyone wants. If its called later it doesnt.
    > So either there is a "penguin has been freed" flag or you pass "is being
    > called by a module" through every single layer of code to that deeply
    > buried function.

    isnt it cleaner to add something like this to chdev_init:

    #if defined (CONFIG_FB)
    + show_fb_logos();

    this is never executed during module init. The solution is IMO not to push
    a flag deep within some function, but to split the function into two
    orthogonal parts. This needs no per-fbdriver #ifdefs.

    -- mingo

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:44    [W:0.020 / U:3.540 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site