Re: [PATCH] kvm: Add compat_ioctl for device control API

2013-05-05 Thread Gleb Natapov
On Tue, Apr 30, 2013 at 08:00:45PM -0500, Scott Wood wrote:
> This API shouldn't have 32/64-bit issues, but VFS assumes it does
> unless told otherwise.
> 
> Signed-off-by: Scott Wood 
Applied, thanks.

> ---
>  virt/kvm/kvm_main.c |3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c
> index 54a14fa..ee0adad 100644
> --- a/virt/kvm/kvm_main.c
> +++ b/virt/kvm/kvm_main.c
> @@ -2221,6 +2221,9 @@ static int kvm_device_release(struct inode *inode, 
> struct file *filp)
>  
>  static const struct file_operations kvm_device_fops = {
>   .unlocked_ioctl = kvm_device_ioctl,
> +#ifdef CONFIG_COMPAT
> + .compat_ioctl = kvm_device_ioctl,
> +#endif
>   .release = kvm_device_release,
>  };
>  
> -- 
> 1.7.10.4

--
Gleb.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] kvm: Add compat_ioctl for device control API

2013-04-30 Thread Scott Wood
This API shouldn't have 32/64-bit issues, but VFS assumes it does
unless told otherwise.

Signed-off-by: Scott Wood 
---
 virt/kvm/kvm_main.c |3 +++
 1 file changed, 3 insertions(+)

diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c
index 54a14fa..ee0adad 100644
--- a/virt/kvm/kvm_main.c
+++ b/virt/kvm/kvm_main.c
@@ -2221,6 +2221,9 @@ static int kvm_device_release(struct inode *inode, struct 
file *filp)
 
 static const struct file_operations kvm_device_fops = {
.unlocked_ioctl = kvm_device_ioctl,
+#ifdef CONFIG_COMPAT
+   .compat_ioctl = kvm_device_ioctl,
+#endif
.release = kvm_device_release,
 };
 
-- 
1.7.10.4

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html