OpenBSD cvs log

created 2025-01-01T22:05:59Z
begin 2024-12-04T00:00:00Z
end 2024-12-05T00:00:00Z
path src/sys
commits 6

date 2024-12-04T09:19:11Z
author mpi
files src/sys/uvm/uvm_amap.c log diff annotate
src/sys/uvm/uvm_amap.h log diff annotate
src/sys/uvm/uvm_fault.c log diff annotate
src/sys/uvm/uvm_map.c log diff annotate
message Pass the rw_enter(9) type to amap_lock() in preparation for using shared locks.

ok tb@

date 2024-12-04T09:21:06Z
author mpi
files src/sys/uvm/uvm_fault.c log diff annotate
message Document that the original page during a CoW can be unlocked earlier.

ok tb@

date 2024-12-04T09:37:33Z
author mpi
files src/sys/dev/dt/dt_dev.c log diff annotate
message Disallow enabling the same probe multiple times.

From Christian Ludwig.

date 2024-12-04T18:20:46Z
author mvs
files src/sys/net/if_gre.c log diff annotate
src/sys/netinet/in_proto.c log diff annotate
message Unlock gre_sysctl().

Both `gre_allow' and `gre_wccp' are atomically accessed integers. They
could have only '0' and '1' values, so no extra dances around
atomic_load_int(9) required.

ok bluhm

date 2024-12-04T22:24:11Z
author mvs
files src/sys/netinet/in_proto.c log diff annotate
src/sys/netinet/ip_icmp.c log diff annotate
message Push locking down to icmp_sysctl().

Keep locking only for ICMPCTL_REDIRTIMEOUT case. It is complicated, so
left it as is.

ICMPCTL_STATS loads per-CPU counters into local data, so no locking
required.

`icmpctl_vars' are atomically accessed integers. Except `icmperrppslim'
they are simply booleans, so nothing special required. Used the local
`icmperrppslim_local' variable to load `icmperrppslim' value because it
it could have negative values. claudio@ proposed to always load such
values to local variables, so I want to try this notation.

ok bluhm

date 2024-12-04T22:48:41Z
author mvs
files src/sys/netinet/in_proto.c log diff annotate
src/sys/netinet/udp_usrreq.c log diff annotate
src/sys/netinet6/in6_proto.c log diff annotate
message Push locking down to udp_sysctl().

No locks required for per-CPU counters based protocol statistics.
Atomically accessed `udpctl_vars' variables are already moved from the
net lock, sysctl(2) related locks are useless for them.

Complicated UDPCTL_BADDYNAMIC and UDPCTL_ROOTONLY cases were left as is.

ok bluhm