On Thu, May 29, 2025 at 04:39:48PM -0700, Sean Christopherson wrote: >Drop the unnecessary and dangerous value-terminated behavior of >direct_access_msrs, and simply iterate over the actual size of the array. >The use in svm_set_x2apic_msr_interception() is especially sketchy, as it >relies on unused capacity being zero-initialized, and '0' being outside >the range of x2APIC MSRs. > >To ensure the array and shadow_msr_intercept stay synchronized, simply >assert that their sizes are identical (note the six 64-bit-only MSRs). > >Note, direct_access_msrs will soon be removed entirely; keeping the assert >synchronized with the array isn't expected to be along-term maintenance >burden. > >Signed-off-by: Sean Christopherson <seanjc@xxxxxxxxxx> Reviewed-by: Chao Gao <chao.gao@xxxxxxxxx>
