Re: [Mesa-dev] [PATCH] st/mesa: conditionally enable GL_NV_vdpau_interop

2016-01-22 Thread Ilia Mirkin
On Fri, Jan 22, 2016 at 2:11 PM, Christian König wrote: > Am 22.01.2016 um 19:24 schrieb Ilia Mirkin: >> >> On Fri, Jan 22, 2016 at 1:15 PM, Christian König >> wrote: Form autofoo perspective things look great. >>> >>> Thanks, that exactly what I wanted to know. >>> Although I seco

Re: [Mesa-dev] [PATCH] st/mesa: conditionally enable GL_NV_vdpau_interop

2016-01-22 Thread Christian König
Am 22.01.2016 um 19:24 schrieb Ilia Mirkin: On Fri, Jan 22, 2016 at 1:15 PM, Christian König wrote: Form autofoo perspective things look great. Thanks, that exactly what I wanted to know. Although I second Ilia's concern - we need a form of runtime detection here. Pretty much all distros shi

Re: [Mesa-dev] [PATCH] st/mesa: conditionally enable GL_NV_vdpau_interop

2016-01-22 Thread Emil Velikov
On 22 January 2016 at 18:15, Christian König wrote: >> Form autofoo perspective things look great. > > Thanks, that exactly what I wanted to know. > >> Although I second Ilia's concern - we need a form of runtime detection >> here. Pretty much all distros ship the vdpau(+ other video driver >> bac

Re: [Mesa-dev] [PATCH] st/mesa: conditionally enable GL_NV_vdpau_interop

2016-01-22 Thread Ilia Mirkin
On Fri, Jan 22, 2016 at 1:15 PM, Christian König wrote: >> Form autofoo perspective things look great. > > Thanks, that exactly what I wanted to know. > >> Although I second Ilia's concern - we need a form of runtime detection >> here. Pretty much all distros ship the vdpau(+ other video driver >>

Re: [Mesa-dev] [PATCH] st/mesa: conditionally enable GL_NV_vdpau_interop

2016-01-22 Thread Christian König
Form autofoo perspective things look great. Thanks, that exactly what I wanted to know. Although I second Ilia's concern - we need a form of runtime detection here. Pretty much all distros ship the vdpau(+ other video driver backend) in a separate library. Thus this will likely get us nowhere w

Re: [Mesa-dev] [PATCH] st/mesa: conditionally enable GL_NV_vdpau_interop

2016-01-22 Thread Emil Velikov
Form autofoo perspective things look great. Although I second Ilia's concern - we need a form of runtime detection here. Pretty much all distros ship the vdpau(+ other video driver backend) in a separate library. Thus this will likely get us nowhere we want - as I'm suspecting this is to assist th

Re: [Mesa-dev] [PATCH] st/mesa: conditionally enable GL_NV_vdpau_interop

2016-01-22 Thread Marek Olšák
Reviewed-by: Marek Olšák Emil might want to comment on the configure.ac change. Marek On Thu, Jan 21, 2016 at 5:05 PM, Christian König wrote: > From: Christian König > > Only enable it when we compile the state tracker as well. > > Signed-off-by: Christian König > --- > configure.ac

Re: [Mesa-dev] [PATCH] st/mesa: conditionally enable GL_NV_vdpau_interop

2016-01-21 Thread Christian König
Advertising NV_vdpau_interop while the backend for the driver isn't installed is fine, cause this only means that the OpenGL side has support for it. Mesa doesn't have a runtime dependency on libvdpau and to initialize NV_vdpau_interop the application must open libvdpau first. So if libvdpau

[Mesa-dev] [PATCH] st/mesa: conditionally enable GL_NV_vdpau_interop

2016-01-21 Thread Christian König
From: Christian König Only enable it when we compile the state tracker as well. Signed-off-by: Christian König --- configure.ac | 1 + src/mesa/state_tracker/st_extensions.c | 2 ++ src/mesa/state_tracker/st_vdpau.c | 6 ++ 3 files changed, 9 insertions(+) d

Re: [Mesa-dev] [PATCH] st/mesa: conditionally enable GL_NV_vdpau_interop

2016-01-21 Thread Ilia Mirkin
I wonder if something more robust is needed. Let's say a distro builds with vdpau and everything else enabled, but then ships the libraries separately... On Thu, Jan 21, 2016 at 11:05 AM, Christian König wrote: > From: Christian König > > Only enable it when we compile the state tracker as well.