On 28/02/2019 14:33, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2019-02-28 14:18:28)
From: Tvrtko Ursulin <[email protected]>

Probably a leftover from test renames:

   tests/Makefile.am:134: warning: variable 'pm_rc6_residency_LDADD' is defined 
but no program or
   tests/Makefile.am:134: library has 'pm_rc6_residency' as canonical name 
(possible typo)

Signed-off-by: Tvrtko Ursulin <[email protected]>
---
  tests/Makefile.am | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/Makefile.am b/tests/Makefile.am
index c5dd210c7163..80bc5f92a13b 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -131,7 +131,7 @@ perf_pmu_LDADD = $(LDADD) $(top_builddir)/lib/libigt_perf.la
  gem_eio_LDADD = $(LDADD) -lrt
  gem_wait_LDADD = $(LDADD) -lrt
  kms_flip_LDADD = $(LDADD) -lrt -lpthread
-pm_rc6_residency_LDADD = $(LDADD) -lrt
+i915_pm_rc6_residency_LDADD = $(LDADD) -lrt

Have we not snuck -lrt into the core library yet, pretty sure libigt.la
now includes clock_gettime() and so must be pulling it in already?

i.e. can we just drop the above line?

Looks like it. And many more in this case.

Regards,

Tvrtko

_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to