What is the purpose of this patch?
I have not really dealt with Xv or XvMC code ever since I took over the development.

(In reply to Kevin Brace from comment #1)
> What is the purpose of this patch?
> I have not really dealt with Xv or XvMC code ever since I took over the
> development.

XvMC is pretty useless today. It can only decode mpeg2.
Some users want to disable it. 
Modern video players don't even support it.

Its similar to xf86-video-intel
https://cgit.freedesktop.org/xorg/driver/xf86-video-intel/tree/configure.ac?id=028c946df0855728b2a34d1d588701dc6c5ad4c1#n369

(In reply to Harri Nieminen from comment #2)

XvMC code was written by previous developers 7 to 10 years ago.
I had no involvement with it.
Please understand that OpenChrome's project development resource is easily 1/30th of Intel graphics stack.
I am the only developer working on the code, and my hands are completely tied up for the next few years.
I am trying to fix various flaws of existing OpenChrome DDX code, and trying to fix the problems of OpenChrome DRM (KMS / TTM supporting version) so that it can be mainlined with the Linux kernel eventually.
As a result, I have virtually no time to work on issues other than those two.
I am leaning towards not incorporating this patch.
I do not see the urgency of why this patch should be incorporated in the first place.

> 
> XvMC is pretty useless today. It can only decode mpeg2.
> Some users want to disable it. 
> Modern video players don't even support it.

(In reply to Kevin Brace from comment #4)
> (In reply to Harri Nieminen from comment #2)
> 
> XvMC code was written by previous developers 7 to 10 years ago.
> I had no involvement with it.
> Please understand that OpenChrome's project development resource is easily
> 1/30th of Intel graphics stack.
> I am the only developer working on the code, and my hands are completely
> tied up for the next few years.
> I am trying to fix various flaws of existing OpenChrome DDX code, and trying
> to fix the problems of OpenChrome DRM (KMS / TTM supporting version) so that
> it can be mainlined with the Linux kernel eventually.
> As a result, I have virtually no time to work on issues other than those two.
> I am leaning towards not incorporating this patch.
> I do not see the urgency of why this patch should be incorporated in the
> first place.
> 
> > 
> > XvMC is pretty useless today. It can only decode mpeg2.
> > Some users want to disable it. 
> > Modern video players don't even support it.

Okay thanks.

Also good luck with your work. Mainlining DRM driver sounds lots of work.
This patch was very low priority for me too.