From: grygorii.strashko@linaro.org (Grygorii.Strashko@linaro.org)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] arm: omap: reduce zImage size on omap2plus_defconfig
Date: Fri, 26 Dec 2014 15:04:00 +0200 [thread overview]
Message-ID: <549D5CC0.2090408@linaro.org> (raw)
In-Reply-To: <549BE333.9060709@compulab.co.il>
On 12/25/2014 12:13 PM, Igor Grinberg wrote:
> Hi Tony,
>
> On 12/24/14 21:04, Tony Lindgren wrote:
>> * Felipe Balbi <balbi@ti.com> [141224 07:52]:
>>> Hi,
>>>
>>> On Wed, Dec 24, 2014 at 01:53:46PM +0200, Igor Grinberg wrote:
>>>> -----BEGIN PGP SIGNED MESSAGE-----
>>>> Hash: SHA1
>>>>
>>>> On 12/23/14 18:19, Felipe Balbi wrote:
>>>>> On Tue, Dec 23, 2014 at 09:30:45AM +0200, Igor Grinberg wrote:
>>>>>> Hi Felipe,
>>>>>>
>>>>>> On 12/22/14 20:05, Felipe Balbi wrote:
>>>>>>
>>>>>> [...]
>>>>>>
>>>>>>> CONFIG_SCSI_SCAN_ASYNC=y
>>>>>>> -CONFIG_ATA=y
>>>>>>> -CONFIG_SATA_AHCI_PLATFORM=y
>>>>>>> -CONFIG_MD=y
>>>>>>> +CONFIG_ATA=m
>>>>>>> +CONFIG_SATA_AHCI_PLATFORM=m
>>>>>>
>>>>>> Isn't this needed for the rootfs on SATA devices?
>>>>>
>>>>> there's no known boards with rootfs on SATA. Until then, we can reduce
>>>>> the size.
>>>>
>>>> What makes you say so?
>>>> The decision for rootfs on SATA is taken dynamically.
>>>> OMAP5 boards (specifically cm-t54) can have rootfs on SATA...
>>>
>>> I'll leave the decision to Tony. Even though they _can_, they really
>>> don't and IIRC, OMAP5's SATA has so many silicon errata that it'd be
>>> annoying to find that special device which works (e.g it can't negotiate
>>> lower speeds with SATA III devices and it won't support SATA I).
>
> Yet, it is not that buggy and at least until now, I di not get any
> reports about badly working SATA from customers...
>
>>>
>>> As of today, we don't know of anybody really shipping anything with
>>> rootfs on SATA and distros would rather ship initiramfs than a giant
>>> zImage anyway.
>
> So, you just continue to ignore what I'm saying... even after I point
> to a device...
>
> Is it SATA that makes it so giant?
> Because I find it worth having in SATA than spare some more k's...
>
SATA code occupies ~179Kbytes - checked on 3.19-rc1
CONFIG_ATA=y
CONFIG_SATA_AHCI_PLATFORM=y
CONFIG_MD=y (seems it will give +-0K)
>>>
>>> Tony, your call
May be it will be good thing to split this patch. That way more information
will be stored in commit log about which set of options gives us what benefits.
And also, It will allow to continue with agreed changes.
?
>>
>> I think we should move omap2plus_defconfig to be mostly modular and
>> usable for distros as a base. Most distros prefer to build almost
>> everything as loadable modules. And my preference is that we should
>> only keep the minimum rootfs for devices and serial support as
>> built-in and rely on initramfs for most drivers. And slowly move
>> also the remaining built-in drivers to be loadable modules.
>>
>> The reasons for having drivers as loadable modules are many. It
>> allows distros to use the same kernel for all the devices without
>> bloating the kernel. It makes developing drivers easier as just the
>> module needs to be reloaded. And loadable modules protect us from
>> cross-framework spaghetti calls in the kernel as the interfaces are
>> clearly defined.
>>
>> Are there people really using SATA as rootfs right now on omaps?
>
> Yes. That is exactly my point.
>
>From my side I'd like to note that I know about few ongoing projects on DRA7x EVM
where SATA is used as rootfs - now It is the fast possible way to start Android.
--
regards,
-grygorii
next prev parent reply other threads:[~2014-12-26 13:04 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-22 18:05 [PATCH] arm: omap: reduce zImage size on omap2plus_defconfig Felipe Balbi
2014-12-22 18:11 ` Felipe Balbi
2014-12-23 7:30 ` Igor Grinberg
2014-12-23 16:19 ` Felipe Balbi
2014-12-23 16:56 ` Tony Lindgren
2014-12-23 17:07 ` Felipe Balbi
2014-12-24 11:53 ` Igor Grinberg
2014-12-24 15:49 ` Felipe Balbi
2014-12-24 19:04 ` Tony Lindgren
2014-12-25 10:13 ` Igor Grinberg
2014-12-26 4:42 ` Felipe Balbi
2014-12-26 11:56 ` Igor Grinberg
2014-12-26 13:42 ` Javier Martinez Canillas
2014-12-26 15:19 ` Felipe Balbi
2014-12-26 19:38 ` Javier Martinez Canillas
2014-12-26 19:47 ` Felipe Balbi
2014-12-26 15:09 ` Felipe Balbi
2014-12-26 16:43 ` Igor Grinberg
2014-12-26 13:04 ` Grygorii.Strashko@linaro.org [this message]
2014-12-26 15:26 ` Felipe Balbi
2014-12-26 16:13 ` Tony Lindgren
2014-12-26 16:26 ` Felipe Balbi
2015-01-08 0:43 ` Tony Lindgren
2014-12-26 4:37 ` Felipe Balbi
2014-12-26 12:08 ` Igor Grinberg
2014-12-26 15:24 ` Felipe Balbi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=549D5CC0.2090408@linaro.org \
--to=grygorii.strashko@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).