<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Aug 19, 2016 at 8:46 PM, Rob Clark <span dir="ltr"><<a href="mailto:robdclark@gmail.com" target="_blank">robdclark@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class=""><div class="h5">
</div></div>perhaps, but if the target audience for driver specific APIs is<br>
libdrm/mesa, which already uses stdint types, then I fail to see the<br>
point..<br>
<br>
It is a bit difference scenario for some of the core kms ioctls which<br>
are not driver specific..  but most of the driver specific gpu related<br>
ioctls tend to be rather complex and not intended for use by something<br>
other than libdrm/mesa.<br>
<br>
BR,<br>
-R<br></blockquote><div><br></div>I personally do not see the reason to break user space API in the first place.<br>I know for a fact that the include file ( linux/types.h ) mentioned by the patch<br>supports C99 integer data types since Linux Kernel Version 2.2.0. This makes<br>the patch that got reverted completely meaningless since the only reason to<br>make this change in the first place would be if the developer intends on<br>backporting the driver to kernels older than version 2.2.0. <br><br></div><div class="gmail_quote">Also the only integer types that could pose problems is if __STRICT_ANSI__<br>is defined because this would cause uint64_t, and int64_t to not be defined.<br></div><div class="gmail_quote"><br></div></div></div>