[PATCH] Add vmlinux to kernel rpm - updated description

[PATCH] Add vmlinux to kernel rpm - updated description

[Josh Hunt]

Resending with updated description:

We are building an automated system to test kernels weekly and need to
provide an rpm to our QA dept.  We would like to use the ability to create
kernel rpms already in the kernel's Makefile, but need the vmlinux file
included in the rpm for later debugging.  This patch adds a compressed vmlinux to
the kernel rpm when doing a make rpm-pkg or binrpm-pkg and upon install
places the vmlinux file in /boot.

Signed-off-by: Josh Hunt <josh@scalex86.org>

Index: linux-2.6.28/scripts/package/mkspec
===================================================================
--- linux-2.6.28.orig/scripts/package/mkspec    2008-12-24
15:26:37.000000000 -0800
+++ linux-2.6.28/scripts/package/mkspec    2009-01-21
16:53:03.000000000 -0800
@@ -86,6 +86,14 @@ echo "%endif"
 echo 'cp System.map $RPM_BUILD_ROOT'"/boot/System.map-$KERNELRELEASE"

 echo 'cp .config $RPM_BUILD_ROOT'"/boot/config-$KERNELRELEASE"
+
+echo "%ifnarch ppc64"
+echo 'cp vmlinux vmlinux.orig'
+echo 'bzip2 -9 vmlinux'
+echo 'mv vmlinux.bz2
$RPM_BUILD_ROOT'"/boot/vmlinux-$KERNELRELEASE.bz2"
+echo 'mv vmlinux.orig vmlinux'
+echo "%endif"
+
 echo ""
 echo "%clean"
 echo '#echo -rf $RPM_BUILD_ROOT'

Re: [PATCH] Add vmlinux to kernel rpm - updated description

[Sam Ravnborg]

On Wed, Feb 11, 2009 at 09:10:57PM -0800, Josh Hunt wrote:
> Resending with updated description:
> 
> We are building an automated system to test kernels weekly and need to
> provide an rpm to our QA dept.  We would like to use the ability to create
> kernel rpms already in the kernel's Makefile, but need the vmlinux file
> included in the rpm for later debugging.  This patch adds a compressed vmlinux to
> the kernel rpm when doing a make rpm-pkg or binrpm-pkg and upon install
> places the vmlinux file in /boot.
> 
> Signed-off-by: Josh Hunt <josh@scalex86.org>

Applied, but...

> 
> Index: linux-2.6.28/scripts/package/mkspec
> ===================================================================
> --- linux-2.6.28.orig/scripts/package/mkspec    2008-12-24
> 15:26:37.000000000 -0800
> +++ linux-2.6.28/scripts/package/mkspec    2009-01-21
> 16:53:03.000000000 -0800
> @@ -86,6 +86,14 @@ echo "%endif"
>  echo 'cp System.map $RPM_BUILD_ROOT'"/boot/System.map-$KERNELRELEASE"
> 
>  echo 'cp .config $RPM_BUILD_ROOT'"/boot/config-$KERNELRELEASE"
> +
> +echo "%ifnarch ppc64"
> +echo 'cp vmlinux vmlinux.orig'
why is this relevant for all architectures except ppc64?
This is not addressed neither in a comment nor in the changelog.

	Sam

Re: [PATCH] Add vmlinux to kernel rpm - updated description

[Josh Hunt]

On Sun, Feb 15, 2009 at 09:01:36AM +0100, Sam Ravnborg wrote:
> On Wed, Feb 11, 2009 at 09:10:57PM -0800, Josh Hunt wrote:
> > Resending with updated description:
> > 
> > We are building an automated system to test kernels weekly and need to
> > provide an rpm to our QA dept.  We would like to use the ability to create
> > kernel rpms already in the kernel's Makefile, but need the vmlinux file
> > included in the rpm for later debugging.  This patch adds a compressed vmlinux to
> > the kernel rpm when doing a make rpm-pkg or binrpm-pkg and upon install
> > places the vmlinux file in /boot.
> > 
> > Signed-off-by: Josh Hunt <josh@scalex86.org>
> 
> Applied, but...
> 
> > 
> > Index: linux-2.6.28/scripts/package/mkspec
> > ===================================================================
> > --- linux-2.6.28.orig/scripts/package/mkspec    2008-12-24
> > 15:26:37.000000000 -0800
> > +++ linux-2.6.28/scripts/package/mkspec    2009-01-21
> > 16:53:03.000000000 -0800
> > @@ -86,6 +86,14 @@ echo "%endif"
> >  echo 'cp System.map $RPM_BUILD_ROOT'"/boot/System.map-$KERNELRELEASE"
> > 
> >  echo 'cp .config $RPM_BUILD_ROOT'"/boot/config-$KERNELRELEASE"
> > +
> > +echo "%ifnarch ppc64"
> > +echo 'cp vmlinux vmlinux.orig'
> why is this relevant for all architectures except ppc64?
> This is not addressed neither in a comment nor in the changelog.

I was under the impression that the ppc64 vmlinuz file could be
unzipped and used for debugging and so adding it to /boot would be
redundant.  Please let me know if this is not the case and I will
submit an updated patch.

Thanks
Josh