From owner-svn-src-head@freebsd.org Thu Sep 24 04:07:04 2015 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id C8BD3A08884 for ; Thu, 24 Sep 2015 04:07:04 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-qk0-f176.google.com (mail-qk0-f176.google.com [209.85.220.176]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 8ADE8187A for ; Thu, 24 Sep 2015 04:07:04 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by qkap81 with SMTP id p81so25590506qka.2 for ; Wed, 23 Sep 2015 21:06:57 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:sender:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=uBE6h4fp+jl9nKmvxNK+UYlvThYTiqMnp7OpeVkOXnM=; b=QBxP8QsdmpSyLNCJLVYauBCEk1WLUWpc6Oc0EzM9VTpJAquzcrQRXULtnRfv3DyYCk i08GG02W8NSjcNGjv5p14Gmk13MDxr9LcWJ0Jj5pJUuL2j5yLqiyyjhAo0yW11J3U8Bs hG35QBhRSny8i+b2yKOzgkDQtXuXTvsEQS+J/v9BuagxxkZBXAQ/jbDkjr2NWtyOp4T2 LlsMhOmDMMmBR1hC04CIqllF6ou/Zy/nrkv0aOtvhYGansWFXAdgQfuYOwcSwP/1YrdA CcEFAHtiUnSKnTeM4N2e7V/+UKyMocQNXXEVDPZOLwybItMZWUmsvUzjYjMx5JriFz0N 0kng== X-Gm-Message-State: ALoCoQnx1BjSH2mQ8rVPFyA2dM7W6A9yg7o3yVhT9zOYpDDjv9cTbzHOZE0HUlRgi2qSypx2kznO MIME-Version: 1.0 X-Received: by 10.55.19.11 with SMTP id d11mr6742643qkh.77.1443067617395; Wed, 23 Sep 2015 21:06:57 -0700 (PDT) Sender: wlosh@bsdimp.com Received: by 10.140.80.167 with HTTP; Wed, 23 Sep 2015 21:06:57 -0700 (PDT) X-Originating-IP: [40.139.248.3] In-Reply-To: <1443064383.14580.3.camel@me.com> References: <35a0f1b6-0236-4b0e-b919-00cab07429be@me.com> <5427AC7C-1B0B-4273-B758-DB0C1BDF656F@bsdimp.com> <1443064383.14580.3.camel@me.com> Date: Wed, 23 Sep 2015 21:06:57 -0700 X-Google-Sender-Auth: IBNmQdbKu_Djns0Lyz4cDeryYrc Message-ID: Subject: Re: svn commit: r287934 - head/sys/boot/efi/loader From: Warner Losh To: Rui Paulo Cc: Adrian Chadd , John Baldwin , src-committers , "svn-src-all@freebsd.org" , "svn-src-head@freebsd.org" Content-Type: text/plain; charset=UTF-8 X-Content-Filtered-By: Mailman/MimeDel 2.1.20 X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 Sep 2015 04:07:05 -0000 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 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 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 > > > 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 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 > > > > > 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 > > > > > > > > > > #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 > >