Messages in this thread | | | Date | Mon, 12 Mar 2018 10:46:09 +0100 | From | Ingo Molnar <> | Subject | Re: 答复: [PATCH ] Delete some unusefull operations for centaur CPU/platform |
| |
* Rafael J. Wysocki <rjw@rjwysocki.net> wrote:
> On Monday, March 12, 2018 9:40:33 AM CET Ingo Molnar wrote: > > > > * David Wang <DavidWang@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
| |