I did an initial smoke test for this phab stack and it worked for me..got a couple questions though

only suggestions, not a request

if possible, it might be worth splitting the ipc changes outside of hot pluggable stack

In D51031#1164660, @glebius wrote:

I am really puzzled how D34550 was created and how did it pass your review :)

use device_get_nameunit(), the attachment displayed should be
vtblkX instead of vtbdX

In D47855#1091215, @emaste wrote:

It's worth adding a comment in the src explaining the ordering

alternate solution committed in 525a177c165740fc697df3de5b92e58b3b41477c

I'm fine with this change as it semantically makes sense and mimics original behavior of pbuf_ctor()

In D43505#991773, @oshogbo wrote:

What about the header fields we have discussed?

In D43505#991783, @imp wrote:

Id also be tempted to always do the requests in host endian and do the happing before we submit. Likewise with the replies. Not sure how feasible that is though.

In D43425#990237, @delphij wrote:

In D43425#990231, @rew wrote:

For ZFS, the argument passed to the shell script is the
pool name to import.

Ah you are right, but the name is not guaranteed to be identical to the filename. Probably zpool import -a -d /dev/$(mdconfig $0).uzip instead? (and maybe -R $1)

incorporate feedback:

• drop vm_page_all_valid() change
• fill added leaf page with mappings

In D43218#985292, @markj wrote:

This seems fine as a stopgap. If you mail me git patches for this and the MTRR review, I'll apply them.

This looks good - thanks for the fix.

address feedback from jhb

address feedback

In D41605#980592, @markj wrote:

Why exactly do we need special handling for FREVOKE?

Do not clear knotes from the TTY until it gets dealloc'ed unless the TTY
is being revoked, in that case delete the knotes when closed is called
on the TTY.

Tested and works as expected - this will be nice to have, thanks.