[Intel-gfx] [PATCH i-g-t] tests/gem_concurrent.c: rename to gem_concurrent_all

Chris Wilson chris at chris-wilson.co.uk
Wed Apr 1 08:45:00 PDT 2015


On Wed, Apr 01, 2015 at 04:36:59PM +0100, tim.gore at intel.com wrote:
> From: Tim Gore <tim.gore at intel.com>
> 
> Recent patch #7763349a9a87.. renamed gem_concurrent_blit.c
> to gem_concurrent.c and then added entries to Makefile.am
> to make two identical executeables (but with different
> names) from this source file. This executeable changes its
> behaviour based on argv[0]. But, this has broken the
> Android build, which does not use autotools.
> 
> This patch instead renames the source file to match
> the name of one executable (gem_concurrent_all.c) and
> creates a second source file which simply #includes the
> first. The Makefile.am entries are also removed.
> This restores the simple test.c -> test executeable
> relationship seen in the rest of IGT and allows the
> Android build system to work without parsing Makfile.am
> or having to incorporate a special workaround for this
> test.
> 
> Signed-off-by: Tim Gore <tim.gore at intel.com>

Neat, I am not even going to complain about /* */\n/* */\n/* */

Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list