On Tue, Oct 03, 2017 at 03:13:43PM +0000, Bart Van Assche wrote: > > + switch (rq->q->rpm_status) { > > + case RPM_SUSPENDED: > > + return false; > > + case RPM_ACTIVE: > > + return rq->rq_flags & RQF_PM; > > + default: > > + return true; > > + } > > } > > Hello Christoph, > > The old code does not process non-PM requests in the RPM_SUSPENDING mode nor > in the RPM_RESUMING code. I think the new code processes all requests in > these two modes. Was that behavior change intended? Yeah, looks like this version has the RPM_ACTIVE and default cases switched. Back to the drawing board.