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@xxxxxxxxxxxx>
> 
> 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 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/