Now let see how well it works with nouveau :P
I fear not at all, cause nouveau never supported chunked decoding.
Regards,
Christian.
Am 25.09.2014 um 12:21 schrieb Emil Velikov:
Hi Leo,
On 24/09/14 16:24, Liu, Leo wrote:
Hi Emil,
-----Original Message-----
From: Emil Velikov [mailto:emil.l.veli...@gmail.com]
Sent: Tuesday, September 23, 2014 3:49 PM
To: Liu, Leo; mesa-dev@lists.freedesktop.org
Cc: emil.l.veli...@gmail.com; Koenig, Christian
Subject: Re: [Mesa-dev] [PATCH 2/6] st/va: skeleton VAAPI state tracker
......
+
+gallium_drv_video_la_LIBADD = \
+ $(top_builddir)/src/gallium/state_trackers/va/libvatracker.la \
+ $(top_builddir)/src/gallium/auxiliary/libgallium.la \
+ $(top_builddir)/src/util/libmesautil.la \
+ $(LIBVA_LIBS) \
Unless I've missed something this link should not be needed. If in doubt just
omit
it, and let the _compiler_ shout about undefined symbols :)
If you talk about $(LIBVA_LIBS), we probably need it.
Why do you insist on guessing when there is someone (normally the linker
but in this case me) else to check for you :'(
$ ldd -r -u gallium_drv_video.so
Unused direct dependencies:
/usr/lib/libva.so.1
Don't mean to come as rude, but did you even build the series ? It seems
to be failing on my system.
On 24/09/14 18:46, Leo Liu wrote:
From: Christian König <christian.koe...@amd.com>
This patch adds a skeleton VA-API state tracker,
which is filled with live in the subsequent patches.
v2: fixes in configure.ac and va state_tracker Makefile.am
"va target Makefile.am" to reflect reality :)
Signed-off-by: Christian König <christian.koe...@amd.com>
Signed-off-by: Leo Liu <leo....@amd.com>
diff --git a/configure.ac b/configure.ac
index 87c616b..5d363d7 100644
--- a/configure.ac
+++ b/configure.ac
[...]
+AC_ARG_WITH([va-libdir],
+ [AS_HELP_STRING([--with-va-libdir=DIR],
+ [directory for the VA libraries])],
[directory for the VA libraries @<:@default=`pkg-config libva
--variable=driverdir`@:>@])],
[...]
--- /dev/null
+++ b/src/gallium/targets/va/Makefile.am
[...]
+gallium_drv_video_la_LIBADD = \
+ $(top_builddir)/src/gallium/state_trackers/va/libvatracker.la \
+ $(top_builddir)/src/gallium/auxiliary/libgallium.la \
+ $(top_builddir)/src/util/libmesautil.la \
+ $(VA_LIBS)
+
Apart from removing VA_LIBS above you'll need to add a library or two.
The compiler/linker will nicely point out what is needed as it builds.
The if branch of the following conditional can be rewritten in 5 lines.
Check out the omx target for an example.
+if HAVE_GALLIUM_STATIC_TARGETS
+
+MEGADRIVERS =
+STATIC_TARGET_CPPFLAGS = -DGALLIUM_STATIC_TARGETS=1
+STATIC_TARGET_LIB_DEPS = \
+ $(top_builddir)/src/loader/libloader.la
+
+if HAVE_GALLIUM_R600
+STATIC_TARGET_LIB_DEPS += \
+ $(top_builddir)/src/gallium/winsys/radeon/drm/libradeonwinsys.la
+else
+if HAVE_GALLIUM_RADEONSI
+STATIC_TARGET_LIB_DEPS += \
+ $(top_builddir)/src/gallium/winsys/radeon/drm/libradeonwinsys.la
+endif
+endif
+
...
Now let see how well it works with nouveau :P
Cheers,
Emil
I will fix the rest in the v2.
Thanks,
Leo
Cheers,
Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
