call vaGetImage directly 

> This will make the conformance test fail because we need to hook 
> i965_PutSurface to dump the decoded yuv file. Is there any other method? 
> 
> Thanks
> 
> Hai Lan
> 
> > -----Original Message-----
> > From: libva-bounces+hai.lan=intel....@lists.freedesktop.org
> > [mailto:libva-bounces+hai.lan=intel....@lists.freedesktop.org] On Behalf
> > Of Gwenole Beauchesne
> > Sent: Tuesday, July 26, 2011 7:48 PM
> > To: libva@lists.freedesktop.org
> > Subject: [Libva] [PATCH] i965_drv_video: don't export internal driver
> > functions.
> > 
> > From: Gwenole Beauchesne <gwenole.beauche...@intel.com>
> > 
> > Make sure to use our internal functions. In particular, we override
> > some DRM functions and they have to be used. e.g. this fixes VA/GLX.
> > ---
> >  i965_drv_video/Makefile.am      |    7 ++++++-
> >  i965_drv_video/i965_drv_video.c |    3 +++
> >  i965_drv_video/intel_compiler.h |   11 +++++++++++
> >  3 files changed, 20 insertions(+), 1 deletions(-)
> > 
> > diff --git a/i965_drv_video/Makefile.am b/i965_drv_video/Makefile.am
> > index c106aae..405489b 100644
> > --- a/i965_drv_video/Makefile.am
> > +++ b/i965_drv_video/Makefile.am
> > @@ -23,7 +23,6 @@
> >  SUBDIRS = shaders
> > 
> >  INCLUDES = \
> > -   -Wall                   \
> >     -I$(top_srcdir)         \
> >     -I$(top_srcdir)/va      \
> >     -I$(top_srcdir)/va/x11  \
> > @@ -31,6 +30,11 @@ INCLUDES = \
> >     $(DRM_CFLAGS)           \
> >     $(NULL)
> > 
> > +driver_cflags = \
> > +   -Wall                   \
> > +   -fvisibility=hidden     \
> > +   $(NULL)
> > +
> >  driver_ldflags = \
> >     -module -avoid-version  \
> >     -no-undefined           \
> > @@ -88,6 +92,7 @@ source_h = \
> > 
> >  i965_drv_video_la_LTLIBRARIES      = i965_drv_video.la
> >  i965_drv_video_ladir               = $(LIBVA_DRIVERS_PATH)
> > +i965_drv_video_la_CFLAGS   = $(driver_cflags)
> >  i965_drv_video_la_LDFLAGS  = $(driver_ldflags)
> >  i965_drv_video_la_LIBADD   = $(top_builddir)/va/libva-x11.la -lpthread
> >  i965_drv_video_la_SOURCES  = $(source_c)
> > diff --git a/i965_drv_video/i965_drv_video.c
> > b/i965_drv_video/i965_drv_video.c
> > index 26b07a2..72f82b7 100644
> > --- a/i965_drv_video/i965_drv_video.c
> > +++ b/i965_drv_video/i965_drv_video.c
> > @@ -2522,6 +2522,9 @@ i965_UnlockSurface(
> >      return vaStatus;
> >  }
> > 
> > +VAStatus DLL_EXPORT
> > +VA_DRIVER_INIT_FUNC(VADriverContextP ctx);
> > +
> >  VAStatus
> >  VA_DRIVER_INIT_FUNC(  VADriverContextP ctx )
> >  {
> > diff --git a/i965_drv_video/intel_compiler.h
> > b/i965_drv_video/intel_compiler.h
> > index f1f24c8..e1c3bd2 100644
> > --- a/i965_drv_video/intel_compiler.h
> > +++ b/i965_drv_video/intel_compiler.h
> > @@ -12,4 +12,15 @@
> >  #  define INLINE
> >  #endif
> > 
> > +/**
> > + * Function visibility
> > + */
> > +#if defined(__GNUC__)
> > +#  define DLL_HIDDEN __attribute__((visibility("hidden")))
> > +#  define DLL_EXPORT __attribute__((visibility("default")))
> > +#else
> > +#  define DLL_HIDDEN
> > +#  define DLL_EXPORT
> > +#endif
> > +
> >  #endif /* _INTEL_COMPILER_H_ */
> > --
> > 1.7.4.1
> > 
> > _______________________________________________
> > Libva mailing list
> > Libva@lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/libva
> _______________________________________________
> Libva mailing list
> Libva@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libva


_______________________________________________
Libva mailing list
Libva@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libva

Reply via email to