diff options
author | Christian Borntraeger <borntraeger@de.ibm.com> | 2008-04-21 07:48:24 -0400 |
---|---|---|
committer | Avi Kivity <avi@qumranet.com> | 2008-07-20 05:40:46 -0400 |
commit | 1e1c65e03ec817a64153751150f6691db9842acd (patch) | |
tree | c7f15ad0f376d7e8a501a2aa7be378cb84ca8e86 /virt/kvm/kvm_main.c | |
parent | 5b664cb235e97afbf34db9c4d77f08ebd725335e (diff) |
KVM: remove long -> void *user -> long cast
kvm_dev_ioctl casts the arg value to void __user *, just to recast it
again to long. This seems unnecessary.
According to objdump the binary code on x86 is unchanged by this patch.
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
Signed-off-by: Avi Kivity <avi@qumranet.com>
Diffstat (limited to 'virt/kvm/kvm_main.c')
-rw-r--r-- | virt/kvm/kvm_main.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c index d4eae6af0738..b6a59498b5a7 100644 --- a/virt/kvm/kvm_main.c +++ b/virt/kvm/kvm_main.c | |||
@@ -1179,7 +1179,6 @@ static int kvm_dev_ioctl_create_vm(void) | |||
1179 | static long kvm_dev_ioctl(struct file *filp, | 1179 | static long kvm_dev_ioctl(struct file *filp, |
1180 | unsigned int ioctl, unsigned long arg) | 1180 | unsigned int ioctl, unsigned long arg) |
1181 | { | 1181 | { |
1182 | void __user *argp = (void __user *)arg; | ||
1183 | long r = -EINVAL; | 1182 | long r = -EINVAL; |
1184 | 1183 | ||
1185 | switch (ioctl) { | 1184 | switch (ioctl) { |
@@ -1196,7 +1195,7 @@ static long kvm_dev_ioctl(struct file *filp, | |||
1196 | r = kvm_dev_ioctl_create_vm(); | 1195 | r = kvm_dev_ioctl_create_vm(); |
1197 | break; | 1196 | break; |
1198 | case KVM_CHECK_EXTENSION: | 1197 | case KVM_CHECK_EXTENSION: |
1199 | r = kvm_dev_ioctl_check_extension((long)argp); | 1198 | r = kvm_dev_ioctl_check_extension(arg); |
1200 | break; | 1199 | break; |
1201 | case KVM_GET_VCPU_MMAP_SIZE: | 1200 | case KVM_GET_VCPU_MMAP_SIZE: |
1202 | r = -EINVAL; | 1201 | r = -EINVAL; |