On Fri, 25 Dec 2020 at 02:31, Lina Iyer <ilina@xxxxxxxxxxxxxx> wrote: > > On Tue, Dec 22 2020 at 03:16 -0700, Ulf Hansson wrote: > >On Wed, 16 Dec 2020 at 18:51, Lina Iyer <ilina@xxxxxxxxxxxxxx> wrote: > >> > >> In order to debug critical domain and device power issues, it may be > >> necessary to disallow certain idle states at runtime. Let the device > >> disallow a domain idle state before suspending.The domain governor shall > >> check for the 'disabled' flag while determining the domain idle state. > > > >For debug purposes, you might as well just set a dev PM Qos latency > >request that corresponds to the state you want to disable. This will > >then prevent the genpd governor from selecting the state. > > > True, but it will also disable idle states deeper as well. Would like to > avoid that. I see. In any case, I am not so excited about adding an exported genpd interface for this that drivers can call, for example. Then I would rather see a proper debugfs interface, where userspace can both see the available states and choose which one to disable/enable. Would that work? [...] Kind regards Uffe