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 have

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 > `enter

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 0

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 platf

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 remo

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 +++ sys/sys/

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 @@ process_reparent(s

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 remote

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 pag

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: /cvs/src/sys

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 is

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, it

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 prefi