From nobody Tue Nov 14 17:11:06 2023 X-Original-To: dev-commits-ports-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4SVCWB6fs1z51XWV; Tue, 14 Nov 2023 17:11:18 +0000 (UTC) (envelope-from marius@btk.zeist.de) Received: from btk.zeist.de (btk.zeist.de [IPv6:2a01:4f8:221:2f4f::2]) (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 "btk.zeist.de", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4SVCWB0rg0z4Qp0; Tue, 14 Nov 2023 17:11:17 +0000 (UTC) (envelope-from marius@btk.zeist.de) Authentication-Results: mx1.freebsd.org; dkim=none; spf=none (mx1.freebsd.org: domain of marius@btk.zeist.de has no SPF policy when checking 2a01:4f8:221:2f4f::2) smtp.mailfrom=marius@btk.zeist.de; dmarc=none Received: from btk.zeist.de (localhost [127.0.0.1]) by btk.zeist.de (8.17.1/8.17.1/BTK.ZEIST.DE) with ESMTPS id 3AEHB6jH077191 (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384 bits=256 verify=NO); Tue, 14 Nov 2023 18:11:06 +0100 (CET) (envelope-from marius@btk.zeist.de) Received: (from marius@localhost) by btk.zeist.de (8.17.1/8.17.1/Submit) id 3AEHB6La077190; Tue, 14 Nov 2023 18:11:06 +0100 (CET) (envelope-from marius) Date: Tue, 14 Nov 2023 18:11:06 +0100 From: Marius Strobl To: Joseph Mingrone 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: References: <202311092025.3A9KPJhe049861@gitrepo.freebsd.org> <867cmld31m.fsf@phe.ftfl.ca> List-Id: Commit messages for all branches of the ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-ports-all@freebsd.org X-BeenThere: dev-commits-ports-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <867cmld31m.fsf@phe.ftfl.ca> X-Spamd-Result: default: False [-1.79 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-1.000]; AUTH_NA(1.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000]; NEURAL_HAM_SHORT(-0.99)[-0.994]; FORGED_SENDER(0.30)[marius@freebsd.org,marius@btk.zeist.de]; MIME_GOOD(-0.10)[text/plain]; MIME_TRACE(0.00)[0:+]; MLMMJ_DEST(0.00)[dev-commits-ports-all@FreeBSD.org,dev-commits-ports-main@FreeBSD.org]; R_SPF_NA(0.00)[no SPF record]; ASN(0.00)[asn:24940, ipnet:2a01:4f8::/32, country:DE]; R_DKIM_NA(0.00)[]; RCVD_TLS_LAST(0.00)[]; DMARC_NA(0.00)[freebsd.org]; FREEFALL_USER(0.00)[marius]; ARC_NA(0.00)[]; FROM_NEQ_ENVFROM(0.00)[marius@freebsd.org,marius@btk.zeist.de]; RCPT_COUNT_THREE(0.00)[4]; TO_MATCH_ENVRCPT_ALL(0.00)[]; TO_DN_SOME(0.00)[]; FROM_HAS_DN(0.00)[]; RCVD_COUNT_TWO(0.00)[2] X-Rspamd-Queue-Id: 4SVCWB0rg0z4Qp0 X-Spamd-Bar: - On Mon, Nov 13, 2023 at 12:35:33PM -0400, Joseph Mingrone wrote: > On Thu, 2023-11-09 at 20:25, Marius Strobl wrote: > > > The branch main has been updated by marius: > > > URL: https://cgit.FreeBSD.org/ports/commit/?id=d12bc36e7698f3737de1ac228bd49eab10cf1bb0 > > > commit d12bc36e7698f3737de1ac228bd49eab10cf1bb0 > > Author: Marius Strobl > > AuthorDate: 2023-11-09 20:20:53 +0000 > > Commit: Marius Strobl > > 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. Hi Joe, it's probably a matter of taste but when it does come to something like updating UCODE on a running - likely critical as otherwise I'd simply reboot with the new firmware in place - system, I typically feel uneasy and want to know exactly what's going to be done. For me, this translates to - as mentioned earlier - using cpucontrol(8) directly, i. e. issuing: # echo /dev/cpuctl* | xargs -n 1 cpucontrol -u instead of a 77 lines of code for microcode_update on top of 175 lines of script for service(8). IMO that's also not particularly more cryptic of harder to remember than: # service microcode_update onestart In any case, as previously stated there's no need to employ the RC script for manual updates either and least as far as I'm concerned, running cpucontrol(8) directly instead is preferable. Thus, I propose to at least switch the default. But I leave that decision up to you. Marius