lkml.org 
[lkml]   [2006]   [Mar]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] powerpc: Move pSeries firmware feature setup into platforms/pseries
    On Thu, Mar 30, 2006 at 09:39:15AM +1100, Michael Ellerman wrote:
    > On Wed, 2006-03-29 at 14:52 -0500, Dave Jones wrote:
    > > On Thu, Mar 23, 2006 at 07:14:49AM +0000, Linux Kernel wrote:
    > > > commit 1965746bce49ddf001af52c7985e16343c768021
    > > > tree d311fce31613545f3430582322d66411566f1863
    > > > parent 0941d57aa7034ef7010bd523752c2e3bee569ef1
    > > > author Michael Ellerman <michael@ellerman.id.au> Fri, 10 Feb 2006 15:47:36 +1100
    > > > committer Paul Mackerras <paulus@samba.org> Fri, 10 Feb 2006 16:52:03 +1100
    > > >
    > > > [PATCH] powerpc: Move pSeries firmware feature setup into platforms/pseries
    > > >
    > > > Currently we have some stuff in firmware.h and kernel/firmware.c that is
    > > > #ifdef CONFIG_PPC_PSERIES. Move it all into platforms/pseries.
    > >
    > > This (or one of the other firmware patches, I've not narrowed it down that close)
    > > breaks ppc64 oprofile.
    > >
    > > modpost now complains with..
    > >
    > > kernel/arch/powerpc/oprofile/oprofile.ko needs unknown symbol ppc64_firmware_features
    >
    > Hi Dave,
    >
    > I'm not sure about that patch, but I think the firmware feature stuff
    > has been broken for modules for a while, we weren't exporting
    > ppc64_firmware_features anywhere.

    It's bizarre that it's only just started complaining about it.
    (I get a nice mail from our buildsystem when a kernel package is built with
    unresolved symbols), and the 2.6.16.1 kernel happily sails through.

    > The fix just got merged in the last day or so:
    > http://kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=d0160bf0b3e87032be8e85f80ddd2f18e107b86f

    still broken with that diff. (I just tested -git15)

    firmware_has_feature() needs fixing up.

    Dave


    --
    http://www.codemonkey.org.uk
    -
    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-03-30 00:54    [W:0.024 / U:2.732 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site