* Rafael J. Wysocki <r...@rjwysocki.net> wrote:

> On Monday, March 12, 2018 9:40:33 AM CET Ingo Molnar wrote:
> > 
> > * David Wang <davidw...@zhaoxin.com> wrote:
> > 
> > > [David] pr->flags.has_cst means BIOS define valid C state table.  And at 
> > > lease 
> > > define 2 entries. On all centaur platform which support C3, this 
> > > condition is 
> > > always true.
> > 
> > > [David] Just as the following comment said, we need not execute WBINVD 
> > > and 
> > > ARB_DISABLE/ARB_ENABLE when entering C3 if bm_check=1 and bm_control=0.  
> > > This 
> > > logic is valid for all platform not only for Centaur, I think.
> > 
> > Ok, fair enough!
> 
> Well, I still have this one queued up as cpuidle material.
> 
> I can drop it if you want to take it instead.

No need to drop it, it looks good to me!

Thanks,

        Ingo

Reply via email to