Re: [PATCH 03/98] drm.h: use __kernel_size_t instead of size_t

From: Emil Velikov
Date: Wed Jun 03 2015 - 13:21:07 EST


On 30 May 2015 at 16:37, Mikko Rapeli <mikko.rapeli@xxxxxx> wrote:
> Fixes userspace compilation error:
>
> drm/drm.h:132:2: error: unknown type name âsize_tâ
>
> Signed-off-by: Mikko Rapeli <mikko.rapeli@xxxxxx>
> ---
> include/uapi/drm/drm.h | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/include/uapi/drm/drm.h b/include/uapi/drm/drm.h
> index ff6ef62..b197650 100644
> --- a/include/uapi/drm/drm.h
> +++ b/include/uapi/drm/drm.h
> @@ -129,11 +129,11 @@ struct drm_version {
> int version_major; /**< Major version */
> int version_minor; /**< Minor version */
> int version_patchlevel; /**< Patch level */
> - size_t name_len; /**< Length of name buffer */
> + __kernel_size_t name_len; /**< Length of name buffer */
> char __user *name; /**< Name of driver */
> - size_t date_len; /**< Length of date buffer */
> + __kernel_size_t date_len; /**< Length of date buffer */
> char __user *date; /**< User-space buffer to hold date */
> - size_t desc_len; /**< Length of desc buffer */
> + __kernel_size_t desc_len; /**< Length of desc buffer */
> char __user *desc; /**< User-space buffer to hold desc */
> };
>
> @@ -143,7 +143,7 @@ struct drm_version {
> * \sa drmGetBusid() and drmSetBusId().
> */
> struct drm_unique {
> - size_t unique_len; /**< Length of unique */
> + __kernel_size_t unique_len; /**< Length of unique */
As the file is used by other platforms than Linux this change will
break them. Can you add a typedef similar to how __u8 and friends are
handled at the top of the file.

Thanks
Emil
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/