diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2013-12-11 05:34:44 -0500 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2013-12-17 20:35:01 -0500 |
commit | 1d6ac185c32134233f77ce44800ceb4ab9361401 (patch) | |
tree | 189d3bac69aa35ed8e8f3f249c0d79a751351d5a /drivers/gpu/drm/radeon/radeon_state.c | |
parent | 57ed0f7b4375f4cb0ec3eccbc81f262294eefbcd (diff) |
drm: Kill DRM_COPY_(TO|FROM)_USER
Less yelling ftw!
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu/drm/radeon/radeon_state.c')
-rw-r--r-- | drivers/gpu/drm/radeon/radeon_state.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/drivers/gpu/drm/radeon/radeon_state.c b/drivers/gpu/drm/radeon/radeon_state.c index 4d20910899d4..956ab7f14e16 100644 --- a/drivers/gpu/drm/radeon/radeon_state.c +++ b/drivers/gpu/drm/radeon/radeon_state.c | |||
@@ -1810,7 +1810,7 @@ static int radeon_cp_dispatch_texture(struct drm_device * dev, | |||
1810 | } | 1810 | } |
1811 | if (!buf) { | 1811 | if (!buf) { |
1812 | DRM_DEBUG("EAGAIN\n"); | 1812 | DRM_DEBUG("EAGAIN\n"); |
1813 | if (DRM_COPY_TO_USER(tex->image, image, sizeof(*image))) | 1813 | if (copy_to_user(tex->image, image, sizeof(*image))) |
1814 | return -EFAULT; | 1814 | return -EFAULT; |
1815 | return -EAGAIN; | 1815 | return -EAGAIN; |
1816 | } | 1816 | } |
@@ -1823,7 +1823,7 @@ static int radeon_cp_dispatch_texture(struct drm_device * dev, | |||
1823 | 1823 | ||
1824 | #define RADEON_COPY_MT(_buf, _data, _width) \ | 1824 | #define RADEON_COPY_MT(_buf, _data, _width) \ |
1825 | do { \ | 1825 | do { \ |
1826 | if (DRM_COPY_FROM_USER(_buf, _data, (_width))) {\ | 1826 | if (copy_from_user(_buf, _data, (_width))) {\ |
1827 | DRM_ERROR("EFAULT on pad, %d bytes\n", (_width)); \ | 1827 | DRM_ERROR("EFAULT on pad, %d bytes\n", (_width)); \ |
1828 | return -EFAULT; \ | 1828 | return -EFAULT; \ |
1829 | } \ | 1829 | } \ |
@@ -2168,7 +2168,7 @@ static int radeon_cp_clear(struct drm_device *dev, void *data, struct drm_file * | |||
2168 | if (sarea_priv->nbox > RADEON_NR_SAREA_CLIPRECTS) | 2168 | if (sarea_priv->nbox > RADEON_NR_SAREA_CLIPRECTS) |
2169 | sarea_priv->nbox = RADEON_NR_SAREA_CLIPRECTS; | 2169 | sarea_priv->nbox = RADEON_NR_SAREA_CLIPRECTS; |
2170 | 2170 | ||
2171 | if (DRM_COPY_FROM_USER(&depth_boxes, clear->depth_boxes, | 2171 | if (copy_from_user(&depth_boxes, clear->depth_boxes, |
2172 | sarea_priv->nbox * sizeof(depth_boxes[0]))) | 2172 | sarea_priv->nbox * sizeof(depth_boxes[0]))) |
2173 | return -EFAULT; | 2173 | return -EFAULT; |
2174 | 2174 | ||
@@ -2436,7 +2436,7 @@ static int radeon_cp_texture(struct drm_device *dev, void *data, struct drm_file | |||
2436 | return -EINVAL; | 2436 | return -EINVAL; |
2437 | } | 2437 | } |
2438 | 2438 | ||
2439 | if (DRM_COPY_FROM_USER(&image, | 2439 | if (copy_from_user(&image, |
2440 | (drm_radeon_tex_image_t __user *) tex->image, | 2440 | (drm_radeon_tex_image_t __user *) tex->image, |
2441 | sizeof(image))) | 2441 | sizeof(image))) |
2442 | return -EFAULT; | 2442 | return -EFAULT; |
@@ -2460,7 +2460,7 @@ static int radeon_cp_stipple(struct drm_device *dev, void *data, struct drm_file | |||
2460 | 2460 | ||
2461 | LOCK_TEST_WITH_RETURN(dev, file_priv); | 2461 | LOCK_TEST_WITH_RETURN(dev, file_priv); |
2462 | 2462 | ||
2463 | if (DRM_COPY_FROM_USER(&mask, stipple->mask, 32 * sizeof(u32))) | 2463 | if (copy_from_user(&mask, stipple->mask, 32 * sizeof(u32))) |
2464 | return -EFAULT; | 2464 | return -EFAULT; |
2465 | 2465 | ||
2466 | RING_SPACE_TEST_WITH_RETURN(dev_priv); | 2466 | RING_SPACE_TEST_WITH_RETURN(dev_priv); |
@@ -2585,13 +2585,13 @@ static int radeon_cp_vertex2(struct drm_device *dev, void *data, struct drm_file | |||
2585 | drm_radeon_prim_t prim; | 2585 | drm_radeon_prim_t prim; |
2586 | drm_radeon_tcl_prim_t tclprim; | 2586 | drm_radeon_tcl_prim_t tclprim; |
2587 | 2587 | ||
2588 | if (DRM_COPY_FROM_USER(&prim, &vertex->prim[i], sizeof(prim))) | 2588 | if (copy_from_user(&prim, &vertex->prim[i], sizeof(prim))) |
2589 | return -EFAULT; | 2589 | return -EFAULT; |
2590 | 2590 | ||
2591 | if (prim.stateidx != laststate) { | 2591 | if (prim.stateidx != laststate) { |
2592 | drm_radeon_state_t state; | 2592 | drm_radeon_state_t state; |
2593 | 2593 | ||
2594 | if (DRM_COPY_FROM_USER(&state, | 2594 | if (copy_from_user(&state, |
2595 | &vertex->state[prim.stateidx], | 2595 | &vertex->state[prim.stateidx], |
2596 | sizeof(state))) | 2596 | sizeof(state))) |
2597 | return -EFAULT; | 2597 | return -EFAULT; |
@@ -2799,7 +2799,7 @@ static int radeon_emit_packet3_cliprect(struct drm_device *dev, | |||
2799 | 2799 | ||
2800 | do { | 2800 | do { |
2801 | if (i < cmdbuf->nbox) { | 2801 | if (i < cmdbuf->nbox) { |
2802 | if (DRM_COPY_FROM_USER(&box, &boxes[i], sizeof(box))) | 2802 | if (copy_from_user(&box, &boxes[i], sizeof(box))) |
2803 | return -EFAULT; | 2803 | return -EFAULT; |
2804 | /* FIXME The second and subsequent times round | 2804 | /* FIXME The second and subsequent times round |
2805 | * this loop, send a WAIT_UNTIL_3D_IDLE before | 2805 | * this loop, send a WAIT_UNTIL_3D_IDLE before |
@@ -3116,7 +3116,7 @@ static int radeon_cp_getparam(struct drm_device *dev, void *data, struct drm_fil | |||
3116 | return -EINVAL; | 3116 | return -EINVAL; |
3117 | } | 3117 | } |
3118 | 3118 | ||
3119 | if (DRM_COPY_TO_USER(param->value, &value, sizeof(int))) { | 3119 | if (copy_to_user(param->value, &value, sizeof(int))) { |
3120 | DRM_ERROR("copy_to_user\n"); | 3120 | DRM_ERROR("copy_to_user\n"); |
3121 | return -EFAULT; | 3121 | return -EFAULT; |
3122 | } | 3122 | } |