Re: [PATCH 1/2] Revert "include/uapi/drm/amdgpu_drm.h: use __u32 and __u64 from <linux/types.h>"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Aug 19, 2016 at 6:54 PM, Emil Velikov <emil.l.velikov@xxxxxxxxx> wrote:
> On 19 August 2016 at 15:26, Christian König <deathsimple@xxxxxxxxxxx> wrote:
>> Am 19.08.2016 um 15:50 schrieb Marek Olšák:
>>>
>>> From: Marek Olšák <marek.olsak@xxxxxxx>
>>>
>>> This reverts commit 2ce9dde0d47f2f94ab25c73a30596a7328bcdf1f.
>>>
>>> See the comment in the code. Basically, don't do cleanups in this header.
>>>
>>> Signed-off-by: Marek Olšák <marek.olsak@xxxxxxx>
>>
>>
>> I completely agree with you that this was a bad move, but I fear that we
>> will run into opposition with that.
>>
> Please check the facts before introducing RATHER ANNOYING AND HARD TO
> READ COMMENT IN ALL CAPS.
>
> Story time:
> I was dreaming of a day were we can stop installing these headers,
> thus making deprecation a bit easier process.
> Yet after failing to convince Dave and Daniel on a number of occasions
> I've accepted that those headers _are_ here to stay. And yes they
> _are_ the UAPI, even though no applications are meant to use them but
> the libdrm 'version'.
> Thus any changes to the libdrm ones should be a mirror of the ones
> here and libdrm should _not_ differ.
>
> But let's ignore all that and imagine that those headers are _not_
> UAPI. That gives us even greater reason to _not_ use the uintx_t types
> but the kernel __uX ones. The series that did these changes had a fair
> few references why we want that.

tbh, I'm all in favor of making it easier to sync kernel headers to libdrm, etc.

But kernel *does* have stdint types.  Just (for some reason that
completely baffles me) not in stdint.h so we can't include that from
the uapi headers themselves.  I'm not a huge fan of the uX/sX types
when the rest of the world has moved on to stdint.  I can *kinda*
(barely) understand the argument for kernel using uX/sX in uapi (to
avoid other userspace src files from needing to #include stdint.h
first).  But I think that is a bad argument (kernel should fix it's
shit to be compatible with stdint.h, not other way around), and it
doesn't even apply for drm uapi (where the consumer of the uapi is
just libdrm_$drivername, not random other consumers) so we can take
care to #include stdint.h first..

tl;dr: I wasn't a big fan of the conversion to uX/sX types.. I kinda
see the argument in the general case (but I think it is bogus, and
even if it was legit it doesn't apply to drm uapi)

</rant>

BR,
-R


> Yes, I can imagine that the situation isn't ideal, and/or not that
> clear. Then again a check with git log should have straightened things
> out.
> If not _please_ help us improve this (documentation and/or others).
>
>
> And last but not least, please share with up what inspired this -
> (build/runtime) regression, attempted sync with libdrm, other ?
>
> Thanks
> Emil
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux