In D58390#1348832, @markj wrote:

Hmm, doesn't fbsdrun_virtio_msix() need to be updated too?

Cross-compiled aarch64->amd64 and on an amd64 machine.

Ok, I've updated the diff to be relative to the parent one (that one applies cleanly to main).

• fixes the cycle in make
• includes updated vmx.h

Could you try this? I will also updated the diff for amd64.

Refactorted with following changes:

• Migrate from msleep to mtx_sleep. Remove cond vars.
• Remove module and bake the transport within p9fs.

In D58390#1345882, @ziaee wrote:

In D58390#1345816, @markj wrote:

In D58390#1345632, @emaste wrote:

@ziaee Yes I think this should be noted in UPDATING.

RELNOTES would be more appropriate. UPDATING is for users tracking main.

LGTM with Relnotes: yes.

Yes, sorry about that.

Could you commit it as well?

Updated per reviewer comments. D58636 is the parent revision, adding the deprecation warning function.

In D57096#1323730, @kib wrote:

Is there some reference or a spec for the p9fs protocol?

I have made the following changes:

Yes, but. I do not expect that it would be so drastic, and there is probably something else broken. VOP_INACTIVE() is skipped when the current lock in vput() is not exclusive and an immediate upgrade request failed because there are other shared lock owners. At very least this means that there are hold owners on the vnode without refs, which is not typical.

Generally VOP_INACTIVE() is a good place, but a (small) problem with it is that it might be skipped

@dave_freedave.net Could you try a bulk poudriere run now? I seem to have gotten a successful run.