[PATCH] arm64: acpi: reenumerate topology ids

On Thu, Jun 28, 2018 at 12:12:00PM -0500, Jeremy Linton wrote:

Hi,

On 06/28/2018 11:30 AM, Sudeep Holla wrote:

[...]

quoted

I am not sure if we can ever guarantee that DT and ACPI will get the
same ids whatever counter we use as it depends on the order presented in
the firmware(DT or ACPI). So I am not for generating ids for core and
threads in that way.

So I would like to keep it simple and just have this counters for
package ids as demonstrated in Shunyong's patch.

So, currently on a non threaded system, the core id's look nice because they
are just the ACPI ids. Its the package id's that look strange, we could just
fix the package ids, but on threaded machines the threads have the nice acpi
ids, and the core ids are then funny numbers. So, I suspect that is driving
this as much as the strange package ids.

Yes, I know that and that's what made be look at topology_get_acpi_cpu_tag
For me, if the PPTT has valid ID, we should use that. Just becuase DT lacks
it and uses counter doesn't mean ACPI also needs to follow that.

I am sure some vendor will put valid UID and expect that to be in the
sysfs.

(and as a side, I actually like the PE has a acpi id behavior, but for
threads its being lost with this patch...)

Given i've seen odd package/core ids on x86s a few years ago, it never
bothered me much as a lot of userspace tools are just using what is
effectively the logical processor number anyway.

Indeed.

Further, this table may be having some clarifications published for some of
these fields. I'm not sure the final wording will help us, but it might.

I prefer that. We can then just use the IDs if valid, else offset if
PPTT ignores those fields. What's the point in having all these in
specification and vendors ignoring them altogether. Since this is new
feature, we can always enforce and say we don't care if firmware doesn't,
sorry.

quoted

Also looking @ topology_get_acpi_cpu_tag again now, we should have
valid flag check instead of level = 0. Jeremy ?

? I'm not sure I understand, but your saying for the leaf nodes we should be
checking the valid flag rather than whether the caller requested level 0?

I don't think that is right. The original PPTT spec is unclear about proper
use of the valid flag. So, while this part of the spec may be clarified in
the near future, (AFAIK) there are already tables in the wild which fail to
set valid on the leaf nodes! So I think using the level check is the safest
at the moment.

If it's unclear, we can fix it. I really don't like to support something
based on the counters as the logic is arbitrary.

Depending on what happens with the next rev of the ACPI spec (or whenever)
some of this whole discussion might be bypassed simply by using whatever id
is marked valid on the node, as you suggest, but until then...

I will check if that can be done or enforced. I am sure if we add some
counter based logic, we will break something in future. So I would rather
stay with UID/offset though it's not mandatory in the specification.

--
Regards,
Sudeep