KVM: PPC: Book3s: Use arg->size directly in kvm_vm_ioctl_create_spapr_tce()
The size variable is just a copy of args->size, neither size nor args are modifed, so just use args->size directly. Signed-off-by: Deming Wang <wangdeming@inspur.com> [mpe: Reword change log] Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> Link: https://lore.kernel.org/r/20220703172932.11329-1-wangdeming@inspur.com
This commit is contained in:
parent
392a58f1ea
commit
6fa1efeaa6
@ -294,14 +294,14 @@ long kvm_vm_ioctl_create_spapr_tce(struct kvm *kvm,
|
|||||||
struct kvmppc_spapr_tce_table *stt = NULL;
|
struct kvmppc_spapr_tce_table *stt = NULL;
|
||||||
struct kvmppc_spapr_tce_table *siter;
|
struct kvmppc_spapr_tce_table *siter;
|
||||||
struct mm_struct *mm = kvm->mm;
|
struct mm_struct *mm = kvm->mm;
|
||||||
unsigned long npages, size = args->size;
|
unsigned long npages;
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
if (!args->size || args->page_shift < 12 || args->page_shift > 34 ||
|
if (!args->size || args->page_shift < 12 || args->page_shift > 34 ||
|
||||||
(args->offset + args->size > (ULLONG_MAX >> args->page_shift)))
|
(args->offset + args->size > (ULLONG_MAX >> args->page_shift)))
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
|
|
||||||
npages = kvmppc_tce_pages(size);
|
npages = kvmppc_tce_pages(args->size);
|
||||||
ret = account_locked_vm(mm, kvmppc_stt_pages(npages), true);
|
ret = account_locked_vm(mm, kvmppc_stt_pages(npages), true);
|
||||||
if (ret)
|
if (ret)
|
||||||
return ret;
|
return ret;
|
||||||
@ -314,7 +314,7 @@ long kvm_vm_ioctl_create_spapr_tce(struct kvm *kvm,
|
|||||||
stt->liobn = args->liobn;
|
stt->liobn = args->liobn;
|
||||||
stt->page_shift = args->page_shift;
|
stt->page_shift = args->page_shift;
|
||||||
stt->offset = args->offset;
|
stt->offset = args->offset;
|
||||||
stt->size = size;
|
stt->size = args->size;
|
||||||
stt->kvm = kvm;
|
stt->kvm = kvm;
|
||||||
mutex_init(&stt->alloc_lock);
|
mutex_init(&stt->alloc_lock);
|
||||||
INIT_LIST_HEAD_RCU(&stt->iommu_tables);
|
INIT_LIST_HEAD_RCU(&stt->iommu_tables);
|
||||||
|
Loading…
x
Reference in New Issue
Block a user