From owner-dev-commits-src-all@freebsd.org Mon Feb 8 14:09:01 2021 Return-Path: Delivered-To: dev-commits-src-all@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 E1F88533BE0; Mon, 8 Feb 2021 14:09:01 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: from smtp.freebsd.org (smtp.freebsd.org [IPv6:2610:1c1:1:606c::24b:4]) (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 "smtp.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4DZ7DY5xxmz3m3G; Mon, 8 Feb 2021 14:09:01 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: from mail-qv1-f45.google.com (mail-qv1-f45.google.com [209.85.219.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) (Authenticated sender: kevans) by smtp.freebsd.org (Postfix) with ESMTPSA id BA4222587A; Mon, 8 Feb 2021 14:09:01 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: by mail-qv1-f45.google.com with SMTP id p12so2059349qvv.5; Mon, 08 Feb 2021 06:09:01 -0800 (PST) X-Gm-Message-State: AOAM530bdGXkN9TPqWwNkV35A06T0uyUIVK5WLmwr1WHOfuI5N+DrQvX oebaCWWpRFu+RAyILPVWhXGZNMr9+T5pdyV0jZc= X-Google-Smtp-Source: ABdhPJzabTmmdYybC8JR9jHoL1dB56n4J0U644GQcyzoLiF1C6GB/Ka7sook/oUDrq10NXs6Wm21a+10tPJ7P2cASk4= X-Received: by 2002:ad4:4f41:: with SMTP id eu1mr16092071qvb.34.1612793341290; Mon, 08 Feb 2021 06:09:01 -0800 (PST) MIME-Version: 1.0 References: <202102080704.11874w7E017472@gitrepo.freebsd.org> In-Reply-To: From: Kyle Evans Date: Mon, 8 Feb 2021 08:08:48 -0600 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: git: 866c8b8d5ddb - main - kldload(8): Add note about using kld_list in rc.conf(5) To: Eugene Grosbein Cc: Warner Losh , Daniel Ebdrup Jensen , src-committers , "" , dev-commits-src-main@freebsd.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-BeenThere: dev-commits-src-all@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for all branches of the src repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 08 Feb 2021 14:09:01 -0000 On Mon, Feb 8, 2021 at 8:00 AM Eugene Grosbein wrote: > > 08.02.2021 14:12, Warner Losh wrote: > > > > > > On Mon, Feb 8, 2021, 12:05 AM Daniel Ebdrup Jensen > wrote: > > > > The branch main has been updated by debdrup (doc committer): > > > > URL: https://cgit.FreeBSD.org/src/commit/?id=3D866c8b8d5ddb982c2b81= 39153a4ddfdb2aac3364 > > > > commit 866c8b8d5ddb982c2b8139153a4ddfdb2aac3364 > > Author: Daniel Ebdrup Jensen > > AuthorDate: 2021-02-08 06:49:32 +0000 > > Commit: Daniel Ebdrup Jensen > > CommitDate: 2021-02-08 06:57:36 +0000 > > > > kldload(8): Add note about using kld_list in rc.conf(5) > > > > While here, also recommend that loader.conf(5) should only be u= sed in > > order to get to mountroot, as rc(8) is less fragile, faster, an= d is > > easier to fix by booting to single-user mode instead of having = to > > blacklist modules in the loader. > > > > > > I'm not sure this is good advice. The only modules that are fragile are= 3rd party things like drm. It's no less hard to fix by unloading in the lo= ader or fixing in single user. The speed difference is true, but unless the= modules are huge, it isn't much. > > +1 > > Plus, Daniel missed two other points: nextboot(8) facility that works wit= h modules > loaded with the loader; and possibility to point to alternative location = of kernel module > with *_name variable documented in the loader.conf(5) manual page. > > The latter makes it possible to load alternative version of same driver, > real example is if_em.ko from base system vs. one from ports that has dif= ferent code base > and set of bugs/features. > > 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.