linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PULL] kbuild fixes for -rc1
@ 2008-10-25 21:11 Sam Ravnborg
  2008-10-25 21:59 ` kbuild maintainer news [Was: kbuild fixes for -rc1] Sam Ravnborg
  2008-10-26  5:12 ` [PATCH] fix allmodconfig breakage Al Viro
  0 siblings, 2 replies; 6+ messages in thread
From: Sam Ravnborg @ 2008-10-25 21:11 UTC (permalink / raw)
  To: Linus Torvalds, Andrew Morton, linux-kbuild, LKML
  Cc: Evgeniy Manachkin, Alan Cox, Stable

Hi Linus.

Please pull the following two fixes for kbuild.
One fix for make rpm (which is also stable material)
And one fix so we are more robust for wrong asm symlinks

None of these has been in -next but they are trivial and
I would like to see them applied soon.

Thanks,

	Sam

The following changes since commit e013e13bf605b9e6b702adffbe2853cfc60e7806:
  Jens Axboe (1):
        libata: fix bug with non-ncq devices

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/sam/kbuild-fixes.git master

Evgeniy Manachkin (1):
      kbuild: mkspec - fix build rpm

Sam Ravnborg (1):
      kbuild: improve check-symlink

 Makefile               |    5 +++++
 scripts/package/mkspec |    3 +++
 2 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/Makefile b/Makefile
index 8e54051..cacb624 100644
--- a/Makefile
+++ b/Makefile
@@ -961,6 +961,7 @@ export CPPFLAGS_vmlinux.lds += -P -C -U$(ARCH)
 
 # The asm symlink changes when $(ARCH) changes.
 # Detect this and ask user to run make mrproper
+# If asm is a stale symlink (point to dir that does not exist) remove it
 define check-symlink
 	set -e;                                                            \
 	if [ -L include/asm ]; then                                        \
@@ -970,6 +971,10 @@ define check-symlink
 			echo "       set ARCH or save .config and run 'make mrproper' to fix it";             \
 			exit 1;                                            \
 		fi;                                                        \
+		test -e $$asmlink || rm include/asm;                       \
+	elif [ -d include/asm ]; then                                      \
+		echo "ERROR: $@ is a directory but a symlink was expected";\
+		exit 1;                                                    \
 	fi
 endef
 
diff --git a/scripts/package/mkspec b/scripts/package/mkspec
index ffd61fe..113d373 100755
--- a/scripts/package/mkspec
+++ b/scripts/package/mkspec
@@ -64,8 +64,10 @@ fi
 echo "%install"
 echo "%ifarch ia64"
 echo 'mkdir -p $RPM_BUILD_ROOT/boot/efi $RPM_BUILD_ROOT/lib/modules'
+echo 'mkdir -p $RPM_BUILD_ROOT/boot/efi $RPM_BUILD_ROOT/lib/firmware'
 echo "%else"
 echo 'mkdir -p $RPM_BUILD_ROOT/boot $RPM_BUILD_ROOT/lib/modules'
+echo 'mkdir -p $RPM_BUILD_ROOT/boot $RPM_BUILD_ROOT/lib/firmware'
 echo "%endif"
 
 echo 'INSTALL_MOD_PATH=$RPM_BUILD_ROOT make %{_smp_mflags} modules_install'
@@ -92,5 +94,6 @@ echo "%files"
 echo '%defattr (-, root, root)'
 echo "%dir /lib/modules"
 echo "/lib/modules/$KERNELRELEASE"
+echo "/lib/firmware"
 echo "/boot/*"
 echo ""

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

* kbuild maintainer news [Was: kbuild fixes for -rc1]
  2008-10-25 21:11 [PULL] kbuild fixes for -rc1 Sam Ravnborg
@ 2008-10-25 21:59 ` Sam Ravnborg
  2008-10-26  1:06   ` Frans Pop
  2008-10-26  5:12 ` [PATCH] fix allmodconfig breakage Al Viro
  1 sibling, 1 reply; 6+ messages in thread
From: Sam Ravnborg @ 2008-10-25 21:59 UTC (permalink / raw)
  To: LKML, linux-kbuild

In case anyone was wondering...
I have been busy moving to a new house the last couple of months.

As we are now well installed (as in the important stuff is finished)
I can now spend some time as kbuild maintainer again.

But in order not to be buried alive in email I deleted
all mails I received in my absense period.

So if you have any pending kbuild patches around please
resend them to me with a cc: to at least linux-kbuild.

But please do not expect promt replies there is still
a shitload of things to do.

	Sam
 

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

* Re: kbuild maintainer news [Was: kbuild fixes for -rc1]
  2008-10-25 21:59 ` kbuild maintainer news [Was: kbuild fixes for -rc1] Sam Ravnborg
@ 2008-10-26  1:06   ` Frans Pop
  0 siblings, 0 replies; 6+ messages in thread
From: Frans Pop @ 2008-10-26  1:06 UTC (permalink / raw)
  To: Sam Ravnborg; +Cc: linux-kernel, linux-kbuild

[-- Attachment #1: Type: text/plain, Size: 400 bytes --]

Sam Ravnborg wrote:
> So if you have any pending kbuild patches around please
> resend them to me with a cc: to at least linux-kbuild.

A similar patch to what you just pushed for rpm's to avoid package 
installation problems due to firmware is waiting also for deb-pkg.
Please push the patch in [1] to Linus and stable. Original mail attached.

Cheers,
FJP

[1] http://lkml.org/lkml/2008/9/13/102



[-- Attachment #2: Jonathan McDowell <noodles@earth.li>: Re: [PATCH] Fixup deb-pkg target to generate separate firmware deb. --]
[-- Type: message/rfc822, Size: 4540 bytes --]

From: Jonathan McDowell <noodles@earth.li>
To: Frans Pop <elendil@planet.nl>
Cc: linux-kernel@vger.kernel.org, linux-kbuild@vger.kernel.org
Subject: Re: [PATCH] Fixup deb-pkg target to generate separate firmware deb.
Date: Sat, 13 Sep 2008 17:08:31 +0100
Message-ID: <20080913160831.GE30927@earth.li>

[Adding linux-kbuild as that seems to be the closest to a maintainer for
scripts/package/]

On Sat, Sep 13, 2008 at 10:35:18AM +0200, Frans Pop wrote:
> On Friday 12 September 2008, Jonathan McDowell wrote:
> > The below is a simplistic fix for "make deb-pkg"; it splits the
> > firmware out to a linux-firmware-image package and adds an
> > (unversioned) Suggests to the linux package for this firmware.
> >
> > Frans, does this solve your problem in an acceptable manner?
> 
> Yes it does. Thanks for the patch Jonathan.
> Tested both with kernel configs that result in firmware and that do not 
> result in firmware.
> 
> A few minor nitpicks below.

Ta, I've fixed both the indents. I've also changed the firmware package
to be arch: all.

> > Signed-Off-By: Jonathan McDowell <noodles@earth.li>
> Acked-by: Frans Pop <elendil@planet.nl>
 
-----
diff --git a/scripts/package/builddeb b/scripts/package/builddeb
index ba6bf5d..1264b8e 100644
--- a/scripts/package/builddeb
+++ b/scripts/package/builddeb
@@ -15,15 +15,18 @@ set -e
 version=$KERNELRELEASE
 revision=`cat .version`
 tmpdir="$objtree/debian/tmp"
+fwdir="$objtree/debian/fwtmp"
 packagename=linux-$version
+fwpackagename=linux-firmware-image
 
 if [ "$ARCH" == "um" ] ; then
 	packagename=user-mode-linux-$version
 fi
 
 # Setup the directory structure
-rm -rf "$tmpdir"
+rm -rf "$tmpdir" "$fwdir"
 mkdir -p "$tmpdir/DEBIAN" "$tmpdir/lib" "$tmpdir/boot"
+mkdir -p "$fwdir/DEBIAN" "$fwdir/lib"
 if [ "$ARCH" == "um" ] ; then
 	mkdir -p "$tmpdir/usr/lib/uml/modules/$version" "$tmpdir/usr/share/doc/$packagename" "$tmpdir/usr/bin"
 fi
@@ -107,6 +110,7 @@ Standards-Version: 3.6.1
 
 Package: $packagename
 Provides: kernel-image-$version, linux-image-$version
+Suggests: $fwpackagename
 Architecture: any
 Description: Linux kernel, version $version
  This package contains the Linux kernel, modules and corresponding other
@@ -118,8 +122,24 @@ fi
 chown -R root:root "$tmpdir"
 chmod -R go-w "$tmpdir"
 
+# Do we have firmware? Move it out of the way and build it into a package.
+if [ -e "$tmpdir/lib/firmware" ]; then
+	mv "$tmpdir/lib/firmware" "$fwdir/lib/"
+
+	cat <<EOF >> debian/control
+
+Package: $fwpackagename
+Architecture: all
+Description: Linux kernel firmware, version $version
+ This package contains firmware from the Linux kernel, version $version
+EOF
+
+	dpkg-gencontrol -isp -p$fwpackagename -P"$fwdir"
+	dpkg --build "$fwdir" ..
+fi
+
 # Perform the final magic
-dpkg-gencontrol -isp
+dpkg-gencontrol -isp -p$packagename
 dpkg --build "$tmpdir" ..
 
 exit 0
-----

J.

-- 
"Just because I'll spend 4 hours automating a task that takes 4 minutes
by hand doesn't mean I'm not lazy." -- Mike Sphar, asr


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

* [PATCH] fix allmodconfig breakage
  2008-10-25 21:11 [PULL] kbuild fixes for -rc1 Sam Ravnborg
  2008-10-25 21:59 ` kbuild maintainer news [Was: kbuild fixes for -rc1] Sam Ravnborg
@ 2008-10-26  5:12 ` Al Viro
  2008-10-26  5:32   ` Alexey Dobriyan
  2008-10-27 14:40   ` Roman Zippel
  1 sibling, 2 replies; 6+ messages in thread
From: Al Viro @ 2008-10-26  5:12 UTC (permalink / raw)
  To: Linus Torvalds; +Cc: Sam Ravnborg, Roman Zippel, linux-kernel

	If you use KCONFIG_ALLCONFIG (even with empty file) you get
broken allmodconfig/allyesconfig; CONFIG_MODULES gets turned off,
with obvious massive fallout.

	Breakage had been introduced when conf_set_all_new_symbols()
got used for allmodconfig et.al.

	What happens is that sym_calc_value(modules_sym) done in
conf_read_simple() sets SYMBOL_VALID on both modules_sym and MODULES.
When we get to conf_set_all_new_symbols(), we set sym->def[S_DEF_USER]
on everything, but it has no effect on sym->curr for the symbols that
already have SYMBOL_VALID - these are stuck.

	Solution: use sym_clear_all_valid() in there.  Note that it
makes reevaluation of modules_sym redundant - sym_clear_all_valid()
will do that itself.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
---
diff --git a/scripts/kconfig/confdata.c b/scripts/kconfig/confdata.c
index b91cf24..830d9ea 100644
--- a/scripts/kconfig/confdata.c
+++ b/scripts/kconfig/confdata.c
@@ -852,8 +852,7 @@ void conf_set_all_new_symbols(enum conf_def_mode mode)
 
 	}
 
-	if (modules_sym)
-		sym_calc_value(modules_sym);
+	sym_clear_all_valid();
 
 	if (mode != def_random)
 		return;

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

* Re: [PATCH] fix allmodconfig breakage
  2008-10-26  5:12 ` [PATCH] fix allmodconfig breakage Al Viro
@ 2008-10-26  5:32   ` Alexey Dobriyan
  2008-10-27 14:40   ` Roman Zippel
  1 sibling, 0 replies; 6+ messages in thread
From: Alexey Dobriyan @ 2008-10-26  5:32 UTC (permalink / raw)
  To: Al Viro; +Cc: Linus Torvalds, Sam Ravnborg, Roman Zippel, linux-kernel

On Sun, Oct 26, 2008 at 05:12:34AM +0000, Al Viro wrote:
> 	If you use KCONFIG_ALLCONFIG (even with empty file) you get
> broken allmodconfig/allyesconfig; CONFIG_MODULES gets turned off,
> with obvious massive fallout.
> 
> 	Breakage had been introduced when conf_set_all_new_symbols()
> got used for allmodconfig et.al.
> 
> 	What happens is that sym_calc_value(modules_sym) done in
> conf_read_simple() sets SYMBOL_VALID on both modules_sym and MODULES.
> When we get to conf_set_all_new_symbols(), we set sym->def[S_DEF_USER]
> on everything, but it has no effect on sym->curr for the symbols that
> already have SYMBOL_VALID - these are stuck.
> 
> 	Solution: use sym_clear_all_valid() in there.  Note that it
> makes reevaluation of modules_sym redundant - sym_clear_all_valid()
> will do that itself.

Fixes http://bugzilla.kernel.org/show_bug.cgi?id=11512

> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
> ---
> diff --git a/scripts/kconfig/confdata.c b/scripts/kconfig/confdata.c
> index b91cf24..830d9ea 100644
> --- a/scripts/kconfig/confdata.c
> +++ b/scripts/kconfig/confdata.c
> @@ -852,8 +852,7 @@ void conf_set_all_new_symbols(enum conf_def_mode mode)
>  
>  	}
>  
> -	if (modules_sym)
> -		sym_calc_value(modules_sym);
> +	sym_clear_all_valid();
>  
>  	if (mode != def_random)
>  		return;

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

* Re: [PATCH] fix allmodconfig breakage
  2008-10-26  5:12 ` [PATCH] fix allmodconfig breakage Al Viro
  2008-10-26  5:32   ` Alexey Dobriyan
@ 2008-10-27 14:40   ` Roman Zippel
  1 sibling, 0 replies; 6+ messages in thread
From: Roman Zippel @ 2008-10-27 14:40 UTC (permalink / raw)
  To: Al Viro; +Cc: Linus Torvalds, Sam Ravnborg, linux-kernel

Hi,

On Sun, 26 Oct 2008, Al Viro wrote:

> 	If you use KCONFIG_ALLCONFIG (even with empty file) you get
> broken allmodconfig/allyesconfig; CONFIG_MODULES gets turned off,
> with obvious massive fallout.
> 
> 	Breakage had been introduced when conf_set_all_new_symbols()
> got used for allmodconfig et.al.
> 
> 	What happens is that sym_calc_value(modules_sym) done in
> conf_read_simple() sets SYMBOL_VALID on both modules_sym and MODULES.
> When we get to conf_set_all_new_symbols(), we set sym->def[S_DEF_USER]
> on everything, but it has no effect on sym->curr for the symbols that
> already have SYMBOL_VALID - these are stuck.
> 
> 	Solution: use sym_clear_all_valid() in there.  Note that it
> makes reevaluation of modules_sym redundant - sym_clear_all_valid()
> will do that itself.
> 
> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>

Thanks for looking into this.

Signed-off-by: Roman Zippel <zippel@linux-m68k.org>

bye, Roman

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

end of thread, other threads:[~2008-10-27 14:41 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-10-25 21:11 [PULL] kbuild fixes for -rc1 Sam Ravnborg
2008-10-25 21:59 ` kbuild maintainer news [Was: kbuild fixes for -rc1] Sam Ravnborg
2008-10-26  1:06   ` Frans Pop
2008-10-26  5:12 ` [PATCH] fix allmodconfig breakage Al Viro
2008-10-26  5:32   ` Alexey Dobriyan
2008-10-27 14:40   ` Roman Zippel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).