On Wed, Jun 21, 2023 at 11:25:27PM +0200, Claudio Jeker wrote: > On Wed, Jun 21, 2023 at 01:03:03PM -0600, Chris Waddey wrote: > > Sorry for breaking the thread, I wasn't subscribed to misc, but found > > this in the archives. > > > > After some testing, it looks like the recent uvm_meter() commit is what > > did this (to my machine at least). > > > > The git commit for that is 71d823ace2523fb9fee2d1ab9b4d92a18d3f5714. > > > > I compiled the commit right before it in the logs and booted no problems > > with a GENERIC.MP kernel config, but that one broke it. > > > > I'm not as familiar with CVS, so apologies for not having the commit > > from there. > > > > Here is the commit message if that helps, though I those on tech will > > know it regardless: > > > > schedcpu, uvm_meter(9): make uvm_meter() an independent timeout > > > > uvm_meter(9) should not base its periodic uvm_loadav() call on the UTC > > clock. It also no longer needs to periodically wake up proc0 because > > proc0 doesn't do any work. schedcpu() itself may change or go away, > > but as kettenis@ notes we probably can't completely remove the concept > > of a "load average" from OpenBSD, given its long Unix heritage. > > > > So, (1) remove the uvm_meter() call from schedcpu(), (2) make > > uvm_meter() an independent timeout started alongside schedcpu() during > > scheduler_start(), and (3) delete the vestigial periodic proc0 wakeup. > > > > With input from deraadt@, kettenis@, and claudio@. deraadt@ cautions > > that this change may confuse administrators who hold the load average > > in high regard. > > > > Thread: https://marc.info/?l=openbsd-tech&m=168710929409153&w=2 > > > > general agreement with this direction from kettenis@ > > ok claudio@ > > > > If I should repost on tech, let me know. > > Just to be sure. Did you verify this with self compiled kernels with and > without that commit? > > Please do not compare self compiled kernels with snapshot kernels since > snapshots may carry additional diffs.
I verified this with self compiled kernels with and without that commit (specifically, the git hash of the last commit for a self-compiled kernel that did not hang on boot was 43e70bb47b84874dbf5bedf42379b789cda2b076, Update ospfd to use new ibuf api). > -- > :wq Claudio >
