Re: [Mesa-dev] [PATCH] st/vdpau: remove nouveau target

2016-09-15 Thread Ilia Mirkin
On Sep 15, 2016 6:41 AM, "Emil Velikov"  wrote:
>
> On 15 September 2016 at 10:56, Emil Velikov 
wrote:
> > On 15 September 2016 at 02:54, Ilia Mirkin  wrote:
> >> On Wed, Sep 14, 2016 at 9:52 PM, Ilia Mirkin 
wrote:
> >>> Recent changes have been made to the VDPAU state tracker to make it
> >>> unusable with nouveau. Don't provide users with an awfully slow
> >>> "hardware" decoding option.
> >>>
> >>> [To preemptively answer the question that will invariably be asked -
> >>> this is due to the state tracker's use of PIPE_BIND_SHARED, which
> >>> nouveau uses to force GART placement to make things with with PRIME.
> >>> However when this is used for output surfaces, which are then blended
on
> >>> (the most common way of implementing an OSD), this results in
> >>> *incredibly* slow operation.]
> >>>
> >>> Signed-off-by: Ilia Mirkin 
> >>
> >> Oops, meant to add a CC to mesa-stable, since the breakage was
> >> introduced in 12.0.
> >>
> > Just to double-check - by "breakage" and "unusable" you mean that it's
> > "awfully slow" correct ?

Yeah. Nowhere close to able to keep up if blending is enabled, which
happens whenever an osd is displayed.

> >
> > Do you/others have any plan to update the state-tracker and/or nouveau
> > to brings things back on par ?

No plans from me.

> >
> Scratch this question - I've just noticed the VDPAU patch which makes
> this one obsolete.

Either this or a "make it work again" patch needs to land.

  -ilia
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH] st/vdpau: remove nouveau target

2016-09-15 Thread Emil Velikov
On 15 September 2016 at 10:56, Emil Velikov  wrote:
> On 15 September 2016 at 02:54, Ilia Mirkin  wrote:
>> On Wed, Sep 14, 2016 at 9:52 PM, Ilia Mirkin  wrote:
>>> Recent changes have been made to the VDPAU state tracker to make it
>>> unusable with nouveau. Don't provide users with an awfully slow
>>> "hardware" decoding option.
>>>
>>> [To preemptively answer the question that will invariably be asked -
>>> this is due to the state tracker's use of PIPE_BIND_SHARED, which
>>> nouveau uses to force GART placement to make things with with PRIME.
>>> However when this is used for output surfaces, which are then blended on
>>> (the most common way of implementing an OSD), this results in
>>> *incredibly* slow operation.]
>>>
>>> Signed-off-by: Ilia Mirkin 
>>
>> Oops, meant to add a CC to mesa-stable, since the breakage was
>> introduced in 12.0.
>>
> Just to double-check - by "breakage" and "unusable" you mean that it's
> "awfully slow" correct ?
>
> Do you/others have any plan to update the state-tracker and/or nouveau
> to brings things back on par ?
>
Scratch this question - I've just noticed the VDPAU patch which makes
this one obsolete.

-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH] st/vdpau: remove nouveau target

2016-09-15 Thread Emil Velikov
On 15 September 2016 at 02:54, Ilia Mirkin  wrote:
> On Wed, Sep 14, 2016 at 9:52 PM, Ilia Mirkin  wrote:
>> Recent changes have been made to the VDPAU state tracker to make it
>> unusable with nouveau. Don't provide users with an awfully slow
>> "hardware" decoding option.
>>
>> [To preemptively answer the question that will invariably be asked -
>> this is due to the state tracker's use of PIPE_BIND_SHARED, which
>> nouveau uses to force GART placement to make things with with PRIME.
>> However when this is used for output surfaces, which are then blended on
>> (the most common way of implementing an OSD), this results in
>> *incredibly* slow operation.]
>>
>> Signed-off-by: Ilia Mirkin 
>
> Oops, meant to add a CC to mesa-stable, since the breakage was
> introduced in 12.0.
>
Just to double-check - by "breakage" and "unusable" you mean that it's
"awfully slow" correct ?

Do you/others have any plan to update the state-tracker and/or nouveau
to brings things back on par ?

Thanks
Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH] st/vdpau: remove nouveau target

2016-09-14 Thread Ilia Mirkin
On Wed, Sep 14, 2016 at 9:52 PM, Ilia Mirkin  wrote:
> Recent changes have been made to the VDPAU state tracker to make it
> unusable with nouveau. Don't provide users with an awfully slow
> "hardware" decoding option.
>
> [To preemptively answer the question that will invariably be asked -
> this is due to the state tracker's use of PIPE_BIND_SHARED, which
> nouveau uses to force GART placement to make things with with PRIME.
> However when this is used for output surfaces, which are then blended on
> (the most common way of implementing an OSD), this results in
> *incredibly* slow operation.]
>
> Signed-off-by: Ilia Mirkin 

Oops, meant to add a CC to mesa-stable, since the breakage was
introduced in 12.0.

> ---
>  src/gallium/targets/vdpau/Makefile.am | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/src/gallium/targets/vdpau/Makefile.am 
> b/src/gallium/targets/vdpau/Makefile.am
> index d388f8b..9549a23 100644
> --- a/src/gallium/targets/vdpau/Makefile.am
> +++ b/src/gallium/targets/vdpau/Makefile.am
> @@ -49,8 +49,6 @@ TARGET_DRIVERS =
>  TARGET_CPPFLAGS =
>  TARGET_LIB_DEPS =
>
> -include $(top_srcdir)/src/gallium/drivers/nouveau/Automake.inc
> -
>  include $(top_srcdir)/src/gallium/drivers/r300/Automake.inc
>  include $(top_srcdir)/src/gallium/drivers/r600/Automake.inc
>  include $(top_srcdir)/src/gallium/drivers/radeonsi/Automake.inc
> --
> 2.7.3
>
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev


[Mesa-dev] [PATCH] st/vdpau: remove nouveau target

2016-09-14 Thread Ilia Mirkin
Recent changes have been made to the VDPAU state tracker to make it
unusable with nouveau. Don't provide users with an awfully slow
"hardware" decoding option.

[To preemptively answer the question that will invariably be asked -
this is due to the state tracker's use of PIPE_BIND_SHARED, which
nouveau uses to force GART placement to make things with with PRIME.
However when this is used for output surfaces, which are then blended on
(the most common way of implementing an OSD), this results in
*incredibly* slow operation.]

Signed-off-by: Ilia Mirkin 
---
 src/gallium/targets/vdpau/Makefile.am | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/gallium/targets/vdpau/Makefile.am 
b/src/gallium/targets/vdpau/Makefile.am
index d388f8b..9549a23 100644
--- a/src/gallium/targets/vdpau/Makefile.am
+++ b/src/gallium/targets/vdpau/Makefile.am
@@ -49,8 +49,6 @@ TARGET_DRIVERS =
 TARGET_CPPFLAGS =
 TARGET_LIB_DEPS =
 
-include $(top_srcdir)/src/gallium/drivers/nouveau/Automake.inc
-
 include $(top_srcdir)/src/gallium/drivers/r300/Automake.inc
 include $(top_srcdir)/src/gallium/drivers/r600/Automake.inc
 include $(top_srcdir)/src/gallium/drivers/radeonsi/Automake.inc
-- 
2.7.3

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev