From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH v2 02/17] compat_ioctl: move drivers to generic_compat_ioctl_ptrarg Date: Wed, 12 Sep 2018 18:20:34 +0200 Message-ID: References: <20180912150142.157913-1-arnd@arndb.de> <20180912150142.157913-2-arnd@arndb.de> <20180912153300.GE5633@ziepe.ca> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20180912153300.GE5633@ziepe.ca> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Jason Gunthorpe Cc: kvm@vger.kernel.org, Alexander Shishkin , Jarkko Sakkinen , virtualization@lists.linux-foundation.org, Benjamin Tissoires , linux-mtd , Peter Huewe , linux1394-devel@lists.sourceforge.net, driverdevel , Marek Vasut , "open list:HID CORE LAYER" , "Winkler, Tomas" , Jiri Kosina , Al Viro , Artem Bityutskiy , gregkh , USB list , Linux Kernel Mailing List , Sudip Mukherjee , Stefan Richter , Networking List-Id: virtualization@lists.linuxfoundation.org On Wed, Sep 12, 2018 at 5:33 PM Jason Gunthorpe wrote: > > On Wed, Sep 12, 2018 at 05:01:03PM +0200, Arnd Bergmann wrote: > > Each of these drivers has a copy of the same trivial helper function to > > convert the pointer argument and then call the native ioctl handler. > > > > We now have a generic implementation of that, so use it. > > > > For vtpm: > > Reviewed-by: Jason Gunthorpe > > Arnd, would you consider including a patch as part of/after this > series to make compat_ioctl in drivers/infiniband/core/uverbs_main.c > use this as well? Looks like a bug too? That should be included in patch 5 in this series. I may have skipped some Cc there since it had too many recipients (sent only to the mailing lists instead). Arnd