Fixed EC vs VP setting for shared mode LPAR configuration.

In the previously added code, it was observed that when the condition max_proc_units > max_virtual_proc is true, the system assigns the maximum virtual processors as returned by the LPAR configuration. However, an issue arises when the LPAR returns a maximum virtual processor count that is less than the max_proc_units. In such cases, the HMC fails to configure the EC vs VP values according to the virtual processors' thumb rule, which mandates that virtual processors should always be greater than or equal to EC processors.

This patch addresses and resolves the issue.

Signed-off-by: Samir Mulani <[email protected]>

Author: SamirMulani

common/OpTestHMC.py

@@ -423,10 +423,10 @@ def change_proc_mode(self, proc_mode, sharing_mode, min_proc_units, desired_proc
             v_max_proc = 0
             max_virtual_proc = self.run_command("lshwres -m %s -r proc --level sys -F curr_sys_virtual_procs" % (self.mg_system))
             max_virtual_proc = int(max_virtual_proc[0])
-            if overcommit_ratio*int(max_proc_units) > max_virtual_proc:
-                v_max_proc = max_virtual_proc
+            if int(max_proc_units) > max_virtual_proc:
+                v_max_proc = int(max_proc_units)
             else:
-                v_max_proc = overcommit_ratio*int(max_proc_units)
+                v_max_proc = max_virtual_proc
 
             self.set_lpar_cfg("proc_mode=shared,sharing_mode=%s,min_proc_units=%s,max_proc_units=%s,"
                               "desired_proc_units=%s,min_procs=%s,desired_procs=%s,max_procs=%s,"