> Date: Tue, 3 May 2016 15:59:33 +0800 > From: Ray Lai <[email protected]> > > This commit broke resume on my x200: > > https://marc.info/?l=openbsd-cvs&m=146194866402207&w=2 > > > CVSROOT: /cvs > > Module name: src > > Changes by: [email protected] 2016/04/29 10:49:53 > > > > Modified files: > > sys/arch/amd64/amd64: cpu.c > > sys/arch/i386/i386: cpu.c > > sys/arch/sparc64/sparc64: cpu.c > > > > Log message: > > Call sched_init_cpu() just before booting secondary CPUs. > > > > This prevent the scheduler from scheduling tasks to CPUs not beeing able > > to execute them during the boot process. > > > > ok visa@, kettenis@
Ah, yes. We call cpu_boot_secondary_porcessors() in the resume path, and re-initializing the per-cpu scheduler state at that point loses some state and creates an other set of idel threads. We'll need to find a different solution for the problem that mpi@ was trying to fix. I've reverted the commit for now. Thabns for tracking this down.
