From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754024AbcHTSDV (ORCPT ); Sat, 20 Aug 2016 14:03:21 -0400 Received: from mail.kapsi.fi ([217.30.184.167]:36295 "EHLO mail.kapsi.fi" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753976AbcHTSDT (ORCPT ); Sat, 20 Aug 2016 14:03:19 -0400 Date: Sat, 20 Aug 2016 21:03:18 +0300 From: Mikko Rapeli To: Ken Phillis Jr Cc: Rob Clark , Emil Velikov , ML dri-devel , linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] Revert "include/uapi/drm/amdgpu_drm.h: use __u32 and __u64 from " Message-ID: <20160820180318.GF5399@lakka.kapsi.fi> References: <1471614628-11585-1-git-send-email-maraeo@gmail.com> <759ce38b-eb22-ee27-243b-f99891519cfb@vodafone.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-SA-Exim-Connect-IP: 2001:1bc8:1004::1 X-SA-Exim-Mail-From: mikko.rapeli@iki.fi X-SA-Exim-Scanned: No (on mail.kapsi.fi); SAEximRunCond expanded to false Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Cc'ing lkml. On Fri, Aug 19, 2016 at 09:18:24PM -0500, Ken Phillis Jr wrote: > On Fri, Aug 19, 2016 at 8:46 PM, Rob Clark wrote: > > > perhaps, but if the target audience for driver specific APIs is > > libdrm/mesa, which already uses stdint types, then I fail to see the > > point.. > > > > It is a bit difference scenario for some of the core kms ioctls which > > are not driver specific.. but most of the driver specific gpu related > > ioctls tend to be rather complex and not intended for use by something > > other than libdrm/mesa. > > > > BR, > > -R > > > > I personally do not see the reason to break user space API in the first > place. > I know for a fact that the include file ( linux/types.h ) mentioned by the > patch > supports C99 integer data types since Linux Kernel Version 2.2.0. This makes > the patch that got reverted completely meaningless since the only reason to > make this change in the first place would be if the developer intends on > backporting the driver to kernels older than version 2.2.0. In userspace linux/types.h does not provide these integer data types to avoid conflict with libc ones. Thus several uapi headers fail to compile in userspace. Fix is to either include libc side stdint.h in userspace but that was rejected by kernel developers including Linus who are in favor of kernel specific double underscore types. -Mikko