[OpenPower-Firmware] [PATCH] skiroot_defconfig: Remove VirtIO and Solarflare

[Joel Stanley]

On Tue, 8 Dec 2020 at 20:21, Klaus Heinrich Kiwi
<klaus at linux.vnet.ibm.com> wrote:
>
>
>
> On 12/8/2020 12:02 AM, Joel Stanley wrote:
> > On Mon, 7 Dec 2020 at 20:28, Klaus Heinrich Kiwi
> > <klaus at linux.vnet.ibm.com> wrote:
> >>
> >> Remove drivers that apparently are not being used:
> >>    * Solarflare
> >>    * VirtIO-related drivers
> >
> > nak, virtio is used for testing in qemu.
> >
> Good point - although perhaps we could use e1000 and megasas without any practical drawbacks?
>
> At any rate, I'm looking for alternatives to keep our zImage.epapr (with bundled rootfs)
> under 16MB - it is looking like this is desirable for maintaining compatibility with
> AIX / IBM i "lids" constructs that have this as an upper limit.
>
> Looking for things that we might not care about on Skiroot (anymore) plus the fact that
> retired Power8 support, some thoughts are:
>
>
>    * Chelsio T3 and T4 and QLogic NetXtreme II iSCSI support - not aware of anyone shipping,
>      or even using this. Not having hardware offloading for iscsi sounds hardly a blocker for
>      a bootloader, but I couldn't even find the iscsi initiator userland configured.. so not
>      sure why we have this enabled?
>    * Alteon AceNIC/3Com 3C985/NetGear GA620 Gigabit support - mac G3 era legacy? Couldn't
>      find why we ever needed this on OpenPower
>    * QLogic bnx2 & CNIC support - looks like tg3 and bnx2x have been the norm lately
>    * Myricom Myri-10G - not aware of anyone using it
>    * Neterion (Exar) Xframe 10Gb - same

I suggest removing all of these. It was never clearly documented why
we had some adaptors enabled, and I suspect at least some of them are
simply because they were in the defconfig when the project was
started.

>
> Other non-kernel choices might be:
>    * rsync - some 500KB shaved
>    * lrsz - 90kb
>    * netcat - 80kb

I would suggest keeping these tools, they are invaluable for debug.

>
> I wonder if we could live by without them. If we want to be under 16MB, we need to be a tad
> aggressive with those choices right now..
>
>
> >>   CONFIG_EXT4_FS_POSIX_ACL=y
> >> @@ -322,4 +316,5 @@ CONFIG_DEBUG_CREDENTIALS=y
> >>   # CONFIG_FTRACE is not set
> >>   # CONFIG_RUNTIME_TESTING_MENU is not set
> >>   CONFIG_BUG_ON_DATA_CORRUPTION=y
> >> +CONFIG_PPC_DISABLE_WERROR=y
> >
> > This is unrelated.
> >
>
> Missed that, thanks
>   -Klaus
> --
> Klaus Heinrich Kiwi <klaus at linux.vnet.ibm.com>