Register XvMC video decoding acceleration

Michel Dänzer michel at daenzer.net
Thu Jul 14 04:23:05 PDT 2011


On Don, 2011-07-14 at 12:08 +0200, Christian König wrote: 
> Hi guys,
> 
> AFAIK Dave is on holiday, and I don't have commit rights to
> xf86-video-ati, so can anybody of you please review and commit the
> attached patch?

[...]

> diff --git a/src/radeon_video.c b/src/radeon_video.c
> index 58e3920..edd6d54 100644
> --- a/src/radeon_video.c
> +++ b/src/radeon_video.c
> @@ -315,6 +315,16 @@ void RADEONInitVideo(ScreenPtr pScreen)
>      if(num_adaptors)
>         xf86XVScreenInit(pScreen, adaptors, num_adaptors);
>  
> +    if(texturedAdaptor) {
> +       XF86MCAdaptorPtr xvmcAdaptor = RADEONCreateAdaptorXvMC(pScreen, texturedAdaptor->name);
> +       if(xvmcAdaptor) {
> +           if(!xf86XvMCScreenInit(pScreen, 1, &xvmcAdaptor))
> +               xf86DrvMsg(pScrn->scrnIndex, X_ERROR, "[XvMC] Failed to initialize extension.\n");
> +           else
> +               xf86DrvMsg(pScrn->scrnIndex, X_INFO, "[XvMC] Extension initialized.\n");
> +       }
> +    }
> +
>      if(newAdaptors)
>         free(newAdaptors);
>  

Should this only be done under circumstances where the client side XvMC
components can actually work? (KMS and >= R300 3D engine?)

What will happen if this gets initialized but the client side can't
work?

Looks good to me otherwise.


-- 
Earthling Michel Dänzer           |                   http://www.amd.com
Libre software enthusiast         |          Debian, X and DRI developer


More information about the xorg-driver-ati mailing list