FreeBSD git weekly: 2026-02-16 to 2026-02-22

Table of contents and commits per category:

(0)		Highlighted commits (these are copies, not in stats)
0	0.0%	Userland programs
5	22.7%	Documentation
3	13.6%	Hardware support
1	4.5%	Networking
1	4.5%	System administration
2	9.1%	Libraries
0	0.0%	Filesystems
4	18.2%	Kernel
0	0.0%	Build system
0	0.0%	Internal organizational stuff
6	27.3%	Testing
0	0.0%	Style, typos, and comments
0	0.0%	Contrib code
0	0.0%	Reverted commits
0	0.0%	Unclassified commits
22	100%	total
		Technical notes about this page

debug: info about the automatic classification

num	%	stage
5	22.7%	02-filenames_wildcards
6	27.3%	02b-filenames_wildcards2
3	13.6%	03-filenames_plain1
7	31.8%	04-filenames_plain2
1	4.5%	05-summary-prefix
0	0.0%	Manually-classified commits
0	0.0%	Unclassified commits

debug: more stats

num	%	stage
0	0.0%	Misclassified commits
22	100.0%	Classified commits, no corrections

debug: groups

0	0.0%	num in revert
0	0.0%	num in fixes
8	36.4%	num in consecutive
8	36.4%	Commits in groups

Documentation

Man pages, release notes, etc.

Commit group #0: open.2: stop making impression that fd must be directory

open.2: stop making impression that fd must be directory

Reviewed by:    des, rmacklem
Sponsored by:   The FreeBSD Foundation
MFC after:      3 days
Differential revision:  https://reviews.freebsd.org/D55302

5bcccc702b29a0e173a5916b001771dd7b280c7c Konstantin Belousov 2026-02-16 07:28:22

debug: classified in 02-filenames_wildcards by '.*\.[1-9]'

open.2: grammar improvements

Submitted by:   matteo
Fixes:  https://cgit.freebsd.org/src/commit/?id=5bcccc702b29a0e173a5916b001771dd7b280c7c
MFC after:      3 days

f1f142c01db43c7dc8d0d172fc1726da2ee01972 Konstantin Belousov 2026-02-16 15:34:35

debug: classified in 02-filenames_wildcards by '.*\.[1-9]'

sh.1: Document the exit status of return

It's not obvious that if `exitstatus` is omitted,
the exit status of the function is taken from the last executed command.

MFC after:              3 days
Reviewed by:            jilles, ziaee
Differential Revision:  https://reviews.freebsd.org/D55026

931d4f0ae4f2dcd088bb92459ff19c8b6aafc2cc Artem Bunichev 2026-02-16 17:14:00

debug: classified in 02-filenames_wildcards by '.*\.[1-9]'

sh.1: Fix ordering of Cm and Aq macros

MFC after:    3 days
Reviewed by:    ziaee
Fixes:  https://cgit.freebsd.org/src/commit/?id=2711852bd9ac (sh.1: Provide detailed job control documentation)
Differential Revision:  https://reviews.freebsd.org/D55194

e5213ca4a3ca742fd8d3b8db413a561b28fa6989 Artem Bunichev 2026-02-16 17:21:21

debug: classified in 02-filenames_wildcards by '.*\.[1-9]'

pdfork.2: add EFAULT as possible error, explain some consequences of it

Reviewed by:  markj
Sponsored by:   The FreeBSD Foundation
MFC after:      1 week
Differential revision:  https://reviews.freebsd.org/D55306

db80ea9b88628168d2bb7f17a60b73568a8ea102 Konstantin Belousov 2026-02-16 10:18:07

debug: classified in 02-filenames_wildcards by '.*\.[1-9]'

Hardware support

Hardware drivers and architecture-specific code.

ichsmb: Add Intel Raptor Lake SMBus controller support

Add PCI device ID 0x7a23 for Intel 700 Series (Raptor Lake) chipset SMBus controller.
This enables hardware monitoring functionality on 13th and 14th generation Intel Core platforms.

Reviewed by:    adrian
Differential Revision:  https://reviews.freebsd.org/D54957

e4bcfe4ebf03e321a5f8317491cdeea49e4f6357 Abdelkader Boudih 2026-02-16 03:58:41

debug: classified in 03-filenames_plain1 by 'sys/dev/'

powerpc: enable HPT superpages by default

It's time to flip these on and see if anything happens in -HEAD.
It can be turned back off before 16.0-RELEASE if there are problems.

Reviewed by:    jhibbits
Differential Revision:  https://reviews.freebsd.org/D55071

f412a5c565dedf5b6ea7cf3e43b57766ec0da63c Adrian Chadd 2026-02-16 17:49:46

debug: classified in 03-filenames_plain1 by 'sys/powerpc/'

powerpc: document the magic constants for 16MB page size

After discussion with jhibbits@, we pulled out what is supported here
and how it's supported.

Reviewed by:    imp
Differential Revision:  https://reviews.freebsd.org/D55093

995090202cae47567cc5b0d92301692d49c1c035 Adrian Chadd 2026-02-16 17:49:52

debug: classified in 03-filenames_plain1 by 'sys/powerpc/'

Libraries

libusb: make libusb_hotplug_get_user_data actually return user_data

MFC After:    2 days
Reviewed by:    adrian
Differential Revision:  https://reviews.freebsd.org/D55291

be522176951d8b542de9354f4ec9ac7603745b71 Baptiste Daroussin 2026-02-15 19:39:18

debug: classified in 04-filenames_plain2 by 'lib/'

libusb: dequeue next transfer on completion to prevent stalls

The transfer proxy callbacks (bulk/interrupt, control, isochronous)
only called libusb10_submit_transfer_sub() in the START path to
pipeline the second kernel transfer slot. On completion or error,
no attempt was made to dequeue the next pending transfer from
tr_head onto the now-free slot.

When more than two async transfers were submitted on the same
endpoint, the third (and subsequent) transfers would remain stuck
on tr_head indefinitely, since no completion ever triggered their
submission. This caused a protocol-level deadlock in applications
like adb that submit header + payload + zero-length terminator as
three separate bulk transfers in sequence.

Fix by calling libusb10_submit_transfer_sub() after every
libusb10_complete_transfer() in all three proxy callbacks.

MFC After:      2 weeks
Reviewed by:    adrian
Differential Revision:  https://reviews.freebsd.org/D55289

38c18332642500fdfe075a82f88e033f6673a53f Baptiste Daroussin 2026-02-15 18:07:07

debug: classified in 04-filenames_plain2 by 'lib/'

Kernel

Kernel stuff (other than networking, filesystems, and drivers).

sys/compat/freebsd32: FF clock struct: Don't pack, use 'ffcounter32'

Packing 'struct ffclock_estimate32', in absence of substitution of
'ffcounter' (some 'uint64_t') by a 32-bit compatible type, was necessary
on amd64 since 'uint64_t' is 8-byte aligned, which leaves a padding gap
of 4-byte between fields 'update_time' and 'update_ffcount'.  This gap
does not exist on i386 (or amd64 32-bit mode), as 'uint64_t' there is
only 4-byte aligned.

Change the type of the 'update_ffcount' and 'leapsec_next' fields to the
recently introduced 'freebsd32_uint64_t', and adapt copy-in and copy-out
accordingly.  Using `CP()` previously worked due to the '__packed__'
attribute.

Reviewed by:    kib
MFC after:      2 weeks
Sponsored by:   The FreeBSD Foundation
Differential Revision:  https://reviews.freebsd.org/D55282

7c2fc4419db43a8a7d1886b0b8e08aa97bfa31e4 Olivier Certner 2026-02-13 17:03:31

debug: classified in 04-filenames_plain2 by 'sys/'

sys/abi_types.h: time32_t is 64-bit on non-x86 architectures

As long as 'sys/compat/freebsd32/freebsd32.h' is used unconditionally on
all platforms (in 'kern_umtx.c' at least), the rule of thumb is to
ensure that 'struct foo32' on a 32-bit arch is type-compatible with
'struct foo' on the same arch.  In practice, this is very simple to
achieve: All 'foo32' types should be compatible with 'foo' on 32-bit
architectures, which is what we are supposed to do already for compat'
structures by design.  The recently introduced 'freebsd32_uint64_t' type
typically supports that.

This change fixes commit 87632ddf67b0 ("openzfs sys/types32.h: use
abi_compat.h for time32_t") which was defining 'time32_t' to 'in32_t'
for all 32-bit architectures, which is wrong but on i386.  By luck, this
did not change the size of whole 'struct ffclock_estimate32' (whose size
is compile-time asserted) because 'struct bintime32''s one would stay
the same, as even if its field 'sec' was incorrectly sized after that
commit, the 'frac' one is 64-bit and 64-bit aligned on all non-x86
architectures so its offset in 'struct bintime32' would stay the same.

Reviewed by:    kib
Fixes:          https://cgit.freebsd.org/src/commit/?id=87632ddf67b0 ("openzfs sys/types32.h: use abi_compat.h for time32_t")
Sponsored by:   The FreeBSD Foundation
Differential Revision:  https://reviews.freebsd.org/D55283

4ccca2100887943b11187787004bc8efc2a149c6 Olivier Certner 2026-02-13 17:20:13

debug: classified in 04-filenames_plain2 by 'sys/'

vmm: Allow the use of PCI passthrough in a jail

After commit e11768e94787 ("vmm: Add PRIV_DRIVER checks for passthru
ioctls"), it is not possible to use PCI passthru from jails, as
PRIV_DRIVER is not granted to jails.  Apparently some users expect this
to work, understanding that jailing bhyve provides little security
benefit in this configuration.

I believe we should disable ppt access in jails even when allow.vmm is
configured.  To provide an escape hatch for users, add a new
allow.vmm_ppt jail configuration knob, and check it when handling ppt
ioctls in jails.  Also add a new PRIV_VMM_PPTDEV to replace the use of
PRIV_DRIVER.

PR:             https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=292750
Reviewed by:    corvink
MFC after:      2 weeks
Sponsored by:   The FreeBSD Foundation
Sponsored by:   Klara, Inc.
Differential Revision:  https://reviews.freebsd.org/D55066

c71354030a26900e564f0c80a8abdff7e77b3c9e Mark Johnston 2026-02-16 14:56:25

debug: classified in 04-filenames_plain2 by 'sys/'

pdrfork(2): do require RFPROCDESC | RFPROC

when RFSPAWN is not specified, as stated in the man page.
rfork(2) cannot modify non-curproc, which is why RFPROC is required,
the syscall cannot act on arbitrary process descriptor.
If RFPROCDESC is not specified, the syscall does not make sense,
use rfork(2).

Reported and tested by: pho
Reviewed by:    markj
Sponsored by:   The FreeBSD Foundation
MFC after:      1 week
Differential revision:  https://reviews.freebsd.org/D55306

b05be03ceea0a95f7783ddc890885f34bb4023d4 Konstantin Belousov 2026-02-14 16:38:23

debug: classified in 04-filenames_plain2 by 'sys/'