Yeah, I was a bit borderline on this change. I think we already don't document the dragonfly-vinum entry in this manpage though.

I think you can use graid() without BIOS support if you don't need to boot from it? Maybe though "software RAID systems such as graid(8) or ZFS"? gconcat(8) and gstripe(8) are no better than the simple mirroring being described in this context

Oh, did this not land yet?

As I note in the other review, this has been unused since:

I do have some old out-of-tree changes to rework how ithreads worked that did change the grouping, but even in that case we could accommodate this.

This was a transition aid before all architectures implemented the proper pcib_if.m methods. It can go away now.

Rebase

Hmm, I'm not sure. Presumably you could create the alternate be, mount it, then do make installworld installetc installkernel DESTDIR=/foo'?

Add asserts

@markj can you test this change on your Ampere box?

Hmm, Warner notes that this does not resolve his reported issue, but his reported issue is also not a regression? (I had been waiting to MFC all my rman changes to stable/14 under the impression it introduced a regression)

This seems sensible to me.

Rebase

Reduce diff

Generate dle from connected socket, avoiding duplicate sockets

Add a note about other threads being stopped

Ping

I've created a PR to do an exp-run both this and D43237 at https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=284039

It would be tempting to hoist the pages calculation out of the this inner loop and just doing if ((bz->map_count + 1) * pages < bz->bz_total_pages && bz->bz_total_pages < maxpages) as the condition. You'd have to basically add 1 more to the map count if BUS_DMA_MIN_ALLOC_COMP was set (and then stop setting the flag in the else). That would avoid continuously reserving pages up to the bz_total_pages limit if you just keep destroying and freeing maps (e.g. if a NIC driver destroyed all its rings and maps for ifconfig down/up). However, if multiple tags share the same zone you can perhaps get starvation across tags. If we wanted to avoid that problem, we could instead track how many pages a tag has allocated so far, and then the expression would be if ((bz->map_count + 1) * pages < dmat->bounce_pages && bz->bz_total_pages < maxpages). That also nicely handles the BUS_DMA_MIN_ALLOC_COMP case and we could drop that flag entirely.

Fix build and rebase

In D48373#1104076, @kib wrote:

In D48373#1104032, @jhb wrote:

@kib btw while documenting this, I do wonder if we shouldn't just merge bus_print_child_domain into bus_print_child_footer?

Why? On the principle that it is not used by anything else? But there are already approx. 10 callers of the function.

I'm fine with this. Don't the locking implementations just bail without panicking though if a panic is already active, or is the nested panic not inside a lock but elsewhere?

I do not know how to actually test this but it builds again. I had to do a hack to build tools/sysinit and then symlink that into my path to get stand/usb to actually build. I can run the test program but it doesn't do anything (and I'm not sure how to get it to do anything).

This syntax is fine

Push curthread check out to callers

Indeed, it is listed for 14.0 through 14.2, and we failed to mention the removal of twe(4) in the 14.0 release notes (we did mention twa(4) and several other drivers)

Not sure if we need to retroactively fix any release notes? It was removed in 14.0 and later.

@allanjude, @kevans, @markj in reference to testing on large arm64 boxes with multiple top-level pcibX devices

stand/usb still does not build though. Removing devclass from DRIVER_MODULE has broken the kshim bits quite a bit and needs some reworking.

@kib btw while documenting this, I do wonder if we shouldn't just merge bus_print_child_domain into bus_print_child_footer?