drm/amdgpu: preserve the num_links in case of reflection

For topology reflection, each socket to every other socket has the
exactly same topology info as the other way around. So it is safe
to keep the reflected num_links value otherwise it will be overriden
by the link info output of GET_PEER_LINKS command.

Signed-off-by: Shiwu Zhang <shiwu.zhang@amd.com>
Reviewed-by: Le Ma <le.ma@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
index 4395c53d09d83e60aa2b45f226d641dbeebef5c2..ea47012795e7f9bdb1a8e0b232064a81c1f91f0e 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
@@ -1402,7 +1402,8 @@ int psp_xgmi_get_topology_info(struct psp_context *psp,
                        topology->nodes[i].num_links = get_extended_data ?
                                        topology->nodes[i].num_links +
                                                        link_info_output->nodes[i].num_links :
-                                       link_info_output->nodes[i].num_links;
+                                       ((requires_reflection && topology->nodes[i].num_links) ? topology->nodes[i].num_links :
+                                        link_info_output->nodes[i].num_links);
 
                        /* reflect the topology information for bi-directionality */
                        if (requires_reflection && topology->nodes[i].num_hops)