[patch] i915: return -EFAULT if copy_to_user fails

2010-08-23 Thread Dan Carpenter
On Wed, Jun 23, 2010 at 07:03:01PM +0200, Dan Carpenter wrote: > copy_to_user() returns the number of bytes remaining to be copied and > I'm pretty sure we want to return a negative error code here. > This patch wasn't applied either. regards, dan carpenter > Signed-off-by: Dan Carpenter > >

[patch] i915: return -EFAULT if copy_to_user fails

2010-08-23 Thread Dan Carpenter
On Sat, Jun 19, 2010 at 03:12:51PM +0200, Dan Carpenter wrote: > copy_to_user returns the number of bytes remaining to be copied, but we > want to return a negative error code here. These are returned to > userspace. > I didn't get a response on this patch. regards, dan carpenter > Signed-off-

[patch] i915: return -EFAULT if copy_to_user fails

2010-08-23 Thread Chris Wilson
On Sat, 19 Jun 2010 15:12:51 +0200, Dan Carpenter wrote: > copy_to_user returns the number of bytes remaining to be copied, but we > want to return a negative error code here. These are returned to > userspace. > > Signed-off-by: Dan Carpenter Reviewed-by: Chris Wilson -- Chris Wilson, Intel

[patch] i915: return -EFAULT if copy_to_user fails

2010-08-23 Thread Chris Wilson
On Wed, 23 Jun 2010 19:03:01 +0200, Dan Carpenter wrote: > copy_to_user() returns the number of bytes remaining to be copied and > I'm pretty sure we want to return a negative error code here. > > Signed-off-by: Dan Carpenter Reviewed-by: Chris Wilson -- Chris Wilson, Intel Open Source Techno

Re: [patch] i915: return -EFAULT if copy_to_user fails

2010-08-23 Thread Chris Wilson
On Sat, 19 Jun 2010 15:12:51 +0200, Dan Carpenter wrote: > copy_to_user returns the number of bytes remaining to be copied, but we > want to return a negative error code here. These are returned to > userspace. > > Signed-off-by: Dan Carpenter Reviewed-by: Chris Wilson -- Chris Wilson, Intel

Re: [patch] i915: return -EFAULT if copy_to_user fails

2010-08-23 Thread Chris Wilson
On Wed, 23 Jun 2010 19:03:01 +0200, Dan Carpenter wrote: > copy_to_user() returns the number of bytes remaining to be copied and > I'm pretty sure we want to return a negative error code here. > > Signed-off-by: Dan Carpenter Reviewed-by: Chris Wilson -- Chris Wilson, Intel Open Source Techno

Re: [patch] i915: return -EFAULT if copy_to_user fails

2010-08-23 Thread Dan Carpenter
On Wed, Jun 23, 2010 at 07:03:01PM +0200, Dan Carpenter wrote: > copy_to_user() returns the number of bytes remaining to be copied and > I'm pretty sure we want to return a negative error code here. > This patch wasn't applied either. regards, dan carpenter > Signed-off-by: Dan Carpenter > >

Re: [patch] i915: return -EFAULT if copy_to_user fails

2010-08-23 Thread Dan Carpenter
On Sat, Jun 19, 2010 at 03:12:51PM +0200, Dan Carpenter wrote: > copy_to_user returns the number of bytes remaining to be copied, but we > want to return a negative error code here. These are returned to > userspace. > I didn't get a response on this patch. regards, dan carpenter > Signed-off-

[patch] i915: return -EFAULT if copy_to_user fails

2010-06-24 Thread Dan Carpenter
copy_to_user() returns the number of bytes remaining to be copied and I'm pretty sure we want to return a negative error code here. Signed-off-by: Dan Carpenter diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index 9ded3da..22691b4 100644 --- a/drivers/gpu/drm/i915

[patch] i915: return -EFAULT if copy_to_user fails

2010-06-23 Thread Dan Carpenter
copy_to_user() returns the number of bytes remaining to be copied and I'm pretty sure we want to return a negative error code here. Signed-off-by: Dan Carpenter diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index 9ded3da..22691b4 100644 --- a/drivers/gpu/drm/i915

[patch] i915: return -EFAULT if copy_to_user fails

2010-06-20 Thread Dan Carpenter
copy_to_user returns the number of bytes remaining to be copied, but we want to return a negative error code here. These are returned to userspace. Signed-off-by: Dan Carpenter diff --git a/drivers/gpu/drm/i915/i915_dma.c b/drivers/gpu/drm/i915/i915_dma.c index 59a2bf8..5fd876e 100644 --- a/dri

[patch] i915: return -EFAULT if copy_to_user fails

2010-06-19 Thread Dan Carpenter
copy_to_user returns the number of bytes remaining to be copied, but we want to return a negative error code here. These are returned to userspace. Signed-off-by: Dan Carpenter diff --git a/drivers/gpu/drm/i915/i915_dma.c b/drivers/gpu/drm/i915/i915_dma.c index 59a2bf8..5fd876e 100644 --- a/dri