From owner-dev-commits-src-main@freebsd.org Tue Feb 9 09:19:26 2021 Return-Path: <owner-dev-commits-src-main@freebsd.org> Delivered-To: dev-commits-src-main@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id A905E537FA5; Tue, 9 Feb 2021 09:19:26 +0000 (UTC) (envelope-from debdrup@freebsd.org) Received: from freefall.freebsd.org (freefall.freebsd.org [96.47.72.132]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "freefall.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4DZcly4Kt6z4lh8; Tue, 9 Feb 2021 09:19:26 +0000 (UTC) (envelope-from debdrup@freebsd.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1612862366; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=PfLm0LDuhspL/uQDXJVFvc8wiwyy+VUlCibvN6ecPrk=; b=fPlqXnjzBiBwk1al6k6bKq2np9Lz1GOyMXhMMnYfRUimOJiwD89kGRqQYjpa9NY2xBvtyy 4XJyhEyI2RSWOgpgbmJy0IQp7O1vuqHJ6TVNa3i45mjyTtecYdwcxHa5dNyvDGS2cyorDx rYAaae4kM2Sd9z27eF5UJSd3N10eC3b5tLu5l8ysSVJAbD3rujrABjMOA9nFSQVPIDIFgz jfNrvEo/IOEMwMWgA0futwOxXVK5XvLts1jqcTwNRKhIpmaPYKSwgAmOIxy9IR3UaQCIWR VgBWQBmHaTHabj106LxoWJFFh4LiA4rx9T6h/IqOPbg7Tx5ntOIxKgwE9nHerg== Received: by freefall.freebsd.org (Postfix, from userid 1471) id 86AF514519; Tue, 9 Feb 2021 09:19:26 +0000 (UTC) Date: Tue, 9 Feb 2021 10:19:24 +0100 From: Daniel Ebdrup Jensen <debdrup@FreeBSD.org> To: dev-commits-src-main@freebsd.org, "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org> Subject: Re: git: 866c8b8d5ddb - main - kldload(8): Add note about using kld_list in rc.conf(5) Message-ID: <20210209091924.2llehdxlndekytkp@nerd-thinkpad.local> Mail-Followup-To: Daniel Ebdrup Jensen <debdrup@FreeBSD.org>, dev-commits-src-main@freebsd.org, "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org> References: <202102080704.11874w7E017472@gitrepo.freebsd.org> <CANCZdfrJjBxDwM5gpccDbAJEhw_HBT7eq+2Tfd-kTxEo9_DitA@mail.gmail.com> <d9acb4ee-5ea4-b006-6fa0-86da8d369eda@grosbein.net> <CACNAnaF0xpfOodU=B4c91sA4za=-wfRgTY_MiJtr9TZuMNpoHw@mail.gmail.com> <f832f431-f151-bc47-948c-fded131e0766@grosbein.net> <CACNAnaFkywYo7bU8GyPYdTzCH4Xy-s4F38beBwJee5k4Odkdsg@mail.gmail.com> <33d6f518-2005-e859-0230-34490c3c5914@grosbein.net> <CACNAnaGte5LpzWJt1+fxYV+SKz9kMp13U4FyM0si+FcRQUytPA@mail.gmail.com> <20210208184822.uejqvgyl2ddv4emc@nerd-thinkpad.local> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="wfpb4v3ij6dlmabh" Content-Disposition: inline In-Reply-To: <20210208184822.uejqvgyl2ddv4emc@nerd-thinkpad.local> ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1612862366; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=PfLm0LDuhspL/uQDXJVFvc8wiwyy+VUlCibvN6ecPrk=; b=tIr1e7xCMnvu9Vv+8kJ5xvaaHLFBaKDJBhegA9J+lpdLQ4O/9xA+DLw4eyome0nn3rRtJ8 HP9CAlpaZ8+A/tEA+lMTnAhWU2tihzbGuHjBxA/EHoEwaJZpkWdc/q0Axv5RaTil7f3yfZ QCcjct8qbTQI9aeZhI0Vmjh8GHKdpKcGhtYMX6DpXSkqiHKS+YfgH2PeDBj9EDE8zIOkGG HIO17TT76yxXXCc2sA1HuwZjR3J4koc+0fGGXNACCSuAjsl0GlELg/60NnU7gZ1QzhBLh8 7CtwVtJ90cpKHjTrpFtdzyymQ7U/+ZhtHVeH/XeEvptH4s7vGuOUGOzlshA12w== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1612862366; a=rsa-sha256; cv=none; b=Gw+27bPYui2TU+44dA40j5Y4VB5rSAA1GDS+YJ0YLBFv/a45yULoI+1sXuaHQA7HWHdzBI Xs4CQMzaOC7NOrv+7Z8f0+8639n/81CIJYNuK1gKvZK79DTUk/i07xXv1AyZ2EsYWJ3RLB MLgHkaGWHun2kXnIQF3yVnbVQjzi9EOlhcNjkiRL+1CgiJ15HTAMCHz+g9e7sscsfkfQGx zo7CeC573ZxI0Ji4/VsnKpRvwc8T2QBIzvlzJsr9w0OvjMi844UJgZ/W8o04czjJjWQhHM afeCRQUmRY+K/xtpWWaJ8SR36B720sUA+pd9j8i86Ab3dGCygDRunQ9GA2raog== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-BeenThere: dev-commits-src-main@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for the main branch of the src repository <dev-commits-src-main.freebsd.org> List-Unsubscribe: <https://lists.freebsd.org/mailman/options/dev-commits-src-main>, <mailto:dev-commits-src-main-request@freebsd.org?subject=unsubscribe> List-Archive: <http://lists.freebsd.org/pipermail/dev-commits-src-main/> List-Post: <mailto:dev-commits-src-main@freebsd.org> List-Help: <mailto:dev-commits-src-main-request@freebsd.org?subject=help> List-Subscribe: <https://lists.freebsd.org/mailman/listinfo/dev-commits-src-main>, <mailto:dev-commits-src-main-request@freebsd.org?subject=subscribe> X-List-Received-Date: Tue, 09 Feb 2021 09:19:26 -0000 --wfpb4v3ij6dlmabh Content-Type: text/plain; charset=us-ascii; format=flowed Content-Disposition: inline On Mon, Feb 08, 2021 at 07:48:22PM +0100, Daniel Ebdrup Jensen wrote: >On Mon, Feb 08, 2021 at 09:22:25AM -0600, Kyle Evans wrote: >>On Mon, Feb 8, 2021 at 9:07 AM Eugene Grosbein <eugen@grosbein.net> wrote: >>> >>>08.02.2021 21:58, Kyle Evans wrote: >>> >>>>>>> kld_list cannot do that. >>>>>> >>>>>> Huh? kld_list accepts a full pathname, which is the same kind of >>>>>> specification you'd need to do with one from port in loader with >>>>>> *_name. >>>>> >>>>> Good, but seems to be undocumented. >>>>> >>>> >>>> In what sense? Is there some other place that kld_list is even >>>> documented than kldload(8)? >>> >>>Naturally: rc.conf(5), also in /etc/defaults/rc.conf >>> >> >>Wow, that documentation is just flat wrong for multiple reasons. >> >> kld_list (str) A whitespace-separated list of kernel modules to load >> right after the local disks are mounted, without any .ko >> extension or path. Loading modules at this point in the boot >> process is much faster than doing it via /boot/loader.conf >> for those modules not necessary for mounting local disks. >> >>The second part of the first sentence is a self-imposed limitation, >>and an incredibly unimportant one at that. Specifying a .ko will only >>break the existing "Is it loaded" behavior and cause it to always try, >>which is mostly a nuisance at best because it will get rejected if the >>kldstat inquiry is wrong -- the kld rc script will append .ko whether >>the path has one or not, so this should be fixed. Despite that, a path >>will work just fine for the most part; kldstat -v shows the fully >>qualified path. If it wanted to be improved, it just needs to basename >>what it was given to be able to detect if it was loaded from any other >>path and pass that in as the -e argument to load_kld if it really was >>a file. >> >>It's not wrong about loading modules being faster here, and at some >>point in the past it was even necessary due to loader(8) being too >>early or problematic in some terrible cases (e.g. nvidia modsetting >>bits), but the second half of that sentence should probably just be >>omitted or reworked to more vaguely refer to "... those modules not >>necessary for booting the system, including those required for >>mounting the root filesystem." The key changes being that there are >>other reasons you might need to load something early enough in boot, >>and that root isn't always a local disk. It's OK to call out the more >>common case for folks, but this feels a lot more absolute than it >>needs to be. >>_______________________________________________ >>dev-commits-src-main@freebsd.org mailing list >>https://lists.freebsd.org/mailman/listinfo/dev-commits-src-main >>To unsubscribe, send any mail to "dev-commits-src-main-unsubscribe@freebsd.org" > >Hi folks, > >I'd originally replied to imp@ when he sent me a comment about this, >although because of a misconfiguration I hadn't seen it was also sent to >a different list, but in it I remarked the same things about >nvidia-modesetting and the speed, however I do think Kyle is right that >it's better to rework the sentence because mountroot isn't really >docuemented in the extant manual pages. > >I'm not sure about wanting to document the other details outlined here, >because I'm not entirely sure I understand them - plus, it's in a >different manual page, so it's free for anyone with an active commit bit >and an understanding to persue. :) > >I'll try and work something out and do a Phabricator review, but let's >try and keep the bike-shedding to a minimum. > >Yours, >Daniel Ebdrup Jensen > >P.S. We all know the best colour is purple. ;) Hi folks, For reference, the review is up at https://reviews.freebsd.org/D28542 and I've added a few people, who contacted me about this, to it. Yours, Daniel Ebdrup Jensen --wfpb4v3ij6dlmabh Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQGTBAABCgB9FiEEDonNJPbg/JLIMoS6Ps5hSHzN87oFAmAiU5xfFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDBF ODlDRDI0RjZFMEZDOTJDODMyODRCQTNFQ0U2MTQ4N0NDREYzQkEACgkQPs5hSHzN 87o63Af+I29+WGzQjR1DWcd1ROlzTJQllnO9EFZIRaav7MDiiFOwbI846554U0pQ 2bkITJo8GDpRPxyu0Qha67UA9dAR7IxznabapgodOGcrkLxExlhY9oGZiOt3WcfS SdrAIMlM7ADuHlUdqjBrNiltUaTVIzVmle+ZEPdqbYHZeOOvcC4QNitD3BuRG2iI tm8lmN/+DwIRAGHDprIJWpFdWke4b86rXw04TOlNofsKW5spNqUWOyrya0PZ0jbd EnzVUk9+ZgxWBRDIGnWrFeDCi1wc90D3DN/Bz2L3PWWxx/cCPNrrYgulw/1mCXrE Uh68Co50HZOGFaAeLFnNvcDNLkekZA== =6T5M -----END PGP SIGNATURE----- --wfpb4v3ij6dlmabh--