[linux-uc20-efi][PATCH 1/1] UBUNTU: [Debian] Add megaraid_sas into initrd

[Jeffrey Lane]

On Tue, Jan 26, 2021 at 12:57 PM Jesse Sung <jesse.sung at canonical.com>
wrote:

> On Wed, Jan 27, 2021 at 1:38 AM Seth Forshee <seth.forshee at canonical.com>
> wrote:
> >
> > On Fri, Jan 22, 2021 at 06:56:00PM +0800, Wen-chien Jesse Sung wrote:
> > > BugLink: https://launchpad.net/bugs/1912752
> > >
> > > Module megaraid_sas is required in order to some systems to boot with
> > > the pc-kernel snap. Append megaraid_sas to ubuntu-core-initramfs.conf
> > > before using ubuntu-core-initramfs to generate the initrd in order to
> > > include the particular module.
> > >
> > > Signed-off-by: Wen-chien Jesse Sung <jesse.sung at canonical.com>
> >
> > Why not change this in ubuntu-core-initramfs instead? This is a kludgy
> > way to add the module to the conf file, so we'd only want to do this if
> > there's some reason it's unacceptable to add it to ubuntu-core-initramfs
> > directly.
>
> Including megaraid_sas into evey initramfs (including the armhf/arm64
> ones) may not be what we want, especially for those devices with
> limited storage space. This driver most likely is required for the
> pc-kernel only.
>

Relating to the project that triggers this bug, I agree, this only
currently applies to one specific AMD64 machine.

There is a much larger discussion needed, however, as in order to support
servers,  the initrd/initramfs for core will need to grow a LOT of new
drivers in order to support deploying core onto any supported server, and
eventually doing so via MAAS.

IMO not doing so by default today makes this only a band-aid until a point
in the future where this will have to be done across architectures anyway
and this will have to revisited, but that may belong to the larger
discussion anyway.  Just my 2 cents worth of unsolicited opinion.

Cheers
Jeff


>
> Thanks,
> Jesse
>
> >
> > Seth
> >
> > > ---
> > >  debian/rules | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/debian/rules b/debian/rules
> > > index 1d85292..fc46042 100755
> > > --- a/debian/rules
> > > +++ b/debian/rules
> > > @@ -50,6 +50,7 @@ override_dh_auto_build:
> > >                       apt-get download $$pkg;
>        \
> > >                       dpkg-deb -x $${pkg}_*.deb $$flavour;
>         \
> > >               );
>         \
> > > +             echo megaraid_sas >>
> /usr/lib/ubuntu-core-initramfs/main/usr/lib/modules-load.d/ubuntu-core-initramfs.conf
> \
> > >               ubuntu-core-initramfs create-initrd --output $(initrd)
>         \
> > >                       --kernelver $(KERNEL_ABI_VERSION)-$$flavour;
>         \
> > >               ubuntu-core-initramfs create-efi --unsigned --output
> $(efi)     \
> > > --
> > > 2.27.0
> > >
> > >
> > > --
> > > kernel-team mailing list
> > > kernel-team at lists.ubuntu.com
> > > https://lists.ubuntu.com/mailman/listinfo/kernel-team
>
> --
> kernel-team mailing list
> kernel-team at lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
>


-- 
Jeff Lane
Engineering Manager
IHV/OEM Alliances and Server Certification

"Entropy isn't what it used to be."
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20210126/6487e269/attachment-0001.html>