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

[Adrian Chadd]

... I'm confused about the "load it by hand" stuff in net80211. Why
don't we just do the kldload at that point?


-a


On 23 September 2015 at 21:06, Warner Losh <imp at bsdimp.com> wrote:
> 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
>>
>