uhidev: remove unused code

2021-11-07 Thread Anton Lindqvist
Hi, The uhidevsubmatch() routine was imported from NetBSD back in 2002[1] along with the reportid locator. The same locator was removed in 2004[2] making this routine and its defines redundant. Comments? OK? [1] https://github.com/openbsd/src/commit/a66cb31e58a16ed957a52ce1b92aafc08fc4900d [2]

Re: bsd.upgrade fails `umount /mnt` with a single partition disk.

2021-11-07 Thread Yuichiro NAITO
Thanks for the comment. I will see it further more. On 11/8/21 12:45, Theo de Raadt wrote: Doesn't this suggest a disk driver failed to perform/complete a write? Not right at that moment but perhaps in the recent past? Yuichiro NAITO wrote: bsd.upgrade fails and shows following messages with

Re: bsd.upgrade fails `umount /mnt` with a single partition disk.

2021-11-07 Thread Theo de Raadt
Doesn't this suggest a disk driver failed to perform/complete a write? Not right at that moment but perhaps in the recent past? Yuichiro NAITO wrote: > bsd.upgrade fails and shows following messages with a single partition disk. > > ``` > Force checking of clean non-root filesystems? [no] no >

bsd.upgrade fails `umount /mnt` with a single partition disk.

2021-11-07 Thread Yuichiro NAITO
bsd.upgrade fails and shows following messages with a single partition disk. ``` Force checking of clean non-root filesystems? [no] no umount: /mnt: Device busy Can't umount sd0a! ``` sd0 has following partitions. ``` #size offset fstype [fsize bsize cpg] a:

Re: armv7 and arm64 media: fail on missing packages

2021-11-07 Thread Theo de Raadt
This change is wrong, because existance of a file is unsatisfactory -- it might be an OLD file. This will not update it. Official builds do something else. And since official builds try to not encode logic directly into the scripts, it is not documented. Having the builds fail in this way would

Re: regress/rpki-client: test openssl in regress target only

2021-11-07 Thread Theo Buehler
> rpki-client seems to be the one-off under regress/ in this regard. Maybe. I wish I had a better idea than this Index: Makefile === RCS file: /cvs/src/regress/usr.sbin/rpki-client/Makefile,v retrieving revision 1.10 diff -u -p -r1.1

Re: regress/rpki-client: test openssl in regress target only

2021-11-07 Thread Klemens Nanni
On Sun, Nov 07, 2021 at 11:33:29PM +0100, Theo Buehler wrote: > On Sun, Nov 07, 2021 at 10:28:22PM +, Klemens Nanni wrote: > > On Sun, Nov 07, 2021 at 11:16:57PM +0100, Theo Buehler wrote: > > > On Sun, Nov 07, 2021 at 10:13:44PM +, Klemens Nanni wrote: > > > > Spotted in `make obj' from /u

armv7 and arm64 media: fail on missing packages

2021-11-07 Thread Klemens Nanni
I built arm64 media on a new machine and forgot to install firmware from packages. It took me some digging to find out which packages these were, plus it failed in the middle of the build rather than up-front. While here, I noticed that arm64 does not use the "dtb" package like armv7 does, but it

Re: regress/rpki-client: test openssl in regress target only

2021-11-07 Thread Theo Buehler
On Sun, Nov 07, 2021 at 10:28:22PM +, Klemens Nanni wrote: > On Sun, Nov 07, 2021 at 11:16:57PM +0100, Theo Buehler wrote: > > On Sun, Nov 07, 2021 at 10:13:44PM +, Klemens Nanni wrote: > > > Spotted in `make obj' from /usr/src: > > > > > > ===> regress/usr.sbin/rpki-client/libressl > >

Re: regress/rpki-client: test openssl in regress target only

2021-11-07 Thread Klemens Nanni
On Sun, Nov 07, 2021 at 11:16:57PM +0100, Theo Buehler wrote: > On Sun, Nov 07, 2021 at 10:13:44PM +, Klemens Nanni wrote: > > Spotted in `make obj' from /usr/src: > > > > ===> regress/usr.sbin/rpki-client/libressl > > /usr/src/regress/usr.sbin/rpki-client/libressl/obj -> > > /usr/obj

Re: regress/rpki-client: test openssl in regress target only

2021-11-07 Thread Theo Buehler
On Sun, Nov 07, 2021 at 10:13:44PM +, Klemens Nanni wrote: > Spotted in `make obj' from /usr/src: > > ===> regress/usr.sbin/rpki-client/libressl > /usr/src/regress/usr.sbin/rpki-client/libressl/obj -> > /usr/obj/regress/usr.sbin/rpki-client/libressl > Run "pkg_add openssl--%

regress/rpki-client: test openssl in regress target only

2021-11-07 Thread Klemens Nanni
Spotted in `make obj' from /usr/src: ===> regress/usr.sbin/rpki-client/libressl /usr/src/regress/usr.sbin/rpki-client/libressl/obj -> /usr/obj/regress/usr.sbin/rpki-client/libressl Run "pkg_add openssl--%1.1" to run tests against OpenSSL 1.1 SKIPPED ===> re

Re: [PATCH] Change maximum size of /usr/src to 3G for autoinstall

2021-11-07 Thread Otto Moerbeek
On Sun, Nov 07, 2021 at 07:44:57PM +0300, Mikhail wrote: > On Sat, Oct 30, 2021 at 11:39:54AM +0300, Mikhail wrote: > > On Sun, Oct 24, 2021 at 02:17:25PM +0300, Mikhail wrote: > > > On Sun, Oct 24, 2021 at 11:32:26AM +0100, Stuart Henderson wrote: > > > > The minimum needs to go up too, a cvs che

Re: [PATCH] Change maximum size of /usr/src to 3G for autoinstall

2021-11-07 Thread Mikhail
On Sat, Oct 30, 2021 at 11:39:54AM +0300, Mikhail wrote: > On Sun, Oct 24, 2021 at 02:17:25PM +0300, Mikhail wrote: > > On Sun, Oct 24, 2021 at 11:32:26AM +0100, Stuart Henderson wrote: > > > The minimum needs to go up too, a cvs checkout is 1.3G already. > > > > > > (Not that I use auto defaults

Re: poll/select: Lazy removal of knotes

2021-11-07 Thread Visa Hankala
On Sat, Nov 06, 2021 at 06:23:27PM +0100, Martin Pieuchot wrote: > On 06/11/21(Sat) 15:53, Visa Hankala wrote: > > On Fri, Nov 05, 2021 at 10:04:50AM +0100, Martin Pieuchot wrote: > > > New poll/select(2) implementation convert 'struct pollfd' and 'fdset' to > > > knotes (kqueue event descriptors)

net.inet6.icmp6.nd6_debug and RDNSS

2021-11-07 Thread Stuart Henderson
net.inet6.icmp6.nd6_debug is fairly noisy if you advertise nameservers, OK to silence those warnings? Nov 7 00:45:34 bamboo /bsd: nd6_options: unsupported option 25 - option ignored Nov 7 04:26:22 bamboo /bsd: nd6_options: unsupported option 25 - option ignored Nov 7 04:26:26 bamboo /bsd: nd6_o