All of lore.kernel.org
 help / color / mirror / Atom feed
* nouveau build failure (was Re: Linux 2.6.37-rc2)
@ 2010-11-16 20:45 Linus Torvalds
  2010-11-16 20:50 ` Randy Dunlap
  0 siblings, 1 reply; 6+ messages in thread
From: Linus Torvalds @ 2010-11-16 20:45 UTC (permalink / raw)
  To: Zimny Lech, Martin Peres, Ben Skeggs, Dave Airlie
  Cc: Linux Kernel Mailing List

Looks like commit 34e9d85a1aae ("drm/nouveau: Add temperature support
(vbios parsing, readings, hwmon)") didn't make things properly
dependent on CONFIG_HWMON (or do the appropriate "select HWMON" or
whatever).

Added some collection of hopefully more relevant people to the discussion..

                  Linus

On Tue, Nov 16, 2010 at 12:35 PM, Zimny Lech
<napohybelskurwysynom2010@gmail.com> wrote:
>
>  LD      vmlinux.o
>  MODPOST vmlinux.o
> WARNING: modpost: Found 7 section mismatch(es).
> To see full details build your kernel with:
> 'make CONFIG_DEBUG_SECTION_MISMATCH=y'
>  GEN     .version
>  CHK     include/generated/compile.h
>  UPD     include/generated/compile.h
>  CC      init/version.o
>  LD      init/built-in.o
>  LD      .tmp_vmlinux1
> drivers/built-in.o: In function `nouveau_pm_fini':
> (.text+0xe1857): undefined reference to `hwmon_device_unregister'
> drivers/built-in.o: In function `nouveau_pm_init':
> (.text+0xe1b4d): undefined reference to `hwmon_device_register'
> drivers/built-in.o: In function `nouveau_pm_init':
> (.text+0xe1bef): undefined reference to `hwmon_device_unregister'
> make[1]: *** [.tmp_vmlinux1] Error 1
> make: *** [sub-make] Error 2

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: nouveau build failure (was Re: Linux 2.6.37-rc2)
  2010-11-16 20:45 nouveau build failure (was Re: Linux 2.6.37-rc2) Linus Torvalds
@ 2010-11-16 20:50 ` Randy Dunlap
  2010-11-16 20:58   ` Ben Skeggs
  0 siblings, 1 reply; 6+ messages in thread
From: Randy Dunlap @ 2010-11-16 20:50 UTC (permalink / raw)
  To: Linus Torvalds
  Cc: Zimny Lech, Martin Peres, Ben Skeggs, Dave Airlie,
	Linux Kernel Mailing List

On Tue, 16 Nov 2010 12:45:43 -0800 Linus Torvalds wrote:

> Looks like commit 34e9d85a1aae ("drm/nouveau: Add temperature support
> (vbios parsing, readings, hwmon)") didn't make things properly
> dependent on CONFIG_HWMON (or do the appropriate "select HWMON" or
> whatever).
> 
> Added some collection of hopefully more relevant people to the discussion..
> 
>                   Linus


This build error was reported for linux-next on 2010.1011 and 2010.1025,
with no replies.

Some process fix would be nice to see.


> On Tue, Nov 16, 2010 at 12:35 PM, Zimny Lech
> <napohybelskurwysynom2010@gmail.com> wrote:
> >
> >  LD      vmlinux.o
> >  MODPOST vmlinux.o
> > WARNING: modpost: Found 7 section mismatch(es).
> > To see full details build your kernel with:
> > 'make CONFIG_DEBUG_SECTION_MISMATCH=y'
> >  GEN     .version
> >  CHK     include/generated/compile.h
> >  UPD     include/generated/compile.h
> >  CC      init/version.o
> >  LD      init/built-in.o
> >  LD      .tmp_vmlinux1
> > drivers/built-in.o: In function `nouveau_pm_fini':
> > (.text+0xe1857): undefined reference to `hwmon_device_unregister'
> > drivers/built-in.o: In function `nouveau_pm_init':
> > (.text+0xe1b4d): undefined reference to `hwmon_device_register'
> > drivers/built-in.o: In function `nouveau_pm_init':
> > (.text+0xe1bef): undefined reference to `hwmon_device_unregister'
> > make[1]: *** [.tmp_vmlinux1] Error 1
> > make: *** [sub-make] Error 2
> --


---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: nouveau build failure (was Re: Linux 2.6.37-rc2)
  2010-11-16 20:50 ` Randy Dunlap
@ 2010-11-16 20:58   ` Ben Skeggs
  2010-11-16 21:20     ` Zimny Lech
  2010-11-17  1:42     ` Martin Peres
  0 siblings, 2 replies; 6+ messages in thread
From: Ben Skeggs @ 2010-11-16 20:58 UTC (permalink / raw)
  To: Randy Dunlap
  Cc: Linus Torvalds, Zimny Lech, Martin Peres, Dave Airlie,
	Linux Kernel Mailing List

On Tue, 2010-11-16 at 12:50 -0800, Randy Dunlap wrote:
> On Tue, 16 Nov 2010 12:45:43 -0800 Linus Torvalds wrote:
> 
> > Looks like commit 34e9d85a1aae ("drm/nouveau: Add temperature support
> > (vbios parsing, readings, hwmon)") didn't make things properly
> > dependent on CONFIG_HWMON (or do the appropriate "select HWMON" or
> > whatever).
> > 
> > Added some collection of hopefully more relevant people to the discussion..
> > 
> >                   Linus
> 
> 
> This build error was reported for linux-next on 2010.1011 and 2010.1025,
> with no replies.
> 
> Some process fix would be nice to see.
It's actually been fixed in the nouveau tree, I have to go through and
get fixes out of it and send them to Dave.

Ben.
> 
> 
> > On Tue, Nov 16, 2010 at 12:35 PM, Zimny Lech
> > <napohybelskurwysynom2010@gmail.com> wrote:
> > >
> > >  LD      vmlinux.o
> > >  MODPOST vmlinux.o
> > > WARNING: modpost: Found 7 section mismatch(es).
> > > To see full details build your kernel with:
> > > 'make CONFIG_DEBUG_SECTION_MISMATCH=y'
> > >  GEN     .version
> > >  CHK     include/generated/compile.h
> > >  UPD     include/generated/compile.h
> > >  CC      init/version.o
> > >  LD      init/built-in.o
> > >  LD      .tmp_vmlinux1
> > > drivers/built-in.o: In function `nouveau_pm_fini':
> > > (.text+0xe1857): undefined reference to `hwmon_device_unregister'
> > > drivers/built-in.o: In function `nouveau_pm_init':
> > > (.text+0xe1b4d): undefined reference to `hwmon_device_register'
> > > drivers/built-in.o: In function `nouveau_pm_init':
> > > (.text+0xe1bef): undefined reference to `hwmon_device_unregister'
> > > make[1]: *** [.tmp_vmlinux1] Error 1
> > > make: *** [sub-make] Error 2
> > --
> 
> 
> ---
> ~Randy
> *** Remember to use Documentation/SubmitChecklist when testing your code ***



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: nouveau build failure (was Re: Linux 2.6.37-rc2)
  2010-11-16 20:58   ` Ben Skeggs
@ 2010-11-16 21:20     ` Zimny Lech
  2010-11-17  1:42     ` Martin Peres
  1 sibling, 0 replies; 6+ messages in thread
From: Zimny Lech @ 2010-11-16 21:20 UTC (permalink / raw)
  To: bskeggs
  Cc: Randy Dunlap, Linus Torvalds, Martin Peres, Dave Airlie,
	Linux Kernel Mailing List

Salve!

2010/11/16 Ben Skeggs <bskeggs@redhat.com>:
> On Tue, 2010-11-16 at 12:50 -0800, Randy Dunlap wrote:
>> On Tue, 16 Nov 2010 12:45:43 -0800 Linus Torvalds wrote:
>>
>> > Looks like commit 34e9d85a1aae ("drm/nouveau: Add temperature support
>> > (vbios parsing, readings, hwmon)") didn't make things properly
>> > dependent on CONFIG_HWMON (or do the appropriate "select HWMON" or
>> > whatever).
>> >
>> > Added some collection of hopefully more relevant people to the discussion..
>> >
>> >                   Linus
>>
>>
>> This build error was reported for linux-next on 2010.1011 and 2010.1025,
>> with no replies.
>>
>> Some process fix would be nice to see.
> It's actually been fixed in the nouveau tree, I have to go through and
> get fixes out of it and send them to Dave.

thanks for the info

>
> Ben.
>>
>>
>> > On Tue, Nov 16, 2010 at 12:35 PM, Zimny Lech
>> > <napohybelskurwysynom2010@gmail.com> wrote:
>> > >
>> > >  LD      vmlinux.o
>> > >  MODPOST vmlinux.o
>> > > WARNING: modpost: Found 7 section mismatch(es).
>> > > To see full details build your kernel with:
>> > > 'make CONFIG_DEBUG_SECTION_MISMATCH=y'
>> > >  GEN     .version
>> > >  CHK     include/generated/compile.h
>> > >  UPD     include/generated/compile.h
>> > >  CC      init/version.o
>> > >  LD      init/built-in.o
>> > >  LD      .tmp_vmlinux1
>> > > drivers/built-in.o: In function `nouveau_pm_fini':
>> > > (.text+0xe1857): undefined reference to `hwmon_device_unregister'
>> > > drivers/built-in.o: In function `nouveau_pm_init':
>> > > (.text+0xe1b4d): undefined reference to `hwmon_device_register'
>> > > drivers/built-in.o: In function `nouveau_pm_init':
>> > > (.text+0xe1bef): undefined reference to `hwmon_device_unregister'
>> > > make[1]: *** [.tmp_vmlinux1] Error 1
>> > > make: *** [sub-make] Error 2
>> > --
>>
>>
>> ---
>> ~Randy
>> *** Remember to use Documentation/SubmitChecklist when testing your code ***
>
>
>



-- 
Slawa!
N.P.S.

Les fleurs du mal unfold
Comme les fleurs du mal
Dark demons of my soul
Un amour fatal

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: nouveau build failure (was Re: Linux 2.6.37-rc2)
  2010-11-16 20:58   ` Ben Skeggs
  2010-11-16 21:20     ` Zimny Lech
@ 2010-11-17  1:42     ` Martin Peres
  2010-11-17  1:43       ` Ben Skeggs
  1 sibling, 1 reply; 6+ messages in thread
From: Martin Peres @ 2010-11-17  1:42 UTC (permalink / raw)
  To: bskeggs
  Cc: Randy Dunlap, Linus Torvalds, Zimny Lech, Dave Airlie,
	Linux Kernel Mailing List

Le 16/11/2010 21:58, Ben Skeggs a écrit :
> On Tue, 2010-11-16 at 12:50 -0800, Randy Dunlap wrote:
>> On Tue, 16 Nov 2010 12:45:43 -0800 Linus Torvalds wrote:
>>
>>> Looks like commit 34e9d85a1aae ("drm/nouveau: Add temperature support
>>> (vbios parsing, readings, hwmon)") didn't make things properly
>>> dependent on CONFIG_HWMON (or do the appropriate "select HWMON" or
>>> whatever).
>>>
>>> Added some collection of hopefully more relevant people to the discussion..
>>>
>>>                    Linus
>>
>> This build error was reported for linux-next on 2010.1011 and 2010.1025,
>> with no replies.
>>
>> Some process fix would be nice to see.
> It's actually been fixed in the nouveau tree, I have to go through and
> get fixes out of it and send them to Dave.
>
> Ben.
Thanks Ben, I should have checked my fix actually went in David's 
drm-fixes branch. I'll try to be more vigilent on this in the future.

Martin

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: nouveau build failure (was Re: Linux 2.6.37-rc2)
  2010-11-17  1:42     ` Martin Peres
@ 2010-11-17  1:43       ` Ben Skeggs
  0 siblings, 0 replies; 6+ messages in thread
From: Ben Skeggs @ 2010-11-17  1:43 UTC (permalink / raw)
  To: Martin Peres
  Cc: Randy Dunlap, Linus Torvalds, Zimny Lech, Dave Airlie,
	Linux Kernel Mailing List

On Wed, 2010-11-17 at 02:42 +0100, Martin Peres wrote:
> Le 16/11/2010 21:58, Ben Skeggs a écrit :
> > On Tue, 2010-11-16 at 12:50 -0800, Randy Dunlap wrote:
> >> On Tue, 16 Nov 2010 12:45:43 -0800 Linus Torvalds wrote:
> >>
> >>> Looks like commit 34e9d85a1aae ("drm/nouveau: Add temperature support
> >>> (vbios parsing, readings, hwmon)") didn't make things properly
> >>> dependent on CONFIG_HWMON (or do the appropriate "select HWMON" or
> >>> whatever).
> >>>
> >>> Added some collection of hopefully more relevant people to the discussion..
> >>>
> >>>                    Linus
> >>
> >> This build error was reported for linux-next on 2010.1011 and 2010.1025,
> >> with no replies.
> >>
> >> Some process fix would be nice to see.
> > It's actually been fixed in the nouveau tree, I have to go through and
> > get fixes out of it and send them to Dave.
> >
> > Ben.
> Thanks Ben, I should have checked my fix actually went in David's 
> drm-fixes branch. I'll try to be more vigilent on this in the future.
Well I'm partially to blame.  I'd prefer, where possible, for patches to
go through me (so, the nouveau tree) and then get them to Dave.  I've
been busy working on other things lately, and missed it too :)

Ben.
> 
> Martin



^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2010-11-17  1:48 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-11-16 20:45 nouveau build failure (was Re: Linux 2.6.37-rc2) Linus Torvalds
2010-11-16 20:50 ` Randy Dunlap
2010-11-16 20:58   ` Ben Skeggs
2010-11-16 21:20     ` Zimny Lech
2010-11-17  1:42     ` Martin Peres
2010-11-17  1:43       ` Ben Skeggs

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.