Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> writes: > Le 14/04/2022 à 15:19, Vitaly Kuznetsov a écrit : ... >> @@ -224,7 +225,7 @@ static inline int __cpumask_to_vpset(struct hv_vpset *vpset, >> * structs are not cleared between calls, we risk flushing unneeded >> * vCPUs otherwise. >> */ >> - for (vcpu_bank = 0; vcpu_bank <= hv_max_vp_index / 64; vcpu_bank++) >> + for (vcpu_bank = 0; vcpu_bank <= max_vcpu_bank; vcpu_bank++) >> vpset->bank_contents[vcpu_bank] = 0; > > and here: > bitmap_clear(vpset->bank_contents, 0, hv_max_vp_index); > or maybe even if it is safe to do so: > bitmap_zero(vpset->bank_contents, hv_max_vp_index); Both your suggestions (including the one for "PATCH v3 07/34]") look good to me, thanks! I'd however want to send them to linux-hyperv@ separately when this series lands through KVM tree just to not make this heavy series even heavier. -- Vitaly
