[PATCH libdrm 2/7] amdgpu: update amdgpu_drm.h for Vega10

Marek Olšák maraeo at gmail.com
Wed Mar 22 10:13:31 UTC 2017

On Mar 22, 2017 2:44 AM, "Michel Dänzer" <michel at daenzer.net> wrote:

On 22/03/17 06:46 AM, Marek Olšák wrote:
> On Tue, Mar 21, 2017 at 10:27 PM, Nicolai Hähnle <nhaehnle at gmail.com>
>> In the past, I was told off for patches that update this file without
>> following the procedure described in include/drm/README. Tbh, that
>> causes some annoyances.
>> Anyway, it's definitely useful to have the patch out on the mailing list
>> any case.
> Yeah, I know the correct process and I plan to ignore it this time if
> I don't get too much backlash, because the alternative
> (#ifdef/#define/#endif) is probably even worse.

FWIW, only AMDGPU_TILING_SET/GET need #undef,
#defined directly, that way the preprocessor will warn if the
definitions in libdrm and Mesa end up being inconsistent for some reason.

The alternative is rushing out a libdrm release and making Mesa require
that, right? That doesn't seem obviously better than a handful of
temporary redundant defines in Mesa, hardly justification for bypassing
the normal process.

I need a libdrm release because of the 3rd patch. I can't allow Mesa to run
without that.


Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20170322/a60d7ae6/attachment.html>

More information about the amd-gfx mailing list