[Intel-gfx] [PATCH i-g-t] lib: use the local intel_aub.h file

Brian Starkey brian.starkey at arm.com
Thu Oct 27 08:44:18 UTC 2016


On Thu, Oct 27, 2016 at 09:28:12AM +0200, Daniel Vetter wrote:
>On Wed, Oct 26, 2016 at 05:48:39PM +0100, Emil Velikov wrote:
>> From: Emil Velikov <emil.velikov at collabora.com>
>>
>> File is provided by the libdrm_intel package which is optional. Since we
>> already have a local copy of the file, we might as well use it ;-)
>>
>> Cc: Brian Starkey <brian.starkey at arm.com>
>> Reported-by: Brian Starkey <brian.starkey at arm.com>
>> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
>> ---
>> An alternative would be to $ git mv tools/intel_aub.h lib/stubs/drm/ and
>> drop the intel_aub.h reference from tools/Makefile.am.
>
>Hm, that sounds like the approach with more consistency, but either
>approach has my ack.
>-Daniel
>

I agree that sounds better. I still needed to fix up
benchmarks/gem_exec_tracer.c on top of this patch - moving the header
would fix that and should avoid any more occurrences.

-Brian

>
>> ---
>>  lib/rendercopy_gen8.c | 2 +-
>>  lib/rendercopy_gen9.c | 2 +-
>>  2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/lib/rendercopy_gen8.c b/lib/rendercopy_gen8.c
>> index a7fc2c4..3c4a040 100644
>> --- a/lib/rendercopy_gen8.c
>> +++ b/lib/rendercopy_gen8.c
>> @@ -22,7 +22,7 @@
>>  #include "intel_reg.h"
>>  #include "igt_aux.h"
>>
>> -#include <intel_aub.h>
>> +#include "tools/intel_aub.h"
>>
>>  #define VERTEX_SIZE (3*4)
>>
>> diff --git a/lib/rendercopy_gen9.c b/lib/rendercopy_gen9.c
>> index 9537480..4fef584 100644
>> --- a/lib/rendercopy_gen9.c
>> +++ b/lib/rendercopy_gen9.c
>> @@ -23,7 +23,7 @@
>>  #include "intel_reg.h"
>>  #include "igt_aux.h"
>>
>> -#include <intel_aub.h>
>> +#include "tools/intel_aub.h"
>>
>>  #define VERTEX_SIZE (3*4)
>>
>> --
>> 2.9.3
>>
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
>-- 
>Daniel Vetter
>Software Engineer, Intel Corporation
>http://blog.ffwll.ch
>_______________________________________________
>Intel-gfx mailing list
>Intel-gfx at lists.freedesktop.org
>https://lists.freedesktop.org/mailman/listinfo/intel-gfx


More information about the Intel-gfx mailing list