tree 19cf740796bebe8648c8fc6be3205759bf6f0376
parent 6f55203b70316baec228073abcd581b64985af93
author Andrei Kartashev <a.kartashev@yadro.com> 1625245470 +0300
committer Andrei Kartashev <a.kartashev@yadro.com> 1625644531 +0300
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQGzBAABCAAdFiEE/uIssz1fdWlJiEZrELry6oSHVrUFAmDlXfcACgkQELry6oSH
 VrVArwwAq7EEzcYuWk2N0COblASNiWqJXfIOEIDMgdTirvigGcz3MxqLs388EGpI
 fuyVNB44+wJ/NIC9ekrwnJx6bJqEuYbledSZYpe6yY+ctJZ3MT4dA9z00iPEd+vW
 yqz9aON/lC5Gej7tN9PNivxN/ux4sNKPJhOnlZxtI1qvu/1yPYkhGjTp8p6gGhD/
 vpxYJtzSiSpRjeRhtoXo79EOfA3+D+nKP4AfBuYk9DqqME5IVdXV13dU4FoGNDdj
 LAYDmDZaoyrlM4OgWMfGJcDV5QetP+mnogFJ18U3KLCfzlPhwYcKPgVNCbZ/H7IP
 MZ/UXeOzIpXQOYh/fz4ZI8gnKzAh9vbqtOiC4GrcU72yjlT5DN8OrG7G8NhUSMzu
 6QT89HZzaOL7gFItuOwZdwNWM8OV9MTbqtN0MivlqdNOK4dN3TpAt9x/RPIrVIOX
 rFjp6DmXN1uKkHRKEQvdfTy0OGmdDfqKLBSnzI4fRJEZe4M/Jg/FCPGzcyyZZdQi
 T65vzBSK
 =964F
 -----END PGP SIGNATURE-----

Refactor handling CPUInfo array

It was found that sometimes peci-pcie crashes with out-of-range read of
cpuInfo vector. This happens on start scan when for some reasons system
failed to initialize vector.

While technically enough to add guard near the line
	scanPCIeDevice(io, objServer, cpuInfo, 0, 0, 0);
it is better to refactor whole related code.

This commit introduces following code optimizations:
* Merge getClientAddrMap and getCPUBusNums to one function getCPUBusMap
  since they both always called together and do same work: fill cpuInfo
  vector. Resulting getCPUBusMap function now return error if no CPUs
  discovered.
* probePCIeDevice doesn't really need cpuInfo vector, it's fine with
  just single CPU address.
* scanPCIeDevice now the only function which read from cpuInfo and it
  verify index before and return if it is not valid.

Tested: ensure no functional changes, no crash if cpuInfo vector empty

Signed-off-by: Andrei Kartashev <a.kartashev@yadro.com>
Change-Id: Ie6bab4e844670ceecd515728f43f7d22efa1aed6