On Mon, Oct 12, 2020 at 1:14 PM Alexander Monakov <amona...@ispras.ru> wrote: > > On Mon, 12 Oct 2020, Rafael J. Wysocki wrote: > > > > @@ -20,7 +20,11 @@ > > > * All CPUs have same idle states as boot CPU > > > * > > > * Chipset BM_STS (bus master status) bit is a NOP > > > - * for preventing entry into deep C-stats > > > + * for preventing entry into deep C-states > > > + * > > > + * CPU will flush caches as needed when entering a C-state via MWAIT > > > > I would rephrase this to mention that the above actually is an assumption. > > This comment block is by itself a list of assumptions.
Ah, OK > It begins with heading > "Design Assumptions" and then lists two assumptions. This patch adds a third > one. > > With that clarified, do you still need me to change this hunk? No need.