Message ID | 20180830114959.63363-1-andriy.shevchenko@linux.intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [v1] KVM: SVM: Switch to bitmap_zalloc() | expand |
On 30/08/2018 13:49, Andy Shevchenko wrote: > Switch to bitmap_zalloc() to show clearly what we are allocating. > Besides that it returns pointer of bitmap type instead of opaque void *. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > --- > arch/x86/kvm/svm.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c > index 6276140044d0..5d7e9f76caba 100644 > --- a/arch/x86/kvm/svm.c > +++ b/arch/x86/kvm/svm.c > @@ -1226,8 +1226,7 @@ static __init int sev_hardware_setup(void) > min_sev_asid = cpuid_edx(0x8000001F); > > /* Initialize SEV ASID bitmap */ > - sev_asid_bitmap = kcalloc(BITS_TO_LONGS(max_sev_asid), > - sizeof(unsigned long), GFP_KERNEL); > + sev_asid_bitmap = bitmap_zalloc(max_sev_asid, GFP_KERNEL); > if (!sev_asid_bitmap) > return 1; > > @@ -1405,7 +1404,7 @@ static __exit void svm_hardware_unsetup(void) > int cpu; > > if (svm_sev_enabled()) > - kfree(sev_asid_bitmap); > + bitmap_free(sev_asid_bitmap); > > for_each_possible_cpu(cpu) > svm_cpu_uninit(cpu); > Queued, thanks. Paolo
diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c index 6276140044d0..5d7e9f76caba 100644 --- a/arch/x86/kvm/svm.c +++ b/arch/x86/kvm/svm.c @@ -1226,8 +1226,7 @@ static __init int sev_hardware_setup(void) min_sev_asid = cpuid_edx(0x8000001F); /* Initialize SEV ASID bitmap */ - sev_asid_bitmap = kcalloc(BITS_TO_LONGS(max_sev_asid), - sizeof(unsigned long), GFP_KERNEL); + sev_asid_bitmap = bitmap_zalloc(max_sev_asid, GFP_KERNEL); if (!sev_asid_bitmap) return 1; @@ -1405,7 +1404,7 @@ static __exit void svm_hardware_unsetup(void) int cpu; if (svm_sev_enabled()) - kfree(sev_asid_bitmap); + bitmap_free(sev_asid_bitmap); for_each_possible_cpu(cpu) svm_cpu_uninit(cpu);
Switch to bitmap_zalloc() to show clearly what we are allocating. Besides that it returns pointer of bitmap type instead of opaque void *. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- arch/x86/kvm/svm.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)