From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965539AbbFCRVC (ORCPT ); Wed, 3 Jun 2015 13:21:02 -0400 Received: from mail-vn0-f49.google.com ([209.85.216.49]:38048 "EHLO mail-vn0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755004AbbFCRTu convert rfc822-to-8bit (ORCPT ); Wed, 3 Jun 2015 13:19:50 -0400 MIME-Version: 1.0 In-Reply-To: <1433000370-19509-4-git-send-email-mikko.rapeli@iki.fi> References: <1433000370-19509-1-git-send-email-mikko.rapeli@iki.fi> <1433000370-19509-4-git-send-email-mikko.rapeli@iki.fi> Date: Wed, 3 Jun 2015 18:19:48 +0100 Message-ID: Subject: Re: [PATCH 03/98] drm.h: use __kernel_size_t instead of size_t From: Emil Velikov To: Mikko Rapeli Cc: "Linux-Kernel@Vger. Kernel. Org" , ML dri-devel , linux-api@vger.kernel.org Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 30 May 2015 at 16:37, Mikko Rapeli wrote: > Fixes userspace compilation error: > > drm/drm.h:132:2: error: unknown type name ‘size_t’ > > Signed-off-by: Mikko Rapeli > --- > 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