Why does it depend on other reviews?
If it's part of an open review, please fix it there.
If not, please fix it regardless of the other changes.

In D30947#696642, @mjg wrote:

Since userspace has to alloc all the memory anyway, you can extend the count by say 20% (or even double it if not going past a threshold). Then you can call the kernel and have it fill out as much as it sees fit and return, giving you a magic token which you can use to continue the iteration (which would normally be a row number). Then it is up to the kernel to make the most of what userspace is offering all while limiting its own memory use as it sees fit. This should probably be a new ioctl.

LGTM

Please MFC this to any version containing the "old" ABI, so that no intermediate versions enter the external visible trains.

In D30175#694137, @rozhuk.im-gmail.com wrote:

In D30175#693715, @donner wrote:

May you please add a paragraph to the rc man page, too?

I am not native speaker, it is a bit hard to me to write for humans.

Thank you for adding context to the diff.

First I'm happy to see such a sophisticated use of netgraph in the default rc context.
Thank you for this idea.

In D30811#693218, @zlei.huang_gmail.com wrote:

Can you please share the discussion? Thanks!

Use arc4random_uniform(3) as recommended by Dimitry Luhtionov.

I'm surprised. There was a discussion about not redirecting /32 on interfaces to loopback. So we see the consequence.

How could this happen?

In D30566#691468, @hselasky wrote:

In this example I use 32-bit timestamps, and then it is assumed that the code needs to handle wraparound.

#ifndef _KERNEL
int time_uptime;

static void *
time_cache(void *arg)
{
   while (1) {
    usleep(1000000);
    time_uptime = XXX;
   }
   return (NULL);
}
static void __attribute__((__constructor__))
init_function(void *arg)
{
    pthread_t pt;
    pthread_create(&pt, NULL, time_cache, NULL);
}
#endif

• Remove an unnecessary local var

rebase

Rebase

Rebase

rebase

rebase

rebase

rebase

Rebase

Rebase

Rebase

Rebase

rebase

Rebase

rebase