Re: bgpd.8 control socket path

2021-01-31 Thread Jason McIntyre
On Sun, Jan 31, 2021 at 11:47:47AM -0500, Daniel Jakots wrote: > Hi, > > Since > https://github.com/openbsd/src/commit/8d7b500c2ac3625a5b524c5e150619612af2dd36, > the control socket path is not exactly correct in bgpd.8. > > Here's a diff to be precise. I took the wording from bgpd.conf.5. > >

Re: kcov_remote_register.9: fix grammar error

2021-01-31 Thread Jason McIntyre
On Mon, Feb 01, 2021 at 11:34:49AM +1100, Eddie Youseph wrote: > Hi, > > Small patch to fix grammar error in man page. > fixed, thanks. jmc > Index: share/man/man9/kcov_remote_register.9 > === > RCS file:

Re: radio(4): fix grammar error in man page

2021-01-31 Thread Jason McIntyre
On Mon, Feb 01, 2021 at 10:59:00AM +1100, Eddie Youseph wrote: > Hi, > > A small patch to fix a grammar error in the radio.4 manpage. > fixed, thanks. jmc > Index: radio.4 > === > RCS file: /cvs/src/share/man/man4/radio.4,v >

Re: broadcast simplex checksum

2021-01-31 Thread Alexander Bluhm
On Mon, Feb 01, 2021 at 08:08:56AM +1300, Richard Procter wrote: > - Might the rule disabling checksum offload for broadcasts on IFF_SIMPLEX > interfaces be weakened to disable checksum offload for all broadcast > packets instead? I just copied the condition from ether_resolve():

kcov_remote_register.9: fix grammar error

2021-01-31 Thread Eddie Youseph
Hi, Small patch to fix grammar error in man page. Index: share/man/man9/kcov_remote_register.9 === RCS file: /cvs/src/share/man/man9/kcov_remote_register.9,v retrieving revision 1.5 diff -u -p -u -p -r1.5 kcov_remote_register.9 ---

radio(4): fix grammar error in man page

2021-01-31 Thread Eddie Youseph
Hi, A small patch to fix a grammar error in the radio.4 manpage. Index: radio.4 === RCS file: /cvs/src/share/man/man4/radio.4,v retrieving revision 1.31 diff -u -p -u -p -r1.31 radio.4 --- radio.4 12 Sep 2017 14:57:15 -

Re: broadcast simplex checksum

2021-01-31 Thread Richard Procter
>> On 20/01/2021, at 1:56 PM, Alexander Bluhm wrote: >> >> Hi, >> >> Simplex interfaces reinject broadcast packets back into the IP >> stack. As this is a software features, no hardware checksumming >> occurs. So local broadcast packets are dropped with wrong checksum >> if the underlying

Use correct config descriptor in ugen_set_config

2021-01-31 Thread Thomas Jeunet
Hello tech, in ugen_set_config, the cached config descriptor (ugen.c:213) is obsolete after the call to usbd_set_config_no (ugen.c:220). Use a refreshed value so the next loop account for the correct number of interfaces. I also believe the dev->cdesc is leaked inside usbd_set_config_index but I

bgpd.8 control socket path

2021-01-31 Thread Daniel Jakots
Hi, Since https://github.com/openbsd/src/commit/8d7b500c2ac3625a5b524c5e150619612af2dd36, the control socket path is not exactly correct in bgpd.8. Here's a diff to be precise. I took the wording from bgpd.conf.5. Cheers, Daniel Index: bgpd.8

disklabel: make use of getline(3)

2021-01-31 Thread Christian Weisgerber
Replace fgetln(3) with getline(3). Since getline() returns a C string, we don't need to carry around the length separately. OK? Index: sbin/disklabel/editor.c === RCS file: /cvs/src/sbin/disklabel/editor.c,v retrieving revision

XCP-ng, OpenBSD and network interface changes

2021-01-31 Thread Denis Fondras
I am using XCP-ng with the latest OpenBSD snapshot. Whenever I make an hardware change in networking on the VM (connect or disconnect an interface, change associated network), the VM panics : openbsd# panic: grant table reference 5912 is held by domain 0: frame 0x1f1a4 flags 0x19 Stopped at

snmpd: Add end of sequence tests

2021-01-31 Thread Martijn van Duren
Now that ober_scanf_elements supports '$' lets use it. Here's a first stab by adding it to snmpd. Passing regress and a few manual checks. 'e' still doesn't consume the element, but I've talked it over with rob@, who said that shouldn't get in the way of using this new feature. OK? martijn@

Re: execve -1 errno 12 Cannot allocate memory

2021-01-31 Thread Philippe Meunier
Jonathan Gray wrote: >MAXTSIZ is 128 MB on i386 >see sys/arch/i386/include/vmparam.h Mark Kettenis wrote: >sys/arch/i386/include/vmparam.h has: >#define MAXTSIZ (128*1024*1024) /* max text size */ Thanks to both of you for the pointer! So what about the patch below? I've

Re: npppd(8)/pppac(4): remove dummy TUNSIFMODE ioctl(2) call

2021-01-31 Thread YASUOKA Masahiko
Yes, ok yasuoka On Fri, 29 Jan 2021 14:32:39 +0300 Vitaliy Makkoveev wrote: > Since OpenBSD 6.7 npppd(8) can't work over tun(4) anymore. I propose to > remove dummy TUNSIFMODE ioctl(2) call. > > Index: sys/net/if_pppx.c > === >