On 6/26/19 5:31 PM, Paul Walmsley wrote:
Hi Sudeep, Atish,
On Mon, 17 Jun 2019, Atish Patra wrote:
From: Sudeep Holla <sudeep.holla@xxxxxxx>
The current ARM DT topology description provides the operating system
with a topological view of the system that is based on leaf nodes
representing either cores or threads (in an SMT system) and a
hierarchical set of cluster nodes that creates a hierarchical topology
view of how those cores and threads are grouped.
However this hierarchical representation of clusters does not allow to
describe what topology level actually represents the physical package or
the socket boundary, which is a key piece of information to be used by
an operating system to optimize resource allocation and scheduling.
Lets add a new "socket" node type in the cpu-map node to describe the
same.
Signed-off-by: Sudeep Holla <sudeep.holla@xxxxxxx>
Reviewed-by: Rob Herring <robh@xxxxxxxxxx>
This one doesn't apply cleanly here on top of v5.2-rc2, Linus's master
branch, and next-20190626. The reject file is below. Am I missing
a patch?
That's weird. I could apply the patch from any git tree (github or
git.kernel.org) but not from mail or patchworks.
git log doesn't show any recent modifications of that file. I am trying
to figure out what's wrong.
- Paul
--- Documentation/devicetree/bindings/arm/topology.txt
+++ Documentation/devicetree/bindings/arm/topology.txt
@@ -185,13 +206,15 @@ Bindings for cluster/cpu/thread nodes are defined as follows:
4 - Example dts
===========================================
-Example 1 (ARM 64-bit, 16-cpu system, two clusters of clusters):
+Example 1 (ARM 64-bit, 16-cpu system, two clusters of clusters in a single
+physical socket):
cpus {
#size-cells = <0>;
#address-cells = <2>;
cpu-map {
+ socket0 {
cluster0 {
cluster0 {
core0 {
--
Regards,
Atish
