> On Jun 25, 2020, at 8:58 PM, SAITOH Masanobu <msai...@execsw.org> wrote: > > The shutting_down variable is new and many drivers don't use it yet. > Should we replace > > if (cold) > > to > if (cold || shutting_down) > > Or any other solutions? (e.g setting cold = 1 after splhigh()).
Perhaps we should abstract this, and replace the uses of "cold" that would need to check both? Or perhaps functions like kpause() could handle this internally? I would not suggest setting "cold = 1" at shutdown. -- thorpej