Re: uvm_fault: access_type fixup for wired mapping

2021-01-13 Thread Vitaliy Makkoveev
On Tue, Jan 12, 2021 at 09:41:15AM -0300, Martin Pieuchot wrote: > Diff below moves `access_type' to the context structure passed down to > the various routines and fix a regression introduced in a previous > refactoring. > > `access_type' is overwritten for wired mapping and the value of >

Re: Wireguard: can't remove multiple peers at once.

2021-01-13 Thread YASUOKA Masahiko
Hi, On Thu, 14 Jan 2021 08:54:36 +0900 Yuichiro NAITO wrote: > Does anybody please review my code? > > Yasuoka-san is my coleague of my work. > So, he is interested in this topic. That’s why I CCed this mail. > I don’t mean he is an reviewer. > >> 2021/01/12 11:27、Yuichiro NAITO のメール: >> I

Re: Cache parent's pid as `ps_ppid' and use it instead of `ps_pptr->ps_pid'.

2021-01-13 Thread Martin Pieuchot
On 02/01/21(Sat) 21:54, Vitaliy Makkoveev wrote: > This allows us to unlock getppid(2). Also NetBSD, DragonflyBSD and OSX > do the same. Seems the way to go, two comments below. > Index: kern/exec_elf.c > === > RCS file:

OpenBSD Errata: January 13th, 2021 (carp)

2021-01-13 Thread Sebastian Benoit
Errata patches for the kernel have been released for OpenBSD 6.8. Use of bpf(4) on a carp interface could result in a use after free error. Binary updates for the amd64, i386, and arm64 platforms are available via the syspatch utility. Source code patches can be found on the respective errata

Re: more refactor bgpd route decision process

2021-01-13 Thread Sebastian Benoit
Claudio Jeker(cje...@diehard.n-r-g.com) on 2021.01.13 13:10:23 +0100: > This is another cleanup round of the route decision process. > This time focusing on prefix_cmp(). Make sure that when using > return (a - b) that the results always fits in an int type. > Also make sure the check of the

struct process: mark `ps_oppid' as atomic

2021-01-13 Thread Vitaliy Makkoveev
Subj. It's integer so assignments are atomic. Index: sys/sys/proc.h === RCS file: /cvs/src/sys/sys/proc.h,v retrieving revision 1.304 diff -u -p -r1.304 proc.h --- sys/sys/proc.h 11 Jan 2021 13:55:53 - 1.304 +++

Re: Cache parent's pid as `ps_ppid' and use it instead of `ps_pptr->ps_pid'.

2021-01-13 Thread Vitaliy Makkoveev
On Wed, Jan 13, 2021 at 02:10:25PM -0300, Martin Pieuchot wrote: > On 02/01/21(Sat) 21:54, Vitaliy Makkoveev wrote: > > This allows us to unlock getppid(2). Also NetBSD, DragonflyBSD and OSX > > do the same. > > Seems the way to go, two comments below. > > > @@ -694,6 +694,7 @@

Re: Wireguard: can't remove multiple peers at once.

2021-01-13 Thread Yuichiro NAITO
Does anybody please review my code? Yasuoka-san is my coleague of my work. So, he is interested in this topic. That’s why I CCed this mail. I don’t mean he is an reviewer. > 2021/01/12 11:27、Yuichiro NAITO のメール: > > > Hi. > > I have set up multiple peers in a wg0 interface, > and tried to

Re: tpm(4): don't use tvtohz(9)

2021-01-13 Thread Scott Cheloha
On Fri, Jan 08, 2021 at 08:21:14PM +0100, Florian Obser wrote: > On Fri, Jan 08, 2021 at 11:48:33AM -0600, Scott Cheloha wrote: > > On Fri, Jan 08, 2021 at 05:41:24PM +0100, Mark Kettenis wrote: > > > > Date: Fri, 8 Jan 2021 10:27:38 -0600 > > > > From: Scott Cheloha > > > > > > > > On Wed, Jan

Re: all platforms: isolate hardclock(9) from statclock()

2021-01-13 Thread Scott Cheloha
On Sat, Jan 09, 2021 at 12:52:22PM -0600, Dale Rahn wrote: > The 'magic' here was that MD code could choose to implement statclock (and > set stathz appropriately), or MD code could not care about the multiple > statclock/hardclock interfaces into the scheduler. Also some clock drivers > on a

Re: bgpd refactor route decision process

2021-01-13 Thread Denis Fondras
Le Tue, Jan 12, 2021 at 05:39:02PM +0100, Claudio Jeker a écrit : > This diff changes two things: > - First, it move the kroute update into rde_generate_updates() simplifying > prefix_evaluate a little bit. > > - Second, it changes prefix_evaluate to take an additional argument for the > old

Re: bgpd refactor route decision process

2021-01-13 Thread Claudio Jeker
On Wed, Jan 13, 2021 at 11:24:32AM +0100, Denis Fondras wrote: > Le Tue, Jan 12, 2021 at 05:39:02PM +0100, Claudio Jeker a écrit : > > This diff changes two things: > > - First, it move the kroute update into rde_generate_updates() simplifying > > prefix_evaluate a little bit. > > > > - Second,

more refactor bgpd route decision process

2021-01-13 Thread Claudio Jeker
This is another cleanup round of the route decision process. This time focusing on prefix_cmp(). Make sure that when using return (a - b) that the results always fits in an int type. Also make sure the check of the remote_addr at the end is done properly. The result is probably the same but this