mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* kbuild patch (20a468b51325b3636785a8ca0047ae514b39cbd5) breaks parallels-config
@ 2006-06-05 16:49 Thomas Glanzmann
  2006-06-05 21:31 ` Sam Ravnborg
  0 siblings, 1 reply; 5+ messages in thread
From: Thomas Glanzmann @ 2006-06-05 16:49 UTC (permalink / raw)
  To: Sam Ravnborg, LKML

Hello Sam,
I use parallels, a hypervisor based virtual machine, for Linux which
ships with binary kernel modules and a bit of gluecode. After your
commit 20a468b51325b3636785a8ca0047ae514b39cbd5 in the linux kernel the
compilation of this kernel modules ends up in an endless-loop. I don't
think that this is a problem of the kernel build system but the way
paralles does abuse the kernel build system (they use autoconf to
generate a makefile which looks obvious broken). However I tracked the
offending delta down to the following:

(mini) [~/work/linux-2.6] git diff -R
diff --git a/scripts/Makefile.build b/scripts/Makefile.build
index acd3b96..e48e60d 100644
--- a/scripts/Makefile.build
+++ b/scripts/Makefile.build
@@ -10,11 +10,12 @@ __build:
 # Read .config if it exist, otherwise ignore
 -include .config

+include scripts/Kbuild.include
+
 # The filename Kbuild has precedence over Makefile
 kbuild-dir := $(if $(filter /%,$(src)),$(src),$(srctree)/$(src))
 include $(if $(wildcard $(kbuild-dir)/Kbuild), $(kbuild-dir)/Kbuild, $(kbuild-dir)/Makefile)

-include scripts/Kbuild.include
 include scripts/Makefile.lib

 ifdef host-progs

If you reverse apply this delta of your patch, it works again. I don't
get what is going wrong here, but maybe you want to have a look at it.

To reproduce:

        * Go to http://www.parallels.com/en/download/ and download the packages for your distribution

        cd /usr/lib/parallels
        ./configure
        make

if you do this with a recent kernel version (newer than
20a468b51325b3636785a8ca0047ae514b39cbd5) it ends in an endless-loop
otherwise it works.

The broken parallels makefiles are in:

        /usr/lib/parallels/drivers/drv_net/linux/Makefile
        /usr/lib/parallels/drivers/drv_net/Makefile
        /usr/lib/parallels/drivers/drv_virtualnic/Makefile
        /usr/lib/parallels/drivers/drv_main/Makefile
        /usr/lib/parallels/drivers/hypervisor/Makefile        <<<<< This is the first one which breaks. The other follow.
        /usr/lib/parallels/drivers/Makefile

I also created a website for my findings: http://thomas.glanzmann.de/parallels/

        Thomas [who thanks Linus for git bisect]

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

end of thread, other threads:[~2006-06-06 18:35 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-06-05 16:49 kbuild patch (20a468b51325b3636785a8ca0047ae514b39cbd5) breaks parallels-config Thomas Glanzmann
2006-06-05 21:31 ` Sam Ravnborg
2006-06-05 22:27   ` Thomas Glanzmann
2006-06-06 18:14     ` Sam Ravnborg
2006-06-06 18:35       ` Thomas Glanzmann

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®