linux-mips
[Top] [All Lists]

Re: [PATCH] MIPS: Octeon: add option to ignore PT_NOTE section

To: Florian Fainelli <florian@openwrt.org>
Subject: Re: [PATCH] MIPS: Octeon: add option to ignore PT_NOTE section
From: Philby John <pjohn@mvista.com>
Date: Fri, 15 Apr 2011 14:25:57 +0530
Cc: David Daney <ddaney@caviumnetworks.com>, linux-mips@linux-mips.org
In-reply-to: <201104151024.07859.florian@openwrt.org>
Original-recipient: rfc822;linux-mips@linux-mips.org
References: <1302710833.14458.1.camel@localhost.localdomain> <4DA5DF7A.1030207@caviumnetworks.com> <201104151024.07859.florian@openwrt.org>
Sender: linux-mips-bounce@linux-mips.org
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.13) Gecko/20101209 Fedora/3.1.7-0.35.b3pre.fc14 Lightning/1.0b2 Thunderbird/3.1.7
On 04/15/2011 01:54 PM, Florian Fainelli wrote:
> Hello,
> 
> On Wednesday 13 April 2011 19:38:02 David Daney wrote:
>> On 04/13/2011 09:07 AM, philby john wrote:
>>> From: Philby John<pjohn@mvista.com>
>>
>> ^^^^^^^^ I believe that statement to be not entirely correct.
>>
>> Perhaps you should change it to something like:
>> From: David Daney <ddaney@caviumnetworks.com>
>>
>>> Date: Wed, 13 Apr 2011 20:46:32 +0530
>>> Subject: [PATCH] MIPS: Octeon: add option to ignore PT_NOTE section
>>>
>>> Some early Octeon bootloaders cannot process PT_NOTE program
>>> headers as reported in numerous sections of the web, here is
>>> an example http://www.spinics.net/lists/mips/msg37799.html
>>> Loading usually fails with such an error ...
>>> Error allocating memory for elf image!
>>>
>>> The work around usually is to strip the .notes section by using
>>> such a command $mips-gnu-strip -R .notes vmlinux -o fixed-vmlinux
>>> It is expected that the vmlinux image got after compilation be
>>> bootable. Add a Kconfig option to ignore the PT_NOTE section.
> 
> Do we really want this to be in the kernel? In my opinion, this is a fixup 
> which distributions should be aware of, but not necessarily take place here 
> in 
> the kernel Makefiles.

You are right in one way. But as an OS vendor company we will definitely
include this patch in our distribution. This incident has been reported
many a times and its a pain to see the image not boot up, throw up an
error, with the user having to search the work around on the web. What
we are trying to do is save all that trouble. If it can be fixed why not
fix it.

Cheers,
Philby

<Prev in Thread] Current Thread [Next in Thread>