lkml.org 
[lkml]   [2019]   [Jun]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v7 1/7] Documentation: DT: arm: add support for sockets defining package boundaries
From
Date
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@arm.com>
>>
>> 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@arm.com>
>> Reviewed-by: Rob Herring <robh@kernel.org>
>
> 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

\
 
 \ /
  Last update: 2019-06-27 04:19    [W:0.106 / U:0.500 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site