Change sysctl_add_oid()'s "arg2" (which is what the macro's "val" maps to) to intmax_t, to ensure it can hold any value.

Hopefully clean up some merge cruft in man9/Makefile, and correct the types of the "val" arguments in the manpage.

Be careful; there aren't CTASSERTs for the type of val and if you add them it may take a while to flush out all misuses there

Huh. I synced and did `svn diff', and man9/Makefile only shows my additions. (Or is that the problem - once I created the review, I shouldn't sync anymore until it's submitted?)

Typoed and forgot to include sysctl.9

Thanks LN.

Looks good to me.

I'm hardly an expert on the queue macros - or C++ - but this change seems reasonable to me. I assume `make universe' works?

One of my colleagues dug into the first LOR - the one that happens when adding em1 to lagg0, which happens both with and without the patch. It looks like this is a known issue:

4079 addresses @rstone's comments. However, the LOR remains:

Updating D1986: Teach lagg(4) to change MTU

Addressing review comments from rstone:

In D1986#4, @ae wrote:

Just a thought. Imagine two interfaces, one has maximum MTU 2200, another 1500. lagg0 has MTU 1400.
Two threads invokes changing MTU in the same time. One wants to change it to 2000, another - to 1500.
It is possible, that when both threads will finish its job, one interface will have MTU 1400, but another - 1500.
I mean, that such changes should be done exclusively without possibility of races in the ioctl code.