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