open-power / skiboot

OPAL boot and runtime firmware for POWER
Apache License 2.0
100 stars 134 forks source link

xive/p9: various small cleanups #254

Closed legoater closed 3 years ago

legoater commented 4 years ago

Hello,

Here are cleanups which try to deobfuscate some magic values used in the driver and add documentation in the commit logs. The only significant change is on the size of the VP space which now depends on the number of EQ descriptors. We have 1M of these and we use 8 priorities per VP. Therefore, our VP space should be limited to 128k.

Tested on a boston with 256 KVM guests each with 4 vCPUS. For the record, it takes 25s for all to reach login.

Thanks,

C.

hegdevasant commented 3 years ago

This was experimental PR. Closing..

-Vasant