Fix iwx wrong ac indexs.

2021-03-17 Thread zxystd
The indexes into the ac array in the iwx_mac_ctxt_cmd_common are from the iwx_ac enum and not the txfs. The current code therefore puts the edca params in the wrong indexes of the array, causing wrong priority for data-streams of different ACs. This bug is not exists in iwm as its txf number is

Re: Remove booting from kernels in raw/qcow2 images in vmd(8)

2021-03-17 Thread Mike Larkin
On Wed, Mar 17, 2021 at 10:29:32PM +0100, Klemens Nanni wrote: > On Sun, Mar 14, 2021 at 11:00:22AM -0400, Dave Voutila wrote: > > Any takers? > Yes, I plan to commit the updated diff at the end until friday > unless someone objects. > no objection, thanks everyone. ok mlarkin > > Here's an upda

Re: Remove booting from kernels in raw/qcow2 images in vmd(8)

2021-03-17 Thread Klemens Nanni
On Sun, Mar 14, 2021 at 11:00:22AM -0400, Dave Voutila wrote: > Any takers? Yes, I plan to commit the updated diff at the end until friday unless someone objects. > Here's an updated diff also removes some logic in config.c related to > checking the value sent by vmctl(8)'s -b flag to see if it's

Re: Remove booting from kernels in raw/qcow2 images in vmd(8)

2021-03-17 Thread Klemens Nanni
On Wed, Mar 17, 2021 at 08:37:26PM +0100, Grégoire Jadi wrote: > I don't know if this patch was committed already, but there is a mention > of ramdisk in vm.conf(5). > > Here is a small patch to replace the example with a boot on disk. That example is fine and not effected by the proposed removal.

Re: Remove booting from kernels in raw/qcow2 images in vmd(8)

2021-03-17 Thread Grégoire Jadi
Dave Voutila writes: Hi, > Any takers? > > Here's an updated diff also removes some logic in config.c related to > checking the value sent by vmctl(8)'s -b flag to see if it's the same as > the root disk image (-d). I don't know if this patch was committed already, but there is a mention of ram

Re: mpw: make "ifconfig mpw0 -mplslabel" work

2021-03-17 Thread Claudio Jeker
On Wed, Mar 17, 2021 at 07:09:23PM +0100, Klemens Nanni wrote: > With that the ifconfig(8) bits (which are still pending in the other > tech@ thread) actually hold up. > > Code is there, noone seemed to have used it so far, though. > > OK? OK claudio@ > Index: if_mpw.c > ==

mpw: make "ifconfig mpw0 -mplslabel" work

2021-03-17 Thread Klemens Nanni
With that the ifconfig(8) bits (which are still pending in the other tech@ thread) actually hold up. Code is there, noone seemed to have used it so far, though. OK? Index: if_mpw.c === RCS file: /cvs/src/sys/net/if_mpw.c,v retrievin

IPv6: allow only temporary global addresses

2021-03-17 Thread Florian Obser
RFC 8981 allows this and it reduces the amount of v6 addresses I have on my laptop. OK? diff --git sbin/ifconfig/ifconfig.c sbin/ifconfig/ifconfig.c index 2c60e652675..527e6e02d1f 100644 --- sbin/ifconfig/ifconfig.c +++ sbin/ifconfig/ifconfig.c @@ -1579,8 +1579,11 @@ setautoconf(const char *cmd,

mpe, mpip, mpw: Only install route with label, fix leak on destroy

2021-03-17 Thread Klemens Nanni
`ifconfig mp* mplslabel N' validates the label both in ifconfig and each driver's ioctl handler, but there is one case where all drivers install a route without looking at the label at all. SIOCSLIFPHYRTABLE in all three drivers just validates the rdomain and sets the label to itself (0 after clon

Re: unlock sys_sendsyslog

2021-03-17 Thread Alexander Bluhm
On Thu, Mar 11, 2021 at 12:40:37AM +0300, Vitaliy Makkoveev wrote: > Since UNIX domain sockets are unlocked it makes sense to unlock > sys_sendsyslog too. Console output still requires kernel lock to be > held but this path is only followed while `syslogf' socket is not set. > > New `syslogf_rwloc

Re: LibreSSL: extend the max. no. of SANs for avoid OOM error

2021-03-17 Thread Renaud Allard
On 3/17/21 11:02 AM, Stuart Henderson wrote: On 2021/03/17 10:53, Jan Klemkow wrote: ping On Tue, Mar 09, 2021 at 03:49:32PM +0100, Jan Klemkow wrote: Hi, The verification of the https://ugos.ugm.ac.id certificate contains 2032 subject alt names which leads to the following error in LibreSS

Re: LibreSSL: extend the max. no. of SANs for avoid OOM error

2021-03-17 Thread Jan Klemkow
ping On Tue, Mar 09, 2021 at 03:49:32PM +0100, Jan Klemkow wrote: > Hi, > > The verification of the https://ugos.ugm.ac.id certificate contains 2032 > subject alt names which leads to the following error in LibreSSL. > > # openssl s_client -connect ugos.ugm.ac.id:443 > ... > verify error:num=17:

Re: mpip, mpw: Use correct rdomain when adding/deleting routes

2021-03-17 Thread Claudio Jeker
On Tue, Mar 16, 2021 at 11:33:53PM +0100, Klemens Nanni wrote: > Found while reading the code and testing ifconfig(8)'s `tunneldomain' in > order to document MPLS ioctls (see other thread on tech@). > > mpe(4) consistently uses the softc's rdomain which is tracked > consistently across the various

Re: ntpd adjtime offset race

2021-03-17 Thread Claudio Jeker
On Tue, Mar 16, 2021 at 03:05:23PM +0100, Alexander Bluhm wrote: > Hi, > > I am running ntpd as a client with three upstream servers. Some > of them are not synchonized and report a time that is off by several > seconds. > > The ntpd client code corrects both T1 and T4 with the current offset >

bgpd code cleanup

2021-03-17 Thread Claudio Jeker
Base gcc warns about 'rp' may be used uninitialized. Now the code is correct and rp is not use uninitalized but it is not as obvious as it could be. Also rp is a bit of a shitty name for the redo queue tail pointer. So cleanup the code and with it stop the warning. OK? -- :wq Claudio Index: rde_

Re: LibreSSL: extend the max. no. of SANs for avoid OOM error

2021-03-17 Thread Stuart Henderson
On 2021/03/17 10:53, Jan Klemkow wrote: > ping > > On Tue, Mar 09, 2021 at 03:49:32PM +0100, Jan Klemkow wrote: > > Hi, > > > > The verification of the https://ugos.ugm.ac.id certificate contains 2032 > > subject alt names which leads to the following error in LibreSSL. > > > > # openssl s_clien