From: Huang Rui <ray.huang@amd.com>
To: Borislav Petkov <bp@alien8.de>
Cc: Len Brown <lenb@kernel.org>,
"Rafael J. Wysocki" <rjw@rjwysocki.net>,
<linux-kernel@vger.kernel.org>,
Fengguang Wu <fengguang.wu@intel.com>,
"Aaron Lu" <aaron.lu@intel.com>, Tony Li <tony.li@amd.com>,
Thomas Gleixner <tglx@linutronix.de>
Subject: Re: Mwait usage on AMD processors
Date: Thu, 14 May 2015 18:17:28 +0800 [thread overview]
Message-ID: <20150514101727.GA6552@hr-slim.amd.com> (raw)
In-Reply-To: <20150514092137.GA29235@pd.tnic>
On Thu, May 14, 2015 at 11:21:37AM +0200, Borislav Petkov wrote:
> On Thu, May 14, 2015 at 02:54:52PM +0800, Huang Rui wrote:
> > Hi Len, Rafael, and all,
> >
> > AMD proposed a new instruction named mwaitx. This is an extension of
> > mwait with a configurable timer (mwaitx = mwait + timer). And mwaitx
> > will act as mwait if timer is disabled. However, mwait/mwaitx cannot
> > let cpu core go to C1 state at current AMD processors, but has less
> > power consumption even at C0 while core is waiting.
> > As you know, mwait/mwaitx would have better performance than halt. So
> > could we propose an implementation to use mwaitx at idle call in boot
> > phase and cpuidle driver after boot phase.
>
> I'd select it in select_idle_routine()...
>
I know, actually, I already completed it in my local side, and
mwaitx_idle will be called with x86_idle at boot phase. :)
I plan to continue to implement it with cpuidle driver if mwaitx_idle
can be accepted.
> > And the mwaitx idle is exposed to user as an optional kernel
> > parameter(idle=...), and decided by user.
>
> If MWAITX is better than C1 (and C1E, which we're doing now AFAIR)
> then I'd enable it by default on those machines so that it is used
> automatically.
>
At current processors, MWAITX cannot go to C1 state, but has faster
waiting exit speed. MWAITX is still at C0 but less power consumption.
So for now, I just want to set it optionally.
HW designer will do deeper low power state (C1, and more) in future
processors. At that time, we can enable it by default.
Thanks,
Rui
next prev parent reply other threads:[~2015-05-14 10:17 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-14 6:54 Mwait usage on AMD processors Huang Rui
2015-05-14 9:21 ` Borislav Petkov
2015-05-14 10:17 ` Huang Rui [this message]
2015-05-14 11:23 ` Borislav Petkov
2015-05-14 13:38 ` Huang Rui
2015-05-14 14:20 ` Borislav Petkov
2015-05-19 7:25 ` Huang Rui
2015-05-19 8:50 ` Borislav Petkov
2015-05-19 9:42 ` Huang Rui
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150514101727.GA6552@hr-slim.amd.com \
--to=ray.huang@amd.com \
--cc=aaron.lu@intel.com \
--cc=bp@alien8.de \
--cc=fengguang.wu@intel.com \
--cc=lenb@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rjw@rjwysocki.net \
--cc=tglx@linutronix.de \
--cc=tony.li@amd.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox