bgpd, allow for bigger (prefixlist) macros

2018-09-04 Thread Denis Fondras
Hi, While generating big prefixlists macros with bgpq3 (big like as2914:as-europe-v6 for example), OpenBGPd cannot load the config file (error "string too long"). This diff implements a dynamic buffer to overcome this limitation. Index: parse.y

Re: bgpd, allow for bigger (prefixlist) macros

2018-09-04 Thread Claudio Jeker
On Tue, Sep 04, 2018 at 08:15:23PM +0200, Denis Fondras wrote: > Hi, > > While generating big prefixlists macros with bgpq3 (big like > as2914:as-europe-v6 > for example), OpenBGPd cannot load the config file (error "string too long"). > This diff implements a dynamic buffer to overcome this

bgpd - simplify prefixlen logic

2018-09-04 Thread Claudio Jeker
Currently a lot of different operations are implemented for prefix matching. This is not really needed because most can be expressed with the range expression (prefixlen A - B). The only exceptions are the != and >< operators. Both of them are a bit excotic and may be removed at a later time.

icmp6_mtudisc_clone() tweak

2018-09-04 Thread Michael Mikonos
Hello, The code executed on failure in icmp6_mtudisc_clone() is always the same so it can be declared in one place. Is anyone happy to OK this? - Michael Index: icmp6.c === RCS file: /cvs/src/sys/netinet6/icmp6.c,v retrieving

Re: icmp6_mtudisc_clone() tweak

2018-09-04 Thread Klemens Nanni
OK kn

Re: bgpd diff

2018-09-04 Thread Sebastian Benoit
reads ok Claudio Jeker(cje...@diehard.n-r-g.com) on 2018.09.04 12:56:52 +0200: > Yet another minor bgpd diff from a much bigger diff. This adds > inet4applymask() similar to inet6applymask() and starts using it in a few > places. This makes some of the INET vs INET6 cases more similar. more

Update to table(5) man page

2018-09-04 Thread Matt Schwartz
Below is a diff to clear up the description of the Userinfo table in table(5). I also added an example of how it can be used with an Alias table. Thanks, Matt Index: table.5 === RCS file: /cvs/src/usr.sbin/smtpd/table.5,v retrieving

Re: iostat: add "sp" column for CP_SPIN

2018-09-04 Thread Solene Rapenne
Stuart Henderson wrote: > On 2018/09/04 14:22, Solene Rapenne wrote: > > About munin I'm trying to get a diff accepted upstream to fix cpu plugin and > > talk about this with kirby@. The cpu plugin uses sysctl kern.cp_time. While > > it's not related, I prefer to announce it here so people don't

Re: iostat: add "sp" column for CP_SPIN

2018-09-04 Thread Landry Breuil
On Tue, Sep 04, 2018 at 03:27:29PM +0200, Solene Rapenne wrote: > Stuart Henderson wrote: > > On 2018/09/04 14:22, Solene Rapenne wrote: > > > About munin I'm trying to get a diff accepted upstream to fix cpu plugin > > > and > > > talk about this with kirby@. The cpu plugin uses sysctl

Re: Minor cal(1) man diff

2018-09-04 Thread Jason McIntyre
On Tue, Sep 04, 2018 at 03:14:46PM +0300, Lauri Tirkkonen wrote: > On Tue, Sep 04 2018 14:08:23 +0200, Martijn van Duren wrote: > > Month is only taken into account if year is also specified. > > This is also how POSIX notates the format.[0] > > that synopsis doesn't match this usage though: > >

Re: 6.4-current fatal protection fault in supervisor mode (under qemu)

2018-09-04 Thread Mike Larkin
On Tue, Sep 04, 2018 at 12:31:11AM -0700, Greg Steuck wrote: > I'm trying to build an OpenBSD image for deployment on GCE. Towards this > end I'm running a script on Debian which runs OpenBSD-snapshot in qemu. The > script worked a few times for 6.3 release and then failed once and worked > once

Re: iostat: add "sp" column for CP_SPIN

2018-09-04 Thread Stuart Henderson
On 2018/09/04 14:22, Solene Rapenne wrote: > About munin I'm trying to get a diff accepted upstream to fix cpu plugin and > talk about this with kirby@. The cpu plugin uses sysctl kern.cp_time. While > it's not related, I prefer to announce it here so people don't waste time > fixing it again by

Re: iostat: add "sp" column for CP_SPIN

2018-09-04 Thread Stuart Henderson
On 2018/09/04 15:27, Solene Rapenne wrote: > Stuart Henderson wrote: > > On 2018/09/04 14:22, Solene Rapenne wrote: > > > About munin I'm trying to get a diff accepted upstream to fix cpu plugin > > > and > > > talk about this with kirby@. The cpu plugin uses sysctl kern.cp_time. > > > While >

6.4-current fatal protection fault in supervisor mode (under qemu)

2018-09-04 Thread Greg Steuck
I'm trying to build an OpenBSD image for deployment on GCE. Towards this end I'm running a script on Debian which runs OpenBSD-snapshot in qemu. The script worked a few times for 6.3 release and then failed once and worked once when I set it to use a snapshot. I would've reproduced this under

Re: bgpd, streamline aspath matching

2018-09-04 Thread Claudio Jeker
On Tue, Sep 04, 2018 at 05:49:29PM +0200, Denis Fondras wrote: > On Tue, Sep 04, 2018 at 01:56:12PM +0200, Claudio Jeker wrote: > > On Mon, Aug 27, 2018 at 12:11:43PM +0200, Claudio Jeker wrote: > > > This is in preparation for introducing as-sets (a fast lookup table for > > > when you want to

Re: bgpd, streamline aspath matching

2018-09-04 Thread Denis Fondras
On Tue, Sep 04, 2018 at 01:56:12PM +0200, Claudio Jeker wrote: > On Mon, Aug 27, 2018 at 12:11:43PM +0200, Claudio Jeker wrote: > > This is in preparation for introducing as-sets (a fast lookup table for > > when you want to make sure that your peering partner is realy only passing > > you traffic

Re: bgpd, streamline aspath matching

2018-09-04 Thread Claudio Jeker
On Mon, Aug 27, 2018 at 12:11:43PM +0200, Claudio Jeker wrote: > This is in preparation for introducing as-sets (a fast lookup table for > when you want to make sure that your peering partner is realy only passing > you traffic he should). > > To make as-set possible lets do some cleanup

Re: xidle: launching program on timeout without active-area

2018-09-04 Thread Rob Pierce
On Tue, Sep 04, 2018 at 01:54:10PM +0200, Claudio Jeker wrote: > On Mon, Sep 03, 2018 at 03:49:46PM +0200, Sebastien Marie wrote: > > ping > > I like it, OK claudio@ but I'm not really a X person. I also like it and it works well for me. ok rob@ with the same caveat. Rob > > On Tue, Aug 14,

Minor cal(1) man diff

2018-09-04 Thread Martijn van Duren
Month is only taken into account if year is also specified. This is also how POSIX notates the format.[0] OK? martijn@ [0] http://pubs.opengroup.org/onlinepubs/9699919799/utilities/cal.html Index: cal.1 === RCS file:

Re: iostat: add "sp" column for CP_SPIN

2018-09-04 Thread Stuart Henderson
On 2018/09/04 12:07, Solene Rapenne wrote: > Naoki Fukaumi wrote: > > hi tech@, > > > > new cpu state, CP_SPIN, was added, > > https://marc.info/?l=openbsd-cvs=152630109526317=2 > > > > but there is no column for it in the header of iostat, > > > > $ iostat > > tty sd0

Re: Minor cal(1) man diff

2018-09-04 Thread Lauri Tirkkonen
On Tue, Sep 04 2018 14:08:23 +0200, Martijn van Duren wrote: > Month is only taken into account if year is also specified. > This is also how POSIX notates the format.[0] that synopsis doesn't match this usage though: Alternatively, a single parameter may be given specifying the name

Re: iostat: add "sp" column for CP_SPIN

2018-09-04 Thread Solene Rapenne
Stuart Henderson wrote: > On 2018/09/04 12:07, Solene Rapenne wrote: > > Naoki Fukaumi wrote: > > > hi tech@, > > > > > > new cpu state, CP_SPIN, was added, > > > https://marc.info/?l=openbsd-cvs=152630109526317=2 > > > > > > but there is no column for it in the header of iostat, > > > > > >

bgpd diff

2018-09-04 Thread Claudio Jeker
Yet another minor bgpd diff from a much bigger diff. This adds inet4applymask() similar to inet6applymask() and starts using it in a few places. This makes some of the INET vs INET6 cases more similar. OK? -- :wq Claudio Index: bgpd.h

Re: bgpd diff

2018-09-04 Thread Denis Fondras
On Tue, Sep 04, 2018 at 12:56:52PM +0200, Claudio Jeker wrote: > Yet another minor bgpd diff from a much bigger diff. This adds > inet4applymask() similar to inet6applymask() and starts using it in a few > places. This makes some of the INET vs INET6 cases more similar. > > OK? OK denis@ > --

Re: usbd_close_pipe.9: usbd_abort_pipe is void

2018-09-04 Thread Jason McIntyre
On Sun, Sep 02, 2018 at 06:54:02PM +0300, Artturi Alm wrote: > Hi, > > spotted this while removing return value from usbd_close_pipe, > following cleanup by pirofti@[0], which made usbd_abort_pipe void. > > > -Artturi > > [0]: >

Re: xidle: launching program on timeout without active-area

2018-09-04 Thread Claudio Jeker
On Mon, Sep 03, 2018 at 03:49:46PM +0200, Sebastien Marie wrote: > ping I like it, OK claudio@ but I'm not really a X person. > On Tue, Aug 14, 2018 at 06:15:08AM +0200, Sebastien Marie wrote: > > ping > > > > On Wed, Jul 25, 2018 at 02:13:49PM +0200, Sebastien Marie wrote: > > > On Wed, Jul

iostat: add "sp" column for CP_SPIN

2018-09-04 Thread Naoki Fukaumi
hi tech@, new cpu state, CP_SPIN, was added, https://marc.info/?l=openbsd-cvs=152630109526317=2 but there is no column for it in the header of iostat, $ iostat tty sd0 cpu tin tout KB/t t/s MB/s us ni sy in id 01 11.473 0.04 0 0 0 0 0100

fix "_nfiles" references for crash dump

2018-09-04 Thread Naoki Fukaumi
hi tech@, "_nfiles" was renamed to "_numfiles" by this commit, https://marc.info/?l=openbsd-cvs=147199491615345=2 then fstat/pstat put following errors, # fstat -M bsd.0.core -N bsd.0 fstat: _nfiles: no such symbol # pstat -f -M bsd.0.core -N bsd.0 pstat: kvm_getfiles: _nfiles: no such

Re: Add $daemon_nice to rc.subr

2018-09-04 Thread Thomas de Grivel
> why ? why ? well all interactive process get a quarter range nice priority advance compared to all daemon tasks, at least for a laptop environment it really makes sense. sndiod and ntpd are unaffected by this change. you're right to criticize in that I did not document my code, the point of

Re: Add $daemon_nice to rc.subr

2018-09-04 Thread Thomas de Grivel
plus it's really 6 new lines in rc.subr, no big deal. Le mar. 4 sept. 2018 à 10:53, Thomas de Grivel a écrit : > > why ? well all interactive process get a quarter range nice priority > advance compared to all daemon tasks, at least for a laptop > environment it really makes sense. sndiod and

Re: Add $daemon_nice to rc.subr

2018-09-04 Thread Solène Rapenne
Le 2018-09-04 10:56, Thomas de Grivel a écrit : why ? why ? well all interactive process get a quarter range nice priority advance compared to all daemon tasks, at least for a laptop environment it really makes sense. sndiod and ntpd are unaffected by this change. you're right to criticize in

Re: Add $daemon_nice to rc.subr

2018-09-04 Thread Stuart Henderson
On 2018/09/04 10:57, Thomas de Grivel wrote: > plus it's really 6 new lines in rc.subr, no big deal. 6 lines in rc.subr, but add this and someone will want to add something else (ulimit? setting environment variables?) and it's more, plus the accompanying documentation (which I think reduces the

Re: Add $daemon_nice to rc.subr

2018-09-04 Thread Craig Skinner
Hi Thomas, This was previously discussed and rejected 5 years ago in thread: "Alter daemon scheduling priority with renice for rc.d" See: https://marc.info/?t=13874606661 Subsequently I found login.conf is perfect. Some examples: daemon-cellar:\ :priority=18:\ :tc=daemon:

Re: iostat: add "sp" column for CP_SPIN

2018-09-04 Thread Solene Rapenne
Naoki Fukaumi wrote: > hi tech@, > > new cpu state, CP_SPIN, was added, > https://marc.info/?l=openbsd-cvs=152630109526317=2 > > but there is no column for it in the header of iostat, > > $ iostat > tty sd0 cpu > tin tout KB/t t/s MB/s us ni sy in id >0