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

[Jesse Sung]

On Wed, Jan 27, 2021 at 4:00 AM Seth Forshee <seth.forshee at canonical.com> wrote:
>
> +Cc Dimitri
>
> On Tue, Jan 26, 2021 at 02:31:30PM -0500, Jeffrey Lane wrote:
> > 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.
>
> Dimitri,
>
> Since the diff below is difficult to read with all the quoting at this
> point, you can find the original here:
>
> https://lists.ubuntu.com/archives/kernel-team/2021-January/116703.html
>
> This strikes me as a pretty kludgy way to add this driver into the
> initramfs, but it looks like ubuntu-core-initramfs doesn't really
> support any way to add a module to the initramfs for only specific
> architectures. Am I missing something, or are we stuck with this kludge
> for now?

Could this be accepted as a temporary workaround until there's a
solution? We really need the module in pc-kernel initramfs for this
cycle...

Thanks,
Jesse

>
> Thanks,
> Seth
>
> >
> > 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."