Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 13 Nov 2023 12:35:33 -0400
From:      Joseph Mingrone <jrm@FreeBSD.org>
To:        Marius Strobl <marius@FreeBSD.org>
Cc:        ports-committers@FreeBSD.org,  dev-commits-ports-all@FreeBSD.org, dev-commits-ports-main@FreeBSD.org
Subject:   Re: git: d12bc36e7698 - main - cpu-microcode-intel: Add knobs for RC RUN_DEPENDS and split files
Message-ID:  <867cmld31m.fsf@phe.ftfl.ca>
In-Reply-To: <202311092025.3A9KPJhe049861@gitrepo.freebsd.org> (Marius Strobl's message of "Thu, 9 Nov 2023 20:25:19 GMT")
References:  <202311092025.3A9KPJhe049861@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
--=-=-=
Content-Type: text/plain

On Thu, 2023-11-09 at 20:25, Marius Strobl <marius@FreeBSD.org> wrote:

> The branch main has been updated by marius:

> URL: https://cgit.FreeBSD.org/ports/commit/?id=d12bc36e7698f3737de1ac228bd49eab10cf1bb0

> commit d12bc36e7698f3737de1ac228bd49eab10cf1bb0
> Author:     Marius Strobl <marius@FreeBSD.org>
> AuthorDate: 2023-11-09 20:20:53 +0000
> Commit:     Marius Strobl <marius@FreeBSD.org>
> CommitDate: 2023-11-09 20:20:53 +0000

>     cpu-microcode-intel: Add knobs for RC RUN_DEPENDS and split files

>     Split UCODE files as well as dependency on sysutils/cpu-microcode-rc
>     are only required for use with cpucontrol(8) and update method two
>     respectively. The first method needs neither so we can spare a few
>     MiB and a superfluous port.
>     Given that the automatic update method one is superior, the RC knob
>     should be deprecated and removed eventually. At that point, split
>     UCODE files still will be useful for developers and if users want
>     to update the microcode without a reboot (but otherwise rely on the
>     loader for automatic updates, i. e. method one).

For the reasons you described, I proposed removing the RC script for the
Intel CPU microcode updates [1].  However, there was a reasonable
argument for keeping it.  As you say, users may still want to update
their Intel CPU microcode this way because updates can be applied on a
running system without requiring downtime for a reboot.

With your commit, I think we have a good compromise.  Users who really
want to save a bit of space and have one fewer port installed can turn
off these two knobs.  Maybe we could make that the default.

Joe

[1] See the comments in https://reviews.freebsd.org/D32349 for details.

--=-=-=
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQKkBAEBCgCOFiEEVbCTpybDiFVxIrrVNqQMg7DW754FAmVSUFZfFIAAAAAALgAo
aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDU1
QjA5M0E3MjZDMzg4NTU3MTIyQkFENTM2QTQwQzgzQjBENkVGOUUQHGpybUBmcmVl
YnNkLm9yZwAKCRA2pAyDsNbvnoYgD/9nCSA87UqKTVwTuRUfJy9kWW7l/+ZW/r7S
gjvfk85Gp4bRKw/re1GmQJ8srVaJJm9U4QGI94C7bSl2SIPWFYEoPI9xrLEUKKCT
4WTa76uSt3jvp4pFJhZDBu2d91ztxTIRAWXbOT0Tcx78JbL4ffJlGbsj7yYNBoer
xMCQbkXrCoUaiQ9WSIf7CbtD+RNhbpTHFOoLvUYMJ3wlWF93ExBHHi0//TG5hm2a
S6uHs1vVEQoHm3aDBAfuEVKENqgAr1GCGUZovLvzcbjmX1NF30RnMlZMryiHQqEG
jrCjPrCklboSE6CMPbkI20ZavbFvYUU8/LpJoAB5s4GOwesMdlIw2MeJMJXfPfQs
bg7sTc/n6ByCrO/YoATbBJgPAz/dF2dFL5x2WzAHiaDluFtIw6BgAO+V3bipSml5
Ci0HR4Nu8FEcZkwmFWIlIt7BpW5bFCSVtf4EQaowwVvtAND9kKl250YaYSTtjtJV
X4XNLwvLn/r+FcYxKp4y/ecp3sE7129U2vahwtT+iO5nDdUtosoW0FrKnqiZp7Lu
5PQTh64f+fueMeQrh2Zlmeb3WQVkY3sZE4mCXPwKHh8lXDjWJZ1EvkDsKQmc+vBP
AACtZFE8gbD9DjgzOs+TP3zC6BmiBJFTtav9YbrHLPRdVY85TUMdaxDR0bYSd9gf
nvGFzYss5w==
=KwXL
-----END PGP SIGNATURE-----
--=-=-=--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?867cmld31m.fsf>