Re: efiboot machine exit for armv7 and amd64

2017-08-06 Thread Jonathan Gray
On Sun, Aug 06, 2017 at 09:54:44PM +0200, Mark Kettenis wrote: > This adds a "machine exit" command like we already have on amd64. > Main motivation is that this would help including our bootloader in > the U-Boot regression tests. > > This also changes the code to use RS->ResetSystem() on arm64.

ifstated: add handing of departed interfaces

2017-08-06 Thread Rob Pierce
The following diff adds support for detecting the state change of a departed interface. ifstated is not a very verbose daemon, so this diff quietly does the right thing (i.e. there is no exttra warning about a departing interface). The re-arrival of a departed interface involves re-indexing the

Enable kprintf format attribute in clang

2017-08-06 Thread Mark Kettenis
This makes clang actually do something for kprintf. This is a minimal diff that actually allows some FreeBSD-specific kprintf type specifiers. But I'd prefer to do a proper overhaul of this code upstream first and keep the diffs minimal at this point. ok? Index:

efiboot machine exit for armv7 and amd64

2017-08-06 Thread Mark Kettenis
This adds a "machine exit" command like we already have on amd64. Main motivation is that this would help including our bootloader in the U-Boot regression tests. This also changes the code to use RS->ResetSystem() on arm64. It seems we don't need the Cubox-i workaround there. Still doesn't do

Re: ifstated: improve routing socket error handling

2017-08-06 Thread Sebastian Benoit
Rob Pierce(r...@2keys.ca) on 2017.08.06 00:04:46 -0400: > Improve error checking during processing of routing messages. Handling of > RTM_DESYNC encouraged by deraadt. Regression tests pass. > > I have another diff ready to go that handles interface depature, but I thought > it best to separate

Re: run nd6_expire when needed

2017-08-06 Thread Martin Pieuchot
On 06/08/17(Sun) 17:04, Florian Obser wrote: > On Sun, Aug 06, 2017 at 10:29:09AM -0400, Martin Pieuchot wrote: > > On 03/08/17(Thu) 09:13, Florian Obser wrote: > > > > > > as mpi pointed out in "nd6 address expiration & NET_LOCK() contention" > > > we run nd6_expire every second. That seems a

Re: run nd6_expire when needed

2017-08-06 Thread Florian Obser
On Sun, Aug 06, 2017 at 10:29:09AM -0400, Martin Pieuchot wrote: > On 03/08/17(Thu) 09:13, Florian Obser wrote: > > > > as mpi pointed out in "nd6 address expiration & NET_LOCK() contention" > > we run nd6_expire every second. That seems a bit silly considering > > that we normally have a pltime

Re: ifstated: consistent use of log.c

2017-08-06 Thread Jeremie Courreges-Anglas
On Thu, Aug 03 2017, Rob Pierce wrote: > As a result ifstated.c no longer needs err.h. > > Index: ifstated.c > === > RCS file: /cvs/src/usr.sbin/ifstated/ifstated.c,v > retrieving revision 1.56 > diff -u -p -r1.56

Re: switchd: no need to include err.h

2017-08-06 Thread Jeremie Courreges-Anglas
On Thu, Aug 03 2017, Rob Pierce wrote: > switchd no longer requires err.h. ok jca@ You could also stop including errno.h and syslog.h in parse.y. > Index: packet.c > === > RCS file:

Re: ifstated: improve routing socket error handling

2017-08-06 Thread Jeremie Courreges-Anglas
On Sun, Aug 06 2017, Rob Pierce wrote: > Improve error checking during processing of routing messages. Handling of > RTM_DESYNC encouraged by deraadt. Regression tests pass. > > I have another diff ready to go that handles interface depature, but I thought > it best to separate

Re: run nd6_expire when needed

2017-08-06 Thread Martin Pieuchot
On 03/08/17(Thu) 09:13, Florian Obser wrote: > > as mpi pointed out in "nd6 address expiration & NET_LOCK() contention" > we run nd6_expire every second. That seems a bit silly considering > that we normally have a pltime of a day. > > With a bit of math we can work out when the timer should