github/acrn-hypervisor
1
0
Fork 0
mirror of https://github.com/projectacrn/acrn-hypervisor.git synced 2025-06-19 12:12:16 +00:00
Code Issues Projects Releases Wiki Activity

config_tools: fix illegal access to the level_types dictionary

Browse Source 
In 10-processors.py, level_types is an unordered dictionary from level type
encodings to human-readable terms, not a list. It is thus illegal to access the
highest level using the `[-1]` notation.

This patch fixes this by calculating the highest level in level_types and use
that level as the index instead.

Tracked-On: #6689
Fixes: b5b22bf98 ("board_inspector: avoid hard-coded topo level names")
Signed-off-by: Junjie Mao <junjie.mao@intel.com>
This commit is contained in:
Junjie Mao 2022-02-08 16:43:22 +08:00 committed by acrnsi-robot
parent 01fa6de42c
commit 845c7dfa0a
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
misc/config_tools/board_inspector/extractors/10-processors.py
View File

@ -74,8 +74,9 @@ def extract_topology(processors_node):
while True:
leaf_topo = parse_cpuid(topo_leaf, subleaf, cpu_id)
if leaf_topo.level_type == 0:
if last_node.tag != level_types[-1]:
n, _ = get_or_create_parent(processors_node, level_types[-1], "0x0")
highest_level = max(level_types.keys())
if last_node.tag != level_types[highest_level]:
n, _ = get_or_create_parent(processors_node, level_types[highest_level], "0x0")
n.append(last_node)
last_node = n
processors_node.append(last_node)