i386 ioapic mtx not initialized

2021-09-02 Thread Martin Pieuchot
Seen with WITNESS, this has already been fixed in amd64, diff below backport the fix, ok? ioapic0 at mainbus0: apid 2 pa 0xfec0witness: lock_object uninitialized: 0xd8841440 Starting stack trace... witness_checkorder(f5547000,fec01000,fec0,d1820adc,d03fb01e) at witness_checkorder+0x85

Re: update to tcpdump(8)

2021-09-02 Thread Theo de Raadt
Jason McIntyre wrote: > On Thu, Sep 02, 2021 at 12:44:56AM -0600, Theo de Raadt wrote: > > Jason McIntyre wrote: > > > > > then i guess i would propose doing exactly that: removing the bulk of > > > the text describing primitives and qualifiers and leave a pointer to > > > pcap-filter.3. we

Re: [PATCH] [src] etc/services - duplicates

2021-09-02 Thread Raf Czlonka
Ping. On Mon, May 24, 2021 at 05:05:31PM BST, Raf Czlonka wrote: > Ping. > > On Sun, May 16, 2021 at 07:10:22PM BST, Raf Czlonka wrote: > > Hello, > > > > During recent services(5)-related threads, I glanced over the file > > and noticed a duplicate - namely(sic!), "nameserver" is being used >

Re: update to tcpdump(8)

2021-09-02 Thread Theo de Raadt
Jason McIntyre wrote: > then i guess i would propose doing exactly that: removing the bulk of > the text describing primitives and qualifiers and leave a pointer to > pcap-filter.3. we could leave a brief description of the main > qualifiers, and perhaps just a list of valid keywords for the

Re: timeout: Prettify man page and usage

2021-09-02 Thread Job Snijders
On Thu, Sep 02, 2021 at 07:23:26AM +0100, Jason McIntyre wrote: > > .Ar time > > -can be integer or decimal numbers. > > +are positive integer or real (decimal) numbers, with an optional > > can you have a negative timeout? Negative values are not permitted $ timeout -- -1 /bin/ls timeout:

Re: update to tcpdump(8)

2021-09-02 Thread Jason McIntyre
On Wed, Sep 01, 2021 at 08:28:14PM +0200, Denis Fondras wrote: > Le Wed, Sep 01, 2021 at 06:42:54PM +0100, Jason McIntyre a ?crit : > > On Wed, Sep 01, 2021 at 06:15:04PM +0200, Denis Fondras wrote: > > > I was searching for the sampling command of tcpdump but could not find it > > > in the > > >

Re: Removal of old users and groups in the upgrade notes

2021-09-02 Thread Raf Czlonka
Ping. On Mon, May 24, 2021 at 05:06:08PM BST, Raf Czlonka wrote: > Ping. > > On Sun, May 09, 2021 at 01:07:15PM BST, Raf Czlonka wrote: > > Hello, > > > > This is both a general question and specific example of removal of > > old users and groups. > > > > With the release of 6.7, rebound(8)

Re: timeout: Prettify man page and usage

2021-09-02 Thread Stuart Henderson
On 2021/09/02 08:56, Job Snijders wrote: > On Thu, Sep 02, 2021 at 07:23:26AM +0100, Jason McIntyre wrote: > > > .Ar time > > > -can be integer or decimal numbers. > > > +are positive integer or real (decimal) numbers, with an optional > > > > can you have a negative timeout? > > Negative

Re: update to tcpdump(8)

2021-09-02 Thread Jason McIntyre
On Thu, Sep 02, 2021 at 12:44:56AM -0600, Theo de Raadt wrote: > Jason McIntyre wrote: > > > then i guess i would propose doing exactly that: removing the bulk of > > the text describing primitives and qualifiers and leave a pointer to > > pcap-filter.3. we could leave a brief description of the

Re: timeout: Prettify man page and usage

2021-09-02 Thread Jason McIntyre
On Wed, Sep 01, 2021 at 09:29:35PM +0200, Leon Fischer wrote: > Here's my thanks for importing timeout(1). > > P.S. The wording could still be improved, especially the -k description. > hi. > Index: timeout.1 > === > RCS file:

Re: timeout: Prettify man page and usage

2021-09-02 Thread Martijn van Duren
On Thu, 2021-09-02 at 08:56 +, Job Snijders wrote: > On Thu, Sep 02, 2021 at 07:23:26AM +0100, Jason McIntyre wrote: > > > .Ar time > > > -can be integer or decimal numbers. > > > +are positive integer or real (decimal) numbers, with an optional > > > > can you have a negative timeout? > >

atactl: remove few printf("%s", NULL)

2021-09-02 Thread Sebastien Marie
Hi, While looking at my worktree, I found some old changes I have locally which could be commited. Here one to remove few printf("%s", NULL) I saw long time ago in atactl. I don't remember exactly when I saw them, which field trigger syslog entry, and if I would still see them. Comments or OK ?

timeout: execvp(2) return is always an error

2021-09-02 Thread Sebastien Marie
Hi, If execvp(2) returns, it is always an error: there is no need to check if the return value is -1. Just unconditionally call err(3). Comments or OK ? -- Sebastien Marie timeout: execvp(2) should not return except on error diff 7821223d9093e8d64d2bab7db6b91e28360fdab3

Re: i386 ioapic mtx not initialized

2021-09-02 Thread Mark Kettenis
> Date: Thu, 2 Sep 2021 09:31:49 +0200 > From: Martin Pieuchot > > Seen with WITNESS, this has already been fixed in amd64, diff below > backport the fix, ok? > > ioapic0 at mainbus0: apid 2 pa 0xfec0witness: lock_object uninitialized: > 0xd8841440 > Starting stack trace... >

Re: Removal of old users and groups in the upgrade notes

2021-09-02 Thread Florian Obser
On 2021-09-02 12:26 +02, Sebastian Benoit wrote: > Raf Czlonka(rczlo...@gmail.com) on 2021.09.02 10:51:19 +0100: >> Ping. >> >> On Mon, May 24, 2021 at 05:06:08PM BST, Raf Czlonka wrote: >> > Ping. >> > >> > On Sun, May 09, 2021 at 01:07:15PM BST, Raf Czlonka wrote: >> > > Hello, >> > > >> > >

iwx(4): fix xtal latency values

2021-09-02 Thread Stefan Sperling
While review device-specific quirks in this driver I noticed that the xtal latency values we send to the chip do no match those used by the Linux driver. ok? diff b81ef55c86817a4ccf18086fd9b7dc3ee49ae415 /usr/src (staged changes) blob - 096caf79896dcd97f16f0744fb8206ad8a12a9d7 blob +

Re: acpibtn.4: Mention sleep putton, lid status and machdep.{lid,pwr}action

2021-09-02 Thread Klemens Nanni
On Wed, Sep 01, 2021 at 07:12:54AM +0100, Jason McIntyre wrote: > hi. i think this is a good change - it makes the page more helpful. > i have only one tweak, inline: Thanks. I'll wait a bit but commit eventually unless someone has more feedback/objection. > > +The lid status is set up as

Re: iwx(4): fix xtal latency values

2021-09-02 Thread Kevin Lo
On Thu, Sep 02, 2021 at 02:24:17PM +0200, Stefan Sperling wrote: > While review device-specific quirks in this driver I noticed that > the xtal latency values we send to the chip do no match those used > by the Linux driver. > > ok? One small nit below. With that fixed, ok kevlo@ > diff

[PATCH] [src] etc/root/root.mail - /usr/X11R6/README no longer provided

2021-09-02 Thread Raf Czlonka
Hello, This diff is simply to just highlight the fact that the file is not longer provided[0], not necessarily remove the information outright and not replace it with anything - I simply can't think of anything concise right now. [0]

Re: timeout: execvp(2) return is always an error

2021-09-02 Thread Ingo Schwarze
Hi Sebastien, Sebastien Marie wrote on Thu, Sep 02, 2021 at 09:09:43AM +0200: > If execvp(2) returns, it is always an error: there is no need to check > if the return value is -1. Just unconditionally call err(3). > > Comments or OK ? OK schwarze@ Ingo > timeout: execvp(2) should not

Re: timeout: Prettify man page and usage

2021-09-02 Thread Theo de Raadt
Jason McIntyre wrote: > On Thu, Sep 02, 2021 at 08:56:29AM +, Job Snijders wrote: > > On Thu, Sep 02, 2021 at 07:23:26AM +0100, Jason McIntyre wrote: > > > > .Ar time > > > > -can be integer or decimal numbers. > > > > +are positive integer or real (decimal) numbers, with an optional > > >

Re: Atomic signal flags for vi(1)

2021-09-02 Thread Ingo Schwarze
Hi Tim, trondd wrote on Wed, Sep 01, 2021 at 08:46:28PM -0400: > Ingo Schwarze wrote: >> Ingo Schwarze wrote on Wed, Sep 01, 2021 at 04:38:51PM +0200: >>> Note that the h_hup() and h_term() signal handlers are still unsafe >>> after this commit because they also set the "killersig" (how

Re: async traceroute(8)

2021-09-02 Thread Florian Obser
On 2021-09-01 04:05 -06, "Theo de Raadt" wrote: > Stuart Henderson wrote: > >> On 2021/09/01 11:25, Florian Obser wrote: >> > So traceroute sends one probe, waits upto 5^W3 seconds for an answer to >> > arrive, sends the next probe and so on. >> > >> > This makes it a bit faster (10x on a path

iwm/iwx suspend/resume improvement

2021-09-02 Thread Stefan Sperling
This patch fixes suspend/resume with an AX201 device for gnezdo@. Tests on any iwm/iwx device would be apreciated. Before testing this make sure to update your tree to -current which contains a very recent fix for a double-free in the resume path of the iwx driver. You won't have great results

Re: timeout: Prettify man page and usage

2021-09-02 Thread Sebastian Benoit
ok Martijn van Duren(openbsd+t...@list.imperialat.at) on 2021.09.02 11:05:24 +0200: > On Thu, 2021-09-02 at 08:56 +, Job Snijders wrote: > > On Thu, Sep 02, 2021 at 07:23:26AM +0100, Jason McIntyre wrote: > > > > .Ar time > > > > -can be integer or decimal numbers. > > > > +are positive

Re: Atomic signal flags for vi(1)

2021-09-02 Thread Martijn van Duren
On Thu, 2021-09-02 at 13:25 +0200, Ingo Schwarze wrote: > Hi Tim, > > trondd wrote on Wed, Sep 01, 2021 at 08:46:28PM -0400: > > Ingo Schwarze wrote: > > > Ingo Schwarze wrote on Wed, Sep 01, 2021 at 04:38:51PM +0200: > > > > > Note that the h_hup() and h_term() signal handlers are still unsafe

Re: enable cu(4) in amd64/GENERIC by default

2021-09-02 Thread Theo de Raadt
lacking full cables is not a driver problem, and users can learn to cope with that. ok deraadt Jan Klemkow wrote: > Hi, > > The card and cables don't have the signaling lines, that getty to use it > as com(4). But with "local" in ttys(5) it works. I have this driver in > productive use for

Re: Removal of old users and groups in the upgrade notes

2021-09-02 Thread Sebastian Benoit
Raf Czlonka(rczlo...@gmail.com) on 2021.09.02 10:51:19 +0100: > Ping. > > On Mon, May 24, 2021 at 05:06:08PM BST, Raf Czlonka wrote: > > Ping. > > > > On Sun, May 09, 2021 at 01:07:15PM BST, Raf Czlonka wrote: > > > Hello, > > > > > > This is both a general question and specific example of

Re: timeout: Prettify man page and usage

2021-09-02 Thread Jason McIntyre
On Thu, Sep 02, 2021 at 08:56:29AM +, Job Snijders wrote: > On Thu, Sep 02, 2021 at 07:23:26AM +0100, Jason McIntyre wrote: > > > .Ar time > > > -can be integer or decimal numbers. > > > +are positive integer or real (decimal) numbers, with an optional > > > > can you have a negative

fix iwm/iwx firmware reloading

2021-09-02 Thread Stefan Sperling
iwm(4) reloads firmware from disk whenever the device is initialized. This includes suspend/resume, where accessing the disk is not a good idea. It seems we have been lucky because iwm(4) defers the wakeup step into a task context, such that disk has resumed when this task gets to run. But this is

iked(8): make proto option accept lists

2021-09-02 Thread Tobias Heider
The diff below makes iked accept a list of protocols for the "proto" config option in iked.conf(5). This would allow us to have a single policy with "proto { ipencap, ipv6 }" to secure a gif(4) tunnel, instead of requiring one policy for each protocol. ok? Index: iked.h

Re: iwx(4): fix xtal latency values

2021-09-02 Thread Stefan Sperling
On Thu, Sep 02, 2021 at 09:32:28PM +0800, Kevin Lo wrote: > On Thu, Sep 02, 2021 at 02:24:17PM +0200, Stefan Sperling wrote: > > While review device-specific quirks in this driver I noticed that > > the xtal latency values we send to the chip do no match those used > > by the Linux driver. > > >

mg(1): 8-bit clean; let ASCII be the default

2021-09-02 Thread sinas
If the terminal LOCALE is set to UTF-8, isprint() considers the first UTF-8 sequence as printable which makes Mg to print one invisible character and the rest of a UTF-8 sequence as \octal. Instead, we can print isascii() bytes, and display everything else -- except control characters -- as

Re: update to tcpdump(8)

2021-09-02 Thread Theo de Raadt
I think the following approach will work. 1. changes from tcpdump.8 -r1.00 to -rHEAD need merging into pcap-filter.5 2) DESCRIPTION from pcap-filter body is then copied into tcpdump, replacing the same chunk. What I see is far more authoritative. 3) EXAMPLES copied from pcap-filter, and

Re: iwm/iwx suspend/resume improvement

2021-09-02 Thread Florian Obser
This survived multiple suspend / resumes on iwm0 at pci1 dev 0 function 0 "Intel Dual Band Wireless-AC 9260" rev 0x29, msix iwm0: hw rev 0x320, fw ver 46.6b541b68.0, address 40:74:e0:38:11:11 thanks On 2021-09-02 15:26 +02, Stefan Sperling wrote: > This patch fixes suspend/resume with an

Re: update to tcpdump(8)

2021-09-02 Thread Jason McIntyre
On Thu, Sep 02, 2021 at 03:33:17AM -0600, Theo de Raadt wrote: > Jason McIntyre wrote: > > > On Thu, Sep 02, 2021 at 12:44:56AM -0600, Theo de Raadt wrote: > > > Jason McIntyre wrote: > > > > > > > then i guess i would propose doing exactly that: removing the bulk of > > > > the text

Re: update to tcpdump(8)

2021-09-02 Thread Jason McIntyre
On Thu, Sep 02, 2021 at 04:27:41PM +0200, Denis Fondras wrote: > Le Thu, Sep 02, 2021 at 07:49:25AM +0100, Jason McIntyre a ?crit : > > why not just paste in the body of pcap-filter in then and we can try and > > keep them in sync thereafter? > > > > OK, I will do that. I am not confident it

Re: update to tcpdump(8)

2021-09-02 Thread Denis Fondras
Le Thu, Sep 02, 2021 at 07:49:25AM +0100, Jason McIntyre a écrit : > why not just paste in the body of pcap-filter in then and we can try and > keep them in sync thereafter? > OK, I will do that. I am not confident it will stay in sync over time :D

Re: [Patch] Document /upgrade.site in sysupgrade(8) man page

2021-09-02 Thread Aaron Poffenberger
Any further thoughts on this patch to the man page? Cheers, --Aaron On 2021-08-28 12:53 -0500, Aaron Poffenberger wrote: > On 2021-08-28 19:45 +0200, Sebastien Marie wrote: > > On Sat, Aug 28, 2021 at 05:05:18PM +, Klemens Nanni wrote: > > > On Sat, Aug 28, 2021 at 10:44:48AM -0500, Aaron

Re: timeout: Prettify man page and usage

2021-09-02 Thread Jason McIntyre
On Thu, Sep 02, 2021 at 05:47:09PM +0200, Leon Fischer wrote: > > > @@ -34,83 +35,74 @@ > > > .Nd run a command with a time limit > > > .Sh SYNOPSIS > > > .Nm > > > -.Op Fl Fl signal Ar sig | Fl s Ar sig > > > -.Op Fl Fl preserve-status > > > -.Op Fl Fl kill-after Ar time | Fl k Ar time > > >

Re: async traceroute(8)

2021-09-02 Thread Florian Obser
On 2021-09-02 15:00 +02, Florian Obser wrote: > On 2021-09-01 04:05 -06, "Theo de Raadt" wrote: >> Stuart Henderson wrote: >> >>> On 2021/09/01 11:25, Florian Obser wrote: >>> > So traceroute sends one probe, waits upto 5^W3 seconds for an answer to >>> > arrive, sends the next probe and so on.

Re: timeout: Prettify man page and usage

2021-09-02 Thread Todd C . Miller
On Thu, 02 Sep 2021 11:05:24 +0200, Martijn van Duren wrote: > There are a few cases where we don't set errno, but do use err(3). OK millert@ - todd

enable cu(4) in amd64/GENERIC by default

2021-09-02 Thread Jan Klemkow
Hi, The card and cables don't have the signaling lines, that getty to use it as com(4). But with "local" in ttys(5) it works. I have this driver in productive use for about 5 years now. OK? bye, Jan Index: arch/amd64/conf/GENERIC

pmap & buffer cache dummy pagers

2021-09-02 Thread Martin Pieuchot
Diff below introduces two dummy pagers for subsystem that manipulate UVM objects that are 'special'. Those pagers will be used to enforce checks in functions that expect a lock to be held, like: KASSERT(obj == NULL || UVM_OBJ_IS_PMAP(obj) || rw_write_held(obj->vmobjlock));

Re: timeout: Prettify man page and usage

2021-09-02 Thread Jason McIntyre
On Thu, Sep 02, 2021 at 10:41:32PM +0200, Leon Fischer wrote: > > > > yes, fair point. i also dislike excess markup. but i think in the first > > sentence it is not excess, it is explanation. i mean "duration"is marked > > up. > > > > so that first sentence should try to talk about all those arg

Re: Removal of old users and groups in the upgrade notes

2021-09-02 Thread Sebastian Benoit
Florian Obser(flor...@openbsd.org) on 2021.09.02 14:04:22 +0200: > On 2021-09-02 12:26 +02, Sebastian Benoit wrote: > > Raf Czlonka(rczlo...@gmail.com) on 2021.09.02 10:51:19 +0100: > >> Ping. > >> > >> On Mon, May 24, 2021 at 05:06:08PM BST, Raf Czlonka wrote: > >> > Ping. > >> > > >> > On

Re: timeout: Prettify man page and usage

2021-09-02 Thread ropers
> Changed to "duration and time may contain a decimal fraction. The value > defaults to seconds unless an unit-specifying suffix is given." a unit, not an unit It's phonetic; it depends on pronunciation, not on the way it's spelled. Hence "a unit" /ˈjuːnɪt/ but "an unreal amount" /ʌnˈrɪəl/.

Re: Removal of old users and groups in the upgrade notes

2021-09-02 Thread Sebastian Benoit
Sebastian Benoit(be...@openbsd.org) on 2021.09.02 21:41:15 +0200: > Florian Obser(flor...@openbsd.org) on 2021.09.02 14:04:22 +0200: > > On 2021-09-02 12:26 +02, Sebastian Benoit wrote: > > > Raf Czlonka(rczlo...@gmail.com) on 2021.09.02 10:51:19 +0100: > > >> Ping. > > >> > > >> On Mon, May 24,

rm.1: remove extraneous word

2021-09-02 Thread Evan Silberman
Speaking of the first sentence of rm(1): Remove extraneous word from command description "non-directory files" reads more naturally and means the same thing as "non-directory type files". diff --git a/bin/rm/rm.1 b/bin/rm/rm.1 index a2526a36392..1be2bf31913 100644 --- a/bin/rm/rm.1 +++

Re: rm.1: remove extraneous word

2021-09-02 Thread lists
Thu, 02 Sep 2021 14:28:54 -0700 Evan Silberman > Speaking of the first sentence of rm(1): Where is that discussion happening, how is it concerning the rm(1) manual page? > Remove extraneous word from command description What is the rationale for this, is it entirely missing or undisclosed

Re: rm.1: remove extraneous word

2021-09-02 Thread Evan Silberman
> On Sep 2, 2021, at 4:05 PM, li...@wrant.com wrote: > > Do all manual pages need to be free form texts to suit non-technical > audiences? Yes? Who do you think the documentation is for?

Re: rm.1: remove extraneous word

2021-09-02 Thread Jason McIntyre
On Thu, Sep 02, 2021 at 02:28:54PM -0700, Evan Silberman wrote: > Speaking of the first sentence of rm(1): > > Remove extraneous word from command description > > "non-directory files" reads more naturally and means the same thing as > "non-directory type files". > true. i wonder if it was

Re: systat(1) counter overflow

2021-09-02 Thread Anindya Mukherjee
On Mon, Aug 30, 2021 at 11:11:55AM +0200, Martin Pieuchot wrote: > On 13/07/21(Tue) 00:55, Anindya Mukherjee wrote: > > On Sat, Jul 03, 2021 at 11:20:42AM +0100, Stuart Henderson wrote: > > > On 2021/07/03 01:09, Anindya Mukherjee wrote: > > > > Thanks for the discussion. This has been very

Re: iwm/iwx suspend/resume improvement

2021-09-02 Thread Kevin Lo
On Thu, Sep 02, 2021 at 03:26:03PM +0200, Stefan Sperling wrote: > > This patch fixes suspend/resume with an AX201 device for gnezdo@. > Tests on any iwm/iwx device would be apreciated. > > Before testing this make sure to update your tree to -current which contains > a very recent fix for a