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 | |
| 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')
| -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; |
