Re: [PATCH v8 0/7] Unify CPU topology across ARM & RISC-V
From: Paul Walmsley
Date: Fri Jul 12 2019 - 13:16:44 EST
Folks,
On Thu, 27 Jun 2019, Atish Patra wrote:
> The cpu-map DT entry in ARM can describe the CPU topology in much better
> way compared to other existing approaches. RISC-V can easily adopt this
> binding to represent its own CPU topology. Thus, both cpu-map DT
> binding and topology parsing code can be moved to a common location so
> that RISC-V or any other architecture can leverage that.
>
> The relevant discussion regarding unifying cpu topology can be found in
> [1].
>
> arch_topology seems to be a perfect place to move the common code. I
> have not introduced any significant functional changes in the moved code.
> The only downside in this approach is that the capacity code will be
> executed for RISC-V as well. But, it will exit immediately after not
> able to find the appropriate DT node. If the overhead is considered too
> much, we can always compile out capacity related functions under a
> different config for the architectures that do not support them.
>
> There was an opportunity to unify topology data structure for ARM32 done
> by patch 3/4. But, I refrained from making any other changes as I am not
> very well versed with original intention for some functions that
> are present in arch_topology.c. I hope this patch series can be served
> as a baseline for such changes in the future.
>
> The patches have been tested for RISC-V, ARM64, ARM32 & compile tested for
> x86.
Since these patches touch files across several different architectures,
and thus really should sit in -next for a while; and because it's late in
the merge window, I'm planning to postpone sending these patches upstream
until after v5.3-rc1 is released.
Once v5.3-rc1 is released, let's plan to get these patches rebased and
reposted and into linux-next as soon as possible.
Sorry for the delay here,
- Paul