Looks good and tested with bhyve on SpacemiT k3 (using generic guest image).
SBI IPI works fine. Thanks!

This resolves a bug in the SBI IPI emulation when booting a guest OS in bhyve. The guest could request hart_mask == 0, in that case we indeed shord return SBI_SUCCESS and treat the call as a no-op, as long as hart_mask_base != -1.
Thanks!

another nit

Looks good and works great, only a minor issue

works fine on AIO for a few months

Tested on Spacemit K3. Transfer speed improved significantly. Thanks!

works fine on Spacemit K3, it detects 40 bit address range

looks ok to me

Commit TIMEOUT change separately

address @emaste comments

Style.

• explicit panic if the string does not fit static buffer
• reuse isa_extensions static buffer in the legacy routine

Fix checksum offload for IPV6, both receive and transmit paths.

In D59191#1377476, @mmel wrote:

The driver advertises the following: IFCAP_VLAN_MTU, IFCAP_HWCSUM and IFCAP_HWCSUM_IPV6. What happens if the user enables one of these? I only see that IFCAP_RXCSUM is implemented, albeit probably only partially. In all other cases, the driver either emits an invalid packet, is unable to receive a valid packet, or it panics.

resubmit correct file

remove wmb() on receive path as we don't set any descriptor bits prior setting OWN

implement IP checksum offload on receive path

Address @mhorne comments.

remove debug

• add panic() calls in place of TODO comments
• add wmb()

fix hartid type (should be uint32_t)

include the full patch

In D59695#1370984, @mhorne wrote:

Glad to see this moving forward. I only skimmed the overall structure, and in particular I am not confident about MSI code.

My main concern is about this 'pool' mechanism. Why is it important to preallocate these irqsrc structures, and is there any way to be smarter about it (upper bound, or runtime expansion of the pool as-needed?). Unless I am mistaken there is not a high reuse rate.

remove irq pool to simplify operation. Allocate and register IRQs on demand

• don't claim to be root PIC
• switch to intc bus
• NIRQ: lower down the value to 8192
• allocate memory for pcpu_data and imsic_irq_pool

Initialize the child PIC with interrupts disabled on the root PIC

Another idea draft for discussion

In D59813#1372460, @jrtc27 wrote:

Please go read all of D40474

Hi let me improve it a bit first as it looks incomplete

In D59191#1358222, @mmel wrote:

Because both the DMA and the CPU may write to descriptors simultaneously, the driver must require BUS_DMA_COHERENT memory.

Oh, I did not know eqos(4) was imported from NetBSD!

Thanks. We needed that for many years as detaching was sometimes panic system:)

In D56414#1291425, @jrtc27 wrote:

No support for ucontext.h posix API (looks like legacy)

This does need to work; what does Linux do?

Implement ucontext.h POSIX API

In D57405#1315493, @jrtc27 wrote:

Hm, seems the current version is the same as the arm file, and now it's the same as the aarch64 one? I'm guessing copying from arm is how this came to be...

Regenerate

In D55492#1272968, @adrian wrote:

also - are D55492 and D55564 supposed to be tested together in a stack? which should be applied first?

Apply D55492, then D55564.
Just tested these in qemu: vtnet put behind an IOMMU controller on RISC-V with translation enabled, seem to work fine!

Same toolchain (latest), but without this change a few functions don't have landing pad: copyin/copyout and sched_idletd.

In D55636#1273367, @andrew wrote:

How does it fix the Branch Target exception? The kernel doesn't care about the note, it's just there to ensure all source files are built with BTI enabled.

In D55564#1272503, @sarah.walker2_arm.com wrote:

Change header tag alignment size.

Could you try https://reviews.freebsd.org/D55564 (in addition to this change set) please?

found another bug here, will resubmit