From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-yw0-f196.google.com (mail-yw0-f196.google.com [209.85.161.196]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3qR1Sk1wC1zDqHw for ; Fri, 18 Mar 2016 08:19:18 +1100 (AEDT) Received: by mail-yw0-f196.google.com with SMTP id g6so7942522ywb.0 for ; Thu, 17 Mar 2016 14:19:18 -0700 (PDT) Date: Thu, 17 Mar 2016 18:19:10 -0300 From: Gustavo Padovan To: Joe Perches Cc: Rob Clark , Gustavo Padovan , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Greg Kroah-Hartman , Linux Kernel Mailing List , devel@driverdev.osuosl.org, "dri-devel@lists.freedesktop.org" , Daniel Stone , Arve =?iso-8859-1?B?SGr4bm5lduVn?= , Riley Andrews , Daniel Vetter , Greg Hackmann , John Harrison , Maarten Lankhorst , Andrew Morton , David Airlie , Daniel Vetter , linuxppc-dev Subject: Re: [PATCH v9 2/3] kernel.h: add to_user_ptr() Message-ID: <20160317211910.GB2611@joana> References: <1458235817-28375-1-git-send-email-gustavo@padovan.org> <1458235817-28375-2-git-send-email-gustavo@padovan.org> <1458236486.9556.11.camel@perches.com> <20160317180523.GA2619@joana> <20160317184318.GA2611@joana> <1458246150.9556.17.camel@perches.com> <1458247207.9556.28.camel@perches.com> <1458249003.9556.30.camel@perches.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 In-Reply-To: <1458249003.9556.30.camel@perches.com> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , 2016-03-17 Joe Perches : > On Thu, 2016-03-17 at 16:50 -0400, Rob Clark wrote: > > On Thu, Mar 17, 2016 at 4:40 PM, Joe Perches wrote: > [] > > > It's a name that seems like it should be a straightforward > > > cast of a kernel pointer to a __user pointer like: > > >=20 > > > static inline void __user *to_user_ptr(void *p) > > > { > > > =A0=A0=A0=A0=A0=A0=A0=A0return (void __user *)p; > > > } > > ahh, ok.=A0=A0I guess I was used to using it in the context of ioctl > > structs..=A0=A0in that context u64 -> (void __user *) made more sense. > >=20 > > Maybe uapi_to_ptr()?=A0=A0(ok, not super-creative.. maybe someone has a > > better idea) >=20 > Maybe u64_to_user_ptr? That is a good name. If everyone agrees I can resend this patch changing it to u64_to_user_ptr. Then should we still keep it on kernel.h? Gustavo