ospf6d: sync database.c with ospfd(8)

2020-01-02 Thread Denis Fondras
This is mostly log messages sync. Index: database.c === RCS file: /cvs/src/usr.sbin/ospf6d/database.c,v retrieving revision 1.18 diff -u -p -r1.18 database.c --- database.c 23 Dec 2019 07:33:49 - 1.18 +++ database.c 2 Jan

vmctl(8): uninitialized value

2020-01-02 Thread Benjamin Baier
Hi. "case CMD_SEND:" sets done=1 so ret will never be written to and the uninitialized value of ret is used to determine the return value of the function vmmaction. -- Ben Index: main.c === RCS file:

Re: snmpd(8): filter pf table addresses

2020-01-02 Thread Sebastian Benoit
Claudio Jeker(cje...@diehard.n-r-g.com) on 2019.12.31 11:44:07 +0100: > On Tue, Dec 31, 2019 at 11:16:37AM +0100, Martijn van Duren wrote: > > I'm on the fence about this. So if you feel strongly about this go > > ahead if it works. > > In some regard I agree but in this case I think it makes

ospf6d: sync hello.c with ospfd

2020-01-02 Thread Denis Fondras
Sync with ospfd's hello.c Index: hello.c === RCS file: /cvs/src/usr.sbin/ospf6d/hello.c,v retrieving revision 1.21 diff -u -p -r1.21 hello.c --- hello.c 23 Dec 2019 11:25:41 - 1.21 +++ hello.c 2 Jan 2020 16:11:19

Re: Add -R alias to -r for scp(1)

2020-01-02 Thread Ingo Schwarze
Hi Marc, Marc Espie wrote on Thu, Jan 02, 2020 at 11:30:35AM +0100: > And if I use scp enough, I'm also likely to use cp -r by mistake. > > Are we likely to actually remove cp -r so the second mistake > doesn't happen ? I wouldn't be opposed to that. It has been deprecated since rev. 1.1 in

Re: TIMESPEC_TO_NSEC(), futex(2) & __thrsleep(2)

2020-01-02 Thread Martin Pieuchot
On 01/01/20(Wed) 22:13, Scott Cheloha wrote: > > On Dec 31, 2019, at 9:35 AM, Martin Pieuchot wrote: > > > > I'd like to stop converting the given timespec to ticks and instead use > > nanoseconds. This is part of the ongoing effort to reduce the use of > > `hz' through the kernel. > > > >

Re: vmctl(8): uninitialized value

2020-01-02 Thread Klemens Nanni
On Thu, Jan 02, 2020 at 04:37:17PM +0100, Benjamin Baier wrote: > "case CMD_SEND:" sets done=1 so ret will never be written to and > the uninitialized value of ret is used to determine the return > value of the function vmmaction. Good catch: $ doas vmctl start -b ~/vm/bsd.rd -m 128M test

Re: sparc64: find root device on hardware RAID

2020-01-02 Thread Klemens Nanni
On Tue, Dec 31, 2019 at 09:12:56AM +1000, Jonathan Matthew wrote: > I think this is probably the most sensible thing we can do here. > ok jmatthew@ but I'd wait and see if anyone has a better idea. I'll commit later this evening so that snapshots will just work on my machine, this makes upgrading

Re: FAQ4.html Commit Revert

2020-01-02 Thread Andras Farkas
For me, faq4.html displays fine in Firefox, lynx, and Chromium, both with and without the CSS. On Thu, Jan 2, 2020 at 10:47 AM Oleg Pahl wrote: > > Hi @All, > > > > could you be so kind to revert this commit in FAQ 4? > >

Re: vmctl(8): uninitialized value

2020-01-02 Thread Pratik Vyas
* Benjamin Baier [2020-01-02 22:01:14 +0100]: On Thu, 2 Jan 2020 18:56:14 +0100 Klemens Nanni wrote: On Thu, Jan 02, 2020 at 04:37:17PM +0100, Benjamin Baier wrote: > "case CMD_SEND:" sets done=1 so ret will never be written to and > the uninitialized value of ret is used to determine the

Re: TIMESPEC_TO_NSEC(), futex(2) & __thrsleep(2)

2020-01-02 Thread Scott Cheloha
> On Jan 2, 2020, at 9:02 AM, Martin Pieuchot wrote: > > On 01/01/20(Wed) 22:13, Scott Cheloha wrote: >>> On Dec 31, 2019, at 9:35 AM, Martin Pieuchot wrote: >>> >>> I'd like to stop converting the given timespec to ticks and instead use >>> nanoseconds. This is part of the ongoing effort to

Re: vmctl(8): uninitialized value

2020-01-02 Thread Benjamin Baier
On Thu, 2 Jan 2020 18:56:14 +0100 Klemens Nanni wrote: > On Thu, Jan 02, 2020 at 04:37:17PM +0100, Benjamin Baier wrote: > > "case CMD_SEND:" sets done=1 so ret will never be written to and > > the uninitialized value of ret is used to determine the return > > value of the function vmmaction. >

Re: PATCH: fix race condition in binutils-2.17 build

2020-01-02 Thread Philip Guenther
On Thu, Jan 2, 2020 at 2:26 AM Marc Espie wrote: > We don't normally hit this thanks to the "expensive job" heuristics > (meaning that we don't start the second target while the first is > building), but still there is a race, and if for whatever reason we > remove that heuristics we hit it

Re: Add -R alias to -r for scp(1)

2020-01-02 Thread Ingo Schwarze
Hi Kurt, Kurt Mosiejczuk wrote on Wed, Jan 01, 2020 at 08:21:04PM -0500: > cp(1) uses -R for recursive copy. scp(1) uses -r. This diff adds -R > as an alias for -r to scp(1) for those assuming consistency with cp(1). even if cp -R and scp -r did the same thing - which, if i understand tedu@

Re: Add -R alias to -r for scp(1)

2020-01-02 Thread Marc Espie
On Thu, Jan 02, 2020 at 11:19:30AM +0100, Ingo Schwarze wrote: > Hi Kurt, > > Kurt Mosiejczuk wrote on Wed, Jan 01, 2020 at 08:21:04PM -0500: > > > cp(1) uses -R for recursive copy. scp(1) uses -r. This diff adds -R > > as an alias for -r to scp(1) for those assuming consistency with cp(1). > >

Re: ospf6d: remove useless orig_rtr_lsa()

2020-01-02 Thread Remi Locherer
On Tue, Dec 31, 2019 at 01:47:08PM +0100, Denis Fondras wrote: > Rename orig_rtr_lsa_area() to orig_rtr_lsa() > > Now that area is part of iface, original orig_rtr_lsa() is useless. Also > verifying that area != NULL is not needed in some cases (these are leftovers > of > the previous diff). >

PATCH: fix race condition in binutils-2.17 build

2020-01-02 Thread Marc Espie
We don't normally hit this thanks to the "expensive job" heuristics (meaning that we don't start the second target while the first is building), but still there is a race, and if for whatever reason we remove that heuristics we hit it fairly hard. The patch is fairly trivial, just run both