dvmrpctl: fix -Wunused-but-set-variable warning

ok guenther@

EAGAIN handling for imsg_read. OK henning@ benno@

pledge "route" no longer needed for if_indextoname & if_nametoindex

pledge "stdio route"; untested. this has the if_nametoindex() problem as
other *ctl programs using their daemon's log.c, and thus requires "route"
for now. we hope to solve that issue soon.

As done for bgpd recently, rename if_mediatype to if_type in dvrmpd.
Remove unused function get_ifms_type(). No ifmedia in here anymore.
"move forward" deraadt@

cope with the EAGAIN API change for msgbuf_write()
ok benno

do not store a time_t or the result of a time_t division in an int

prompted by deraadt@, ok claudio@ chl@ guenther@

Implement "log verbose" and "log brief" to enable or disable verbose debug
logging. henning, sthen, michele like the idea

Switch the various link state printing codes to use the new if_media
independent LINK_STATE_DESCRIPTIONS. Code is now more or less a one to
one copy of get_linkstate() in route/route.c.
OK henning, michele, sthen, deraadt

make dvmrpd imsg-in-a-lib ready as well.
``put it in'' claudio@, ok eric@

Reflect dvmrpd changes, nbr_cnt is no more needed.

ok norby@

sync synopsis/usage.

specifying int instead of just unsigned is better style

yuck, no cookie for me

add additional link states to report the half duplex / full duplex
state, if known by the driver. this is required to check the full
duplex state without depending on the ifmedia ioctl which can't be
called in the kernel without process context.

ok henning@, brad@

Controller for dvmrpd.

Almost all available information in dvmrpd can be read.
Not connected to builds yet.

ok claudio@

EAGAIN handling for imsg_read. OK henning@ benno@

pledge "route" no longer needed for if_indextoname & if_nametoindex

pledge "stdio route"; untested. this has the if_nametoindex() problem as
other *ctl programs using their daemon's log.c, and thus requires "route"
for now. we hope to solve that issue soon.

As done for bgpd recently, rename if_mediatype to if_type in dvrmpd.
Remove unused function get_ifms_type(). No ifmedia in here anymore.
"move forward" deraadt@

cope with the EAGAIN API change for msgbuf_write()
ok benno

do not store a time_t or the result of a time_t division in an int

prompted by deraadt@, ok claudio@ chl@ guenther@

Implement "log verbose" and "log brief" to enable or disable verbose debug
logging. henning, sthen, michele like the idea

Switch the various link state printing codes to use the new if_media
independent LINK_STATE_DESCRIPTIONS. Code is now more or less a one to
one copy of get_linkstate() in route/route.c.
OK henning, michele, sthen, deraadt

make dvmrpd imsg-in-a-lib ready as well.
``put it in'' claudio@, ok eric@

Reflect dvmrpd changes, nbr_cnt is no more needed.

ok norby@

sync synopsis/usage.

specifying int instead of just unsigned is better style

yuck, no cookie for me

add additional link states to report the half duplex / full duplex
state, if known by the driver. this is required to check the full
duplex state without depending on the ifmedia ioctl which can't be
called in the kernel without process context.

ok henning@, brad@

Controller for dvmrpd.

Almost all available information in dvmrpd can be read.
Not connected to builds yet.

ok claudio@