I didn't think that MMC CAM supported the 'partitions' that mmc can do... that's half of the partitions listed...

I think strlcpy is wrong. serial_num and device_id are documented as follows:

In D25969#582988, @mckusick wrote:

A new review? If so, I suggest that you close this one and give a pointer to the new one.

This looks OK to my eyes, modulo the comments about HEXDUMP_PAGER ifdef and how that gets defined... But I'm good either way with that...

rebase

feedback...

Ah, just use the standard definition of offsetof.

this isn't needed either...

this isn't needed, remove it.

Last second tweaks that I missed the need for before a full build.

Much of the problematic elements of this patch can be fixed by using the proper facilities, and/or extending a couple.
I've taken the liberty of doing this work for you and posting the review to https://reviews.freebsd.org/D26218

In D25969#582538, @mckusick wrote:

What is the status with this change?

last minute nits

In D26140#582298, @mjg wrote:

I think one key problem is that there are setups where non-"global" root can provoke event notification -- not only this poses question about proper sanitization of data, it poses the question what to do with flooding the device. For example jailed root can be allowed to mount/unmount filesystems which now triggers a HUGE message.

markj comments

Update per kib/kevans review. Fixed the things I added, but didn't try
to do them all that were pre-existing.

• Retire devctl_notify_f()

Will do another spin... sbuf isn't setup for 'callouts' like this :(

Ping? Are we good? Or have I missed something ?

Ping? Are people OK with implementing 4.4BSD behavior, or do I need to the re-echo twice?

looks good to me.

update zone creation when disable and to cope with the tunable case.

handle destory now.

tweaks

Update per comments from kib@ and my comparison to 4.4BSD and NetBSD's
implementations. We should be closer in edge cases to historic behavior,
if not actually match historic behavior.

update to use sbuf, though in a bit of a naughty way.

crib off NetBSD / our old implementation for how to implement FLUSHO.