Re: km_alloc(9) for i386 pmap

2021-04-23 Thread Mike Larkin
On Fri, Apr 23, 2021 at 08:07:43PM +0200, Martin Pieuchot wrote: > Diff below convert the last uses of uvm_km_alloc(9) and uvm_km_zalloc(9) > to km_alloc(9). > > One of the allocations below uses `kp_pageable' instead of `kp_zero' > because the mapping for `pm_pdir_intel' is lost when PAE is

Re: km_alloc(9) for i386 pmap

2021-04-23 Thread Alexander Bluhm
On Fri, Apr 23, 2021 at 08:07:43PM +0200, Martin Pieuchot wrote: > Diff below convert the last uses of uvm_km_alloc(9) and uvm_km_zalloc(9) > to km_alloc(9). > > One of the allocations below uses `kp_pageable' instead of `kp_zero' > because the mapping for `pm_pdir_intel' is lost when PAE is

Re: arpinit_done arpinit()

2021-04-23 Thread Vitaliy Makkoveev
On Fri, Apr 23, 2021 at 11:19:01PM +0200, Alexander Bluhm wrote: > Hi, > > Setting variable arpinit_done is not MP save if we want ot execute > arp_rtrequest() in parallel. Move initialization to arpinit() > function. > > ok? > ok mvs@ > bluhm > > Index: netinet/if_ether.c >

Re: arpinit_done arpinit()

2021-04-23 Thread Mark Kettenis
> Date: Fri, 23 Apr 2021 23:19:01 +0200 > From: Alexander Bluhm > Content-Type: text/plain; charset=us-ascii > Content-Disposition: inline > > Hi, > > Setting variable arpinit_done is not MP save if we want ot execute > arp_rtrequest() in parallel. Move initialization to arpinit() > function.

arpinit_done arpinit()

2021-04-23 Thread Alexander Bluhm
Hi, Setting variable arpinit_done is not MP save if we want ot execute arp_rtrequest() in parallel. Move initialization to arpinit() function. ok? bluhm Index: netinet/if_ether.c === RCS file:

Re: Unlock top part of uvm_fault()

2021-04-23 Thread Christian Weisgerber
Martin Pieuchot: > Diff below remove the KERNEL_LOCK()/UNLOCK() dance from uvm_fault() for > both amd64 and sparc64. That means the kernel lock will only be taken > for lower faults and some amap/anon code will now run without it. > > I'd be interested to have this tested and see how much does

Re: sysctl net.inet.ip.arpqueued read only

2021-04-23 Thread Vitaliy Makkoveev
> On 23 Apr 2021, at 20:39, Alexander Bluhm wrote: > > Hi, > > The variable la_hold_total contains the number of packets currently > in the arp queue. So the sysctl net.inet.ip.arpqueued must be read > only. In if_ether.c include the header with the decalration of > la_hold_total to ensure

Re: km_alloc(9) for i386 pmap

2021-04-23 Thread Mark Kettenis
> Date: Fri, 23 Apr 2021 20:07:43 +0200 > From: Martin Pieuchot > > Diff below convert the last uses of uvm_km_alloc(9) and uvm_km_zalloc(9) > to km_alloc(9). > > One of the allocations below uses `kp_pageable' instead of `kp_zero' > because the mapping for `pm_pdir_intel' is lost when PAE is

km_alloc(9) for i386 pmap

2021-04-23 Thread Martin Pieuchot
Diff below convert the last uses of uvm_km_alloc(9) and uvm_km_zalloc(9) to km_alloc(9). One of the allocations below uses `kp_pageable' instead of `kp_zero' because the mapping for `pm_pdir_intel' is lost when PAE is enabled and need to be re-established when a fault happens. This is consistent

Re: forwarding in parallel

2021-04-23 Thread Alexander Bluhm
On Thu, Apr 22, 2021 at 08:34:35PM +0200, Hrvoje Popovski wrote: > r620-1# ppupaavnamn_iinccif::ca :u p > lpptooo(ooo0llx__lcc_faacfcafccfhhhfeef__eii_fttifet8eme2m_m2__4m > m3aamgga9iig0ci8cc,__ cc_hhc0eehxcebcc,kk ::k0 :mm ,bbm ub1ufu)fpf > pp-ll> lcc ppceup > fkfrerfererneee e e llli:ils

Re: umm_map returns unaligned address?

2021-04-23 Thread Mike Larkin
On Fri, Apr 23, 2021 at 01:55:14PM +0200, Alessandro Pistocchi wrote: > Hi all, > > I am fairly new to openbsd so if this is something obvious that I missed > please be understanding. > > I am adding a syscall to openbsd 6.8. I am working on a raspberry pi. > > During the syscall I allocate some

sysctl net.inet.ip.arpqueued read only

2021-04-23 Thread Alexander Bluhm
Hi, The variable la_hold_total contains the number of packets currently in the arp queue. So the sysctl net.inet.ip.arpqueued must be read only. In if_ether.c include the header with the decalration of la_hold_total to ensure that the definition matches. ok? bluhm Index: netinet/if_ether.c

Re: Driver for Cadence UART

2021-04-23 Thread Mark Kettenis
> Date: Fri, 23 Apr 2021 14:59:03 + > From: Visa Hankala > Cc: tech@openbsd.org > Content-Type: text/plain; charset=utf-8 > Content-Disposition: inline > > On Fri, Apr 23, 2021 at 04:26:34PM +0200, Mark Kettenis wrote: > > > Date: Fri, 23 Apr 2021 12:54:54 + > > > From: Visa Hankala > >

Re: [External] : Re: pf_state_key_link_reverse() is prone to race on parallel forwarding

2021-04-23 Thread Alexander Bluhm
On Fri, Apr 23, 2021 at 01:01:26AM +0200, Alexandr Nedvedicky wrote: > sure, updated diff is below. Thanks for analysis. OK bluhm@ > > looks like there must be one more difference between OpenBSD and genua. > It looks like genua-pf is able to run state expiration concurrently with

Re: Driver for Cadence UART

2021-04-23 Thread Visa Hankala
On Fri, Apr 23, 2021 at 04:26:34PM +0200, Mark Kettenis wrote: > > Date: Fri, 23 Apr 2021 12:54:54 + > > From: Visa Hankala > > > > This diff adds an initial driver for Cadence UART. This controller block > > is present at least on Xilinx Zynq system-on-chips. > > > > I have tested the

Re: Driver for Cadence UART

2021-04-23 Thread Mark Kettenis
> Date: Fri, 23 Apr 2021 12:54:54 + > From: Visa Hankala > > This diff adds an initial driver for Cadence UART. This controller block > is present at least on Xilinx Zynq system-on-chips. > > I have tested the driver on Zynq-7000. The driver is functional enough > to give a working console

Driver for Cadence UART

2021-04-23 Thread Visa Hankala
This diff adds an initial driver for Cadence UART. This controller block is present at least on Xilinx Zynq system-on-chips. I have tested the driver on Zynq-7000. The driver is functional enough to give a working console on RAMDISK. The driver might work on Zynq UltraScale+ as well, but that I

umm_map returns unaligned address?

2021-04-23 Thread Alessandro Pistocchi
Hi all, I am fairly new to openbsd so if this is something obvious that I missed please be understanding. I am adding a syscall to openbsd 6.8. I am working on a raspberry pi. During the syscall I allocate some memory that I want to share between the kernel and the calling process. When it's

Re: re-enable A-MSDU support with iwm(4) and iwx(4) fixed

2021-04-23 Thread Stefan Sperling
On Fri, Apr 23, 2021 at 12:28:42PM +0200, Matthias Schmidt wrote: > I had a new kernel with only your following patch running all day and > never encountered the situation as described in my last email. > Connection was stable and transfer rates remained around 3MB/s. This is > less then the

Re: re-enable A-MSDU support with iwm(4) and iwx(4) fixed

2021-04-23 Thread Matthias Schmidt
Hi Stefan, * Stefan Sperling wrote: > On Thu, Apr 22, 2021 at 07:47:29PM +0200, Matthias Schmidt wrote: > > I have a kernel with your patch running since several hours and > > noticed a regression. My usual "test case" is copying several large > > files from my file server via NFSv3 to my

Re: re-enable A-MSDU support with iwm(4) and iwx(4) fixed

2021-04-23 Thread Stefan Sperling
On Thu, Apr 22, 2021 at 07:47:29PM +0200, Matthias Schmidt wrote: > I have a kernel with your patch running since several hours and > noticed a regression. My usual "test case" is copying several large > files from my file server via NFSv3 to my laptop. In the beginning the > transfer rate was

Re: Unlock top part of uvm_fault()

2021-04-23 Thread Janne Johansson
Den tors 22 apr. 2021 kl 23:13 skrev Scott Bennett : > > Diff below remove the KERNEL_LOCK()/UNLOCK() dance from uvm_fault() for > > both amd64 and sparc64. That means the kernel lock will only be taken > > for lower faults and some amap/anon code will now run without it. > > > > I'd be