svn commit: r287934 - head/sys/boot/efi/loader

[Warner Losh]

You're right about the Wifi drivers. There's some number you'll want loaded
and we should have sensible defaults. But how to get there from here may
be a bit interesting...  Though if I go with the devd.conf writer early in
boot,
I can make them be rc.conf variable controlled.

Warner

On Wed, Sep 23, 2015 at 8:13 PM, Rui Paulo <rpaulo at me.com> wrote:

> Those were the issues that I encountered when I started using MINIMAL.
> I didn't do a thorough investigation.
>
> Auto loading is a much bigger problem that just loading drivers for
> PCI/USB/etc devices.  For example, net80211 doesn't auto load the wlan
> crypto modules by default nor the amrr module.
>
> On Mon, 2015-09-21 at 17:59 -0600, Warner Losh wrote:
> > Apart from the inlining issue John raised (which I agree with his
> > solution on, btw)
> > and the one cam ctl module, what other modules are meaningfully
> > different when
> > compiled as modules.
> >
> > Assume that the auto-loading bit is solved, at least for devices on
> > self-enumerating
> > busses.
> >
> > Warner
> >
> >
> > > On Sep 21, 2015, at 4:53 PM, Rui Paulo <rpaulo at me.com> wrote:
> > >
> > > No, that doesn't work very well.  Not only the modules don't auto
> > > -load, the way the modules are compiled is different.  See, for
> > > example, cam ctl which doesn't compile the sg code when it's built
> > > into the kernel, but compiles it when it's built as a module.  The
> > > sg code is currently buggy and causes insta-panics with GNOME 3
> > > (perhaps the auto-mounter in hald (?)).
> > > --
> > > Rui Paulo
> > >
> > >
> > > On Sep 21, 2015, at 11:24 AM, Adrian Chadd <adrian.chadd at gmail.com>
> > > wrote:
> > >
> > > > Hi,
> > > >
> > > > Warner has been working on the modular kernel thing. But
> > > > honestly, I
> > > > think we should just start biting that bullet and ship a modules
> > > > -only
> > > > GENERIC by default..
> > > >
> > > >
> > > > -a
> > > >
> > > >
> > > > On 21 September 2015 at 11:02, Rui Paulo <rpaulo at me.com> wrote:
> > > > > So, we're going to keep ignoring the problem and keep patching
> > > > > things up?
> > > > > It's a bit sad that a single driver (pmspcv) is able to cause
> > > > > so much
> > > > > problems.
> > > > >
> > > > > --
> > > > > Rui Paulo
> > > > >
> > > > >
> > > > > On Sep 17, 2015, at 01:36 PM, John Baldwin <jhb at FreeBSD.org>
> > > > > wrote:
> > > > >
> > > > > Author: jhb
> > > > > Date: Thu Sep 17 20:36:46 2015
> > > > > New Revision: 287934
> > > > > URL: https://svnweb.freebsd.org/changeset/base/287934
> > > > >
> > > > >
> > > > > Log:
> > > > > The EFI boot loader allocates a single chunk of contiguous
> > > > > memory to
> > > > > hold the kernel, modules, and any other loaded data. This
> > > > > memory block
> > > > > is relocated to the kernel's expected location during the
> > > > > transfer of
> > > > > control from the loader to the kernel.
> > > > >
> > > > > The GENERIC kernel on amd64 has recently grown such that a
> > > > > kernel + zfs.ko
> > > > > no longer fits in the default staging size. Bump the default
> > > > > size from
> > > > > 32MB to 48MB to provide more breathing room.
> > > > >
> > > > > PR: 201679
> > > > > Reviewed by: imp
> > > > > MFC after: 1 week
> > > > > Differential Revision: https://reviews.freebsd.org/D3666
> > > > >
> > > > >
> > > > > Modified:
> > > > > head/sys/boot/efi/loader/copy.c
> > > > >
> > > > > Modified: head/sys/boot/efi/loader/copy.c
> > > > > ===============================================================
> > > > > ===============
> > > > > --- head/sys/boot/efi/loader/copy.c Thu Sep 17 20:36:34 2015
> > > > > (r287933)
> > > > > +++ head/sys/boot/efi/loader/copy.c Thu Sep 17 20:36:46 2015
> > > > > (r287934)
> > > > > @@ -38,7 +38,7 @@ __FBSDID("$FreeBSD$");
> > > > > #include <efilib.h>
> > > > >
> > > > > #ifndef EFI_STAGING_SIZE
> > > > > -#define EFI_STAGING_SIZE 32
> > > > > +#define EFI_STAGING_SIZE 48
> > > > > #endif
> > > > >
> > > > > #define STAGE_PAGES ((EFI_STAGING_SIZE) * 1024 * 1024 / 4096)
> > > > >
> >
>
> --
> Rui Paulo
>
>