[PATCH v2 1/4] powerpc/pseries/vas: Define global hv_cop_caps struct

Nathan Lynch nathanl at linux.ibm.com
Wed Feb 9 02:48:34 AEDT 2022


Haren Myneni <haren at linux.ibm.com> writes:
> The coprocessor capabilities struct is used to get default and
> QoS capabilities from the hypervisor during init, DLPAR event and
> migration. So instead of allocating this struct for each event,
> define global struct and reuse it, especially eliminate memory
> allocation failure during migration.

Which allows the migration code to avoid adding an error path. I could
go either way, but this approach seems fine to me assuming all users of
the global object are guarded by an appropriate lock.

Acked-by: Nathan Lynch <nathanl at linux.ibm.com>



More information about the Linuxppc-dev mailing list