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_kms.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_kms.c')
-rw-r--r-- | drivers/gpu/drm/radeon/radeon_kms.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/gpu/drm/radeon/radeon_kms.c b/drivers/gpu/drm/radeon/radeon_kms.c index 55d0b474bd37..daa28b6a6832 100644 --- a/drivers/gpu/drm/radeon/radeon_kms.c +++ b/drivers/gpu/drm/radeon/radeon_kms.c | |||
@@ -223,7 +223,7 @@ int radeon_info_ioctl(struct drm_device *dev, void *data, struct drm_file *filp) | |||
223 | *value = rdev->accel_working; | 223 | *value = rdev->accel_working; |
224 | break; | 224 | break; |
225 | case RADEON_INFO_CRTC_FROM_ID: | 225 | case RADEON_INFO_CRTC_FROM_ID: |
226 | if (DRM_COPY_FROM_USER(value, value_ptr, sizeof(uint32_t))) { | 226 | if (copy_from_user(value, value_ptr, sizeof(uint32_t))) { |
227 | DRM_ERROR("copy_from_user %s:%u\n", __func__, __LINE__); | 227 | DRM_ERROR("copy_from_user %s:%u\n", __func__, __LINE__); |
228 | return -EFAULT; | 228 | return -EFAULT; |
229 | } | 229 | } |
@@ -269,7 +269,7 @@ int radeon_info_ioctl(struct drm_device *dev, void *data, struct drm_file *filp) | |||
269 | * | 269 | * |
270 | * When returning, the value is 1 if filp owns hyper-z access, | 270 | * When returning, the value is 1 if filp owns hyper-z access, |
271 | * 0 otherwise. */ | 271 | * 0 otherwise. */ |
272 | if (DRM_COPY_FROM_USER(value, value_ptr, sizeof(uint32_t))) { | 272 | if (copy_from_user(value, value_ptr, sizeof(uint32_t))) { |
273 | DRM_ERROR("copy_from_user %s:%u\n", __func__, __LINE__); | 273 | DRM_ERROR("copy_from_user %s:%u\n", __func__, __LINE__); |
274 | return -EFAULT; | 274 | return -EFAULT; |
275 | } | 275 | } |
@@ -281,7 +281,7 @@ int radeon_info_ioctl(struct drm_device *dev, void *data, struct drm_file *filp) | |||
281 | break; | 281 | break; |
282 | case RADEON_INFO_WANT_CMASK: | 282 | case RADEON_INFO_WANT_CMASK: |
283 | /* The same logic as Hyper-Z. */ | 283 | /* The same logic as Hyper-Z. */ |
284 | if (DRM_COPY_FROM_USER(value, value_ptr, sizeof(uint32_t))) { | 284 | if (copy_from_user(value, value_ptr, sizeof(uint32_t))) { |
285 | DRM_ERROR("copy_from_user %s:%u\n", __func__, __LINE__); | 285 | DRM_ERROR("copy_from_user %s:%u\n", __func__, __LINE__); |
286 | return -EFAULT; | 286 | return -EFAULT; |
287 | } | 287 | } |
@@ -417,7 +417,7 @@ int radeon_info_ioctl(struct drm_device *dev, void *data, struct drm_file *filp) | |||
417 | *value = rdev->fastfb_working; | 417 | *value = rdev->fastfb_working; |
418 | break; | 418 | break; |
419 | case RADEON_INFO_RING_WORKING: | 419 | case RADEON_INFO_RING_WORKING: |
420 | if (DRM_COPY_FROM_USER(value, value_ptr, sizeof(uint32_t))) { | 420 | if (copy_from_user(value, value_ptr, sizeof(uint32_t))) { |
421 | DRM_ERROR("copy_from_user %s:%u\n", __func__, __LINE__); | 421 | DRM_ERROR("copy_from_user %s:%u\n", __func__, __LINE__); |
422 | return -EFAULT; | 422 | return -EFAULT; |
423 | } | 423 | } |
@@ -465,7 +465,7 @@ int radeon_info_ioctl(struct drm_device *dev, void *data, struct drm_file *filp) | |||
465 | DRM_DEBUG_KMS("Invalid request %d\n", info->request); | 465 | DRM_DEBUG_KMS("Invalid request %d\n", info->request); |
466 | return -EINVAL; | 466 | return -EINVAL; |
467 | } | 467 | } |
468 | if (DRM_COPY_TO_USER(value_ptr, (char*)value, value_size)) { | 468 | if (copy_to_user(value_ptr, (char*)value, value_size)) { |
469 | DRM_ERROR("copy_to_user %s:%u\n", __func__, __LINE__); | 469 | DRM_ERROR("copy_to_user %s:%u\n", __func__, __LINE__); |
470 | return -EFAULT; | 470 | return -EFAULT; |
471 | } | 471 | } |