Bob Beck <[email protected]> wrote: > CVSROOT: /cvs > Module name: src > Changes by: [email protected] 2026/01/25 17:06:47 > > Modified files: > sys/uvm : uvm_pdaemon.c uvmexp.h > > Log message: > Modify uvmexp.paging only with atomic operations. > > This was done in 1.3.1, but in conjunction with some undesirable
1.131 > other changes for where this was changed and used. > > This change makes the value be changed only with atomics, without > changing where or by how much it is changed, and restores the > comment in uvmexp.h to indicate it is changed by atomic ops. > > ok kettenis@ claudio@ Putting these atomic operations back was my proposal. I wrote a version locally and tested it after discovering that a kernel without the atomics still deadlocked the landisk, even after all the upper parts of 1.131 and the whole of 1.138s were ripped out. Then I proposed putting the atomics back -- to the group. Then the commit happened without telling the reason and story for putting the atomics back.
