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? - 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 {