[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#837588: hppa: Please disable CONFIG_FTRACE for hppa architecture



Control: reopen -1
Control: retitle -1 vmlinux in linux-image packages is unstripped

On Thu, 2016-09-22 at 21:18 +0200, Helge Deller wrote:
> Hi Ben,
> 
> On 19.09.2016 20:56, Ben Hutchings wrote:
> > 
> > On Mon, 2016-09-19 at 20:50 +0200, Helge Deller wrote:
> > > 
> > > On 19.09.2016 19:41, Ben Hutchings wrote:
> > > > 
> > > > On Mon, 2016-09-12 at 20:28 +0100, Ben Hutchings wrote:
> > > > > 
> > > > > On Mon, 2016-09-12 at 18:17 +0200, Helge Deller wrote:
> > > > > > 
> > > > > > 
> > > > > > So, can you please deactivate CONFIG_FTRACE completely for 32- and
> > > > > > 64bit hppa configs?
> > > > > > Is it sufficient to add "CONFIG_FTRACE=n" to
> > > > > > debian/config/hppa/config ?
> > > > > 
> > > > > Done.
> 
> Argh - this didn't fixed the size issues on hppa.
> I was under the wrong assumption, that CONFIG_FTRACE (which was implemented in
> kernel 4.7 for the first time) was the culprit which increased the kernel size.
> This assumption was wrong.
> 
> The attached patch finally fixes the issue.
> It re-enables the CONFIG_FTRACE option (which is set by your upper config files),
> and disables CONFIG_DEBUG_INFO which was now suddenly enabled by the debian
> kernel 4.7 (it wasn't in 4.6!).
> I enabled CONFIG_DEBUG_RODATA to write-protect kernel memory as well.
> 
> The main problem was, that the generated dwarf debug info increases the 
> kernel vmlinux image from 17MB to 97MB in disk size.
> I've compile- and run-tested the patch and built a +b1 kernel. 
> 
> Can you apply the attached patch on top of current git?
> I can commit it myself as well if you like.

This is not the correct fix, nor is this an hppa-specific problem.  We
need to strip debug information out of the kernel image that goes into
linux-image-<version>[-unsigned], and currently we don't do that
explicitly.  Previously we only enabled debug information for x86 and
s390, and for those the upstream build rules already strip debug
information before compressing the kernel.

Here's the current status, so far as I can see:

alpha:       vmlinux.gz, stripped(?) and compressed
arm64:       Image, stripped and uncompressed
armel,armhf: zImage,
stripped and compressed
hppa:        vmlinux, UNstripped and
uncompressed
m68k:        vmlinux.gz, stripped and compressed
mips*:    
  vmlinux, UNstripped and uncompressed
powerpc*:    vmlinux, UNstripped
and uncompressed
s390x:       image, stripped and uncompressed
sh4:      
  zImage, stripped(?) and compressed
sparc64:     zImage, stripped and
compressed
x86:         bzImage, stripped and compressed

(No binary packages of 4.7 are available on ftp.debian-ports.org for
alpha or sh4 yet, so those results are based on reading the respective
Makefiles.)

It seems that we only need a special case in debian/rules.real for
those architectures where we install vmlinux directly.

Ben.

-- 
Ben Hutchings
The first rule of tautology club is the first rule of tautology club.

Attachment: signature.asc
Description: This is a digitally signed message part


Reply to: