[MPlayer-dev-eng] [PATCH]Add vo vdpau

Carl Eugen Hoyos cehoyos at ag.or.at
Mon Jan 5 10:09:55 CET 2009


Uoti Urpala <uoti.urpala <at> pp1.inet.fi> writes:

> On Mon, 2009-01-05 at 04:28 +0100, Carl Eugen Hoyos wrote:
> > Attached is a first version of vo vdpau.
> > When I shortly tested it, it seemed as fast as xv.
> > It is of course precondition for HW accelerated VDPAU decoding.
> 
> It seems to have several problems.
> All the VDPAU calls use the CHECK_ST macro with does printf+exit(1) if
> there's any problem.

Could you propose a more correct solution?

> config() silently returns success if reconfigured
> without doing anything. config() also has "vo_directrendering = 1;".
> That breakage is probably copied from vo_xvmc, but it makes even less
> sense for vdpau; there's no get_image implementation at all so the
> direct rendering code can't do anything sensible. Window creation code
> is copied from an older version of vo_xv.c from before Reimar changed
> it.

I was hoping he (or somebody else) who has more insight than myself could
quickly point me to these issues.

> There are likely more such issues.
> 
> The start_slice() function tries to disable slice support but there's
> also a draw_slice() function. Does this combination actually do
> something? At least it should be documented if so.
> 
> There's also lots of smaller stuff like bad grammar in comments, bad
> code formatting (at least some copied from vo_xv), unused variable
> "max_references"

All other issues will disappear when hardware decoding is added to vo_vdpau. If
you think it makes sense I can try to remove more of the unused stuff.

> and an mp_msg() call saying "vo_x11:".

I will fix this one.

Thank you, Carl Eugen





More information about the MPlayer-dev-eng mailing list