From: Saravana Kannan <saravanak@google.com>
To: Rob Herring <robh@kernel.org>
Cc: David Dai <davidai@google.com>,
"Rafael J. Wysocki" <rafael@kernel.org>,
Viresh Kumar <viresh.kumar@linaro.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Conor Dooley <conor+dt@kernel.org>,
Sudeep Holla <sudeep.holla@arm.com>,
Quentin Perret <qperret@google.com>,
Masami Hiramatsu <mhiramat@google.com>,
Will Deacon <will@kernel.org>,
Peter Zijlstra <peterz@infradead.org>,
Vincent Guittot <vincent.guittot@linaro.org>,
Marc Zyngier <maz@kernel.org>,
Oliver Upton <oliver.upton@linux.dev>,
Dietmar Eggemann <dietmar.eggemann@arm.com>,
Pavan Kondeti <quic_pkondeti@quicinc.com>,
Gupta Pankaj <pankaj.gupta@amd.com>,
Mel Gorman <mgorman@suse.de>,
kernel-team@android.com, linux-pm@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v5 1/2] dt-bindings: cpufreq: add virtual cpufreq device
Date: Wed, 31 Jan 2024 10:23:03 -0800 [thread overview]
Message-ID: <CAGETcx8S0oS67oMZsPKk6_MGAtygoHEf_LN1gbcNDEBqRJ4PPg@mail.gmail.com> (raw)
In-Reply-To: <20240131170608.GA1441369-robh@kernel.org>
On Wed, Jan 31, 2024 at 9:06 AM Rob Herring <robh@kernel.org> wrote:
>
> On Fri, Jan 26, 2024 at 04:43:15PM -0800, David Dai wrote:
> > Adding bindings to represent a virtual cpufreq device.
> >
> > Virtual machines may expose MMIO regions for a virtual cpufreq device
> > for guests to read frequency information or to request frequency
> > selection. The virtual cpufreq device has an individual controller for
> > each frequency domain. Performance points for a given domain can be
> > normalized across all domains for ease of allowing for virtual machines
> > to migrate between hosts.
> >
> > Co-developed-by: Saravana Kannan <saravanak@google.com>
> > Signed-off-by: Saravana Kannan <saravanak@google.com>
> > Signed-off-by: David Dai <davidai@google.com>
> > ---
> > .../cpufreq/qemu,cpufreq-virtual.yaml | 110 ++++++++++++++++++
>
> > + const: qemu,virtual-cpufreq
>
> Well, the filename almost matches the compatible.
>
> > +
> > + reg:
> > + maxItems: 1
> > + description:
> > + Address and size of region containing frequency controls for each of the
> > + frequency domains. Regions for each frequency domain is placed
> > + contiguously and contain registers for controlling DVFS(Dynamic Frequency
> > + and Voltage) characteristics. The size of the region is proportional to
> > + total number of frequency domains. This device also needs the CPUs to
> > + list their OPPs using operating-points-v2 tables. The OPP tables for the
> > + CPUs should use normalized "frequency" values where the OPP with the
> > + highest performance among all the vCPUs is listed as 1024 KHz. The rest
> > + of the frequencies of all the vCPUs should be normalized based on their
> > + performance relative to that 1024 KHz OPP. This makes it much easier to
> > + migrate the VM across systems which might have different physical CPU
> > + OPPs.
> > +
> > +required:
> > + - compatible
> > + - reg
> > +
> > +additionalProperties: false
> > +
> > +examples:
> > + - |
> > + // This example shows a two CPU configuration with a frequency domain
> > + // for each CPU showing normalized performance points.
> > + cpus {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > +
> > + cpu@0 {
> > + compatible = "arm,armv8";
> > + device_type = "cpu";
> > + reg = <0x0>;
> > + operating-points-v2 = <&opp_table0>;
> > + };
> > +
> > + cpu@1 {
> > + compatible = "arm,armv8";
> > + device_type = "cpu";
> > + reg = <0x0>;
> > + operating-points-v2 = <&opp_table1>;
> > + };
> > + };
> > +
> > + opp_table0: opp-table-0 {
> > + compatible = "operating-points-v2";
> > +
> > + opp64000 { opp-hz = /bits/ 64 <64000>; };
>
> opp-64000 is the preferred form.
>
> > + opp128000 { opp-hz = /bits/ 64 <128000>; };
> > + opp192000 { opp-hz = /bits/ 64 <192000>; };
> > + opp256000 { opp-hz = /bits/ 64 <256000>; };
> > + opp320000 { opp-hz = /bits/ 64 <320000>; };
> > + opp384000 { opp-hz = /bits/ 64 <384000>; };
> > + opp425000 { opp-hz = /bits/ 64 <425000>; };
> > + };
> > +
> > + opp_table1: opp-table-1 {
> > + compatible = "operating-points-v2";
> > +
> > + opp64000 { opp-hz = /bits/ 64 <64000>; };
> > + opp128000 { opp-hz = /bits/ 64 <128000>; };
> > + opp192000 { opp-hz = /bits/ 64 <192000>; };
> > + opp256000 { opp-hz = /bits/ 64 <256000>; };
> > + opp320000 { opp-hz = /bits/ 64 <320000>; };
> > + opp384000 { opp-hz = /bits/ 64 <384000>; };
> > + opp448000 { opp-hz = /bits/ 64 <448000>; };
> > + opp512000 { opp-hz = /bits/ 64 <512000>; };
> > + opp576000 { opp-hz = /bits/ 64 <576000>; };
> > + opp640000 { opp-hz = /bits/ 64 <640000>; };
> > + opp704000 { opp-hz = /bits/ 64 <704000>; };
> > + opp768000 { opp-hz = /bits/ 64 <768000>; };
> > + opp832000 { opp-hz = /bits/ 64 <832000>; };
> > + opp896000 { opp-hz = /bits/ 64 <896000>; };
> > + opp960000 { opp-hz = /bits/ 64 <960000>; };
> > + opp1024000 { opp-hz = /bits/ 64 <1024000>; };
> > +
> > + };
>
> I don't recall your prior versions having an OPP table. Maybe it was
> incomplete. You are designing the "h/w" interface. Why don't you make it
> discoverable or implicit (fixed for the h/w)?
We also need the OPP tables to indicate which CPUs are part of the
same cluster, etc. Don't want to invent a new "protocol" and just use
existing DT bindings.
> Do you really need it if the frequency is normalized?
Yeah, we can have little and big CPUs and want to emulate different
performance levels. So while the Fmax on big is 1024, we still want to
be able to say little is 425. So we definitely need frequency tables.
> Also, we have "opp-level" for opaque values that aren't Hz.
Still want to keep it Hz to be compatible with arch_freq_scale and
when virtualized CPU perf counters are available.
-Saravana
next prev parent reply other threads:[~2024-01-31 18:23 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-01-27 0:43 [PATCH v5 0/2] Improve VM CPUfreq and task placement behavior David Dai
2024-01-27 0:43 ` [PATCH v5 1/2] dt-bindings: cpufreq: add virtual cpufreq device David Dai
2024-01-31 17:06 ` Rob Herring
2024-01-31 18:23 ` Saravana Kannan [this message]
2024-02-02 15:53 ` Rob Herring
2024-02-04 10:23 ` Marc Zyngier
2024-03-11 11:40 ` Quentin Perret
2024-02-05 8:38 ` Viresh Kumar
2024-02-05 16:39 ` Rob Herring
2024-02-15 11:26 ` Sudeep Holla
2024-05-02 20:17 ` David Dai
2024-05-07 10:21 ` Sudeep Holla
2024-05-17 20:59 ` David Dai
2024-01-27 0:43 ` [PATCH v5 2/2] cpufreq: add virtual-cpufreq driver David Dai
2024-01-31 1:13 ` kernel test robot
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=CAGETcx8S0oS67oMZsPKk6_MGAtygoHEf_LN1gbcNDEBqRJ4PPg@mail.gmail.com \
--to=saravanak@google.com \
--cc=conor+dt@kernel.org \
--cc=davidai@google.com \
--cc=devicetree@vger.kernel.org \
--cc=dietmar.eggemann@arm.com \
--cc=kernel-team@android.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=maz@kernel.org \
--cc=mgorman@suse.de \
--cc=mhiramat@google.com \
--cc=oliver.upton@linux.dev \
--cc=pankaj.gupta@amd.com \
--cc=peterz@infradead.org \
--cc=qperret@google.com \
--cc=quic_pkondeti@quicinc.com \
--cc=rafael@kernel.org \
--cc=robh@kernel.org \
--cc=sudeep.holla@arm.com \
--cc=vincent.guittot@linaro.org \
--cc=viresh.kumar@linaro.org \
--cc=will@kernel.org \
/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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).