Re: [PATCH v4 4/4] KVM: s390: Minor refactor of base/ext facility lists

Janosch Frank
2023-12-20T10:48:38Z

On 12/19/23 15:08, Nina Schoetterl-Glausch wrote:
> Directly use the size of the arrays instead of going through the
> indirection of kvm_s390_fac_size().
> Don't use magic number for the number of entries in the non hypervisor
> managed facility bit mask list.
> Make the constraint of that number on kvm_s390_fac_base obvious.
> Get rid of implicit double anding of stfle_fac_list.
>
> Reviewed-by: Claudio Imbrenda <imbrenda@linux.ibm.com>
> Signed-off-by: Nina Schoetterl-Glausch <nsg@linux.ibm.com>

@Nina: I'm currently still recovering from a cold and hence I'm not
fully able to grasp this patch.

May I drop it and we re-visit it next year for 6.9?