Re: get rid of pmap_copy()

2023-04-02 Thread Jonathan Gray
On Sun, Apr 02, 2023 at 07:29:02PM +, Miod Vallat wrote: > pmap_copy() is an optional pmap interface which has never been > implemented. In pure Mary Kondo style, we should thank it for the joy it > brought to CSRG people, and move it to the recycling bin - it's not > going to be implemented an

Re: installer: disk crypto: always correct root disk default

2023-04-02 Thread Richard Toohey
On Sun, 2 Apr 2023 at 1:52 PM, Klemens Nanni wrote: > For install, the root disk defaults to the first available [sw]d* disk. > > The new encryption question creates a softraid volume for you, which is > obviously to be used as root disk. > > But on systems with multiple disks, the next one avail

Re: remove pci_{io,mem}_find

2023-04-02 Thread Jonathan Gray
On Sun, Apr 02, 2023 at 07:09:40PM +, Miod Vallat wrote: > pci_{io,mem}_find() have intentionally been left undocumented, > developers being (rightfully) advised to use pci_mapreg_info() instead. > > The following diff removes these undocumented functions and converts > their last few users. >

get rid of pmap_copy()

2023-04-02 Thread Miod Vallat
pmap_copy() is an optional pmap interface which has never been implemented. In pure Mary Kondo style, we should thank it for the joy it brought to CSRG people, and move it to the recycling bin - it's not going to be implemented anytime soon. Index: share/man/man9/pmap.9 ===

remove pci_{io,mem}_find

2023-04-02 Thread Miod Vallat
pci_{io,mem}_find() have intentionally been left undocumented, developers being (rightfully) advised to use pci_mapreg_info() instead. The following diff removes these undocumented functions and converts their last few users. Bonus changes introduced while looking into this: - correctly cope with

Re: [PATCH] Add -executable, -readable, and -writable options to /usr/bin/find

2023-04-02 Thread Andreas Kusalananda Kähäri
On Sun, Apr 02, 2023 at 10:39:00AM +0200, Kusalananda Kähäri wrote: > On Sat, Apr 01, 2023 at 08:18:56AM +0200, Solène Rapenne wrote: > > Le Fri, 31 Mar 2023 19:40:45 -0700, > > Jared Harper a écrit : > > > > > I have put together a patch that adds -executable, -readable, and > > > -writable to /

Re: [installer] default answer to "Is the disk partition already mounted?"

2023-04-02 Thread Mikhail
On Sat, Apr 01, 2023 at 04:31:59PM +, Klemens Nanni wrote: > 01.04.2023 13:33, Mikhail пишет: > $_answer may be unset, but you still pass it and thus rely on ask_yn()'s > default 'no' answer. > > I'd just default-initialise up front and flip if conditions are met. > This should read and reflec

Re: [PATCH] Add -executable, -readable, and -writable options to /usr/bin/find

2023-04-02 Thread Andreas Kusalananda Kähäri
On Sat, Apr 01, 2023 at 08:18:56AM +0200, Solène Rapenne wrote: > Le Fri, 31 Mar 2023 19:40:45 -0700, > Jared Harper a écrit : > > > I have put together a patch that adds -executable, -readable, and > > -writable to /usr/bin/find. > > > > When I first started working on this patch, I implemented

Re: [PATCH] Add -executable, -readable, and -writable options to /usr/bin/find

2023-04-02 Thread Anthony J. Bentley
Stuart Henderson writes: > On 2023/04/01 11:27, Jared Harper wrote: > > For some reason I haven't received the email from Solène (even after > > requesting it re-sent on lists.openbsd.org; nor is it in spam; I will > > look further into this issue), so I'm adding my reply in-line here: > > Solène's

Re: [PATCH] Add -executable, -readable, and -writable options to /usr/bin/find

2023-04-02 Thread Stuart Henderson
On 2023/03/31 19:40, Jared Harper wrote: > However, I had some difficulty finding the source of access(2) so I could not > vet by assumptions. see sys_access() in sys/kern/vfs_syscalls.c

Re: [PATCH] Add -executable, -readable, and -writable options to /usr/bin/find

2023-04-02 Thread Stuart Henderson
On 2023/04/01 11:27, Jared Harper wrote: > For some reason I haven't received the email from Solène (even after > requesting it re-sent on lists.openbsd.org; nor is it in spam; I will > look further into this issue), so I'm adding my reply in-line here: Solène's domain publishes a DMARC p=reject r